diff options
author | Abhishek Koneru <akoneru@redhat.com> | 2012-05-18 13:17:34 -0400 |
---|---|---|
committer | Ade Lee <alee@redhat.com> | 2012-05-24 23:08:28 -0400 |
commit | a4db0f39e257950a5c89203452c1184c7080e5bd (patch) | |
tree | 2d3ebb818945db19450ed7c51e1c0e6b1582e6bc /base/util/src/netscape/security | |
parent | 307199314968b85059de62fbb694bd62b2502f46 (diff) | |
download | pki-a4db0f39e257950a5c89203452c1184c7080e5bd.tar.gz pki-a4db0f39e257950a5c89203452c1184c7080e5bd.tar.xz pki-a4db0f39e257950a5c89203452c1184c7080e5bd.zip |
Fixes for Coverity Defects of Category : FB.DM_NUMBER_CTOR, FB.DM_STRING_CTOR, FB.DM_STRING_VOID_CTOR
Diffstat (limited to 'base/util/src/netscape/security')
3 files changed, 9 insertions, 10 deletions
diff --git a/base/util/src/netscape/security/extensions/ExtendedKeyUsageExtension.java b/base/util/src/netscape/security/extensions/ExtendedKeyUsageExtension.java index 072a7c974..7c9f8450d 100644 --- a/base/util/src/netscape/security/extensions/ExtendedKeyUsageExtension.java +++ b/base/util/src/netscape/security/extensions/ExtendedKeyUsageExtension.java @@ -132,18 +132,18 @@ public class ExtendedKeyUsageExtension extends Extension implements CertAttrSet } public String toString() { + String presentation = "oid=" + ExtendedKeyUsageExtension.OID + " "; if (critical) { presentation += "critical=true"; } if (extensionValue != null) { - String extByteValue = new String(" val="); - + StringBuffer extByteValue = new StringBuffer(" val="); for (int i = 0; i < extensionValue.length; i++) { - extByteValue += (extensionValue[i] + " "); + extByteValue.append(extensionValue[i] + " "); } - presentation += extByteValue; + presentation += extByteValue.toString(); } return presentation; } diff --git a/base/util/src/netscape/security/extensions/InhibitAnyPolicyExtension.java b/base/util/src/netscape/security/extensions/InhibitAnyPolicyExtension.java index 5295cf6a1..181360f52 100644 --- a/base/util/src/netscape/security/extensions/InhibitAnyPolicyExtension.java +++ b/base/util/src/netscape/security/extensions/InhibitAnyPolicyExtension.java @@ -110,7 +110,7 @@ public class InhibitAnyPolicyExtension presentation += "Criticality=false"; } if (extensionValue != null) { - String extByteValue = new String(" skipCerts=" + mSkipCerts); + String extByteValue = " skipCerts=" + mSkipCerts; presentation += extByteValue; } diff --git a/base/util/src/netscape/security/extensions/OCSPNoCheckExtension.java b/base/util/src/netscape/security/extensions/OCSPNoCheckExtension.java index bd6cf5aa7..323cfbdbe 100644 --- a/base/util/src/netscape/security/extensions/OCSPNoCheckExtension.java +++ b/base/util/src/netscape/security/extensions/OCSPNoCheckExtension.java @@ -96,18 +96,17 @@ public class OCSPNoCheckExtension extends Extension implements CertAttrSet { } public String toString() { - String presentation = "oid=" + OID + " "; + String presentation = "oid=" + ExtendedKeyUsageExtension.OID + " "; if (critical) { presentation += "critical=true"; } if (extensionValue != null) { - String extByteValue = new String(" val="); - + StringBuffer extByteValue = new StringBuffer(" val="); for (int i = 0; i < extensionValue.length; i++) { - extByteValue += (extensionValue[i] + " "); + extByteValue.append(extensionValue[i] + " "); } - presentation += extByteValue; + presentation += extByteValue.toString(); } return presentation; } |