diff options
| author | Abhishek Koneru <akoneru@redhat.com> | 2012-05-23 13:46:25 -0400 |
|---|---|---|
| committer | Ade Lee <alee@redhat.com> | 2012-05-24 23:09:24 -0400 |
| commit | 2408bec41a56378fcf942a68a1ab290464c001d7 (patch) | |
| tree | db5b4ba123c2023c854882188851a42fd35348bf /base/common/src/com/netscape/cmscore/security/JssSubsystem.java | |
| parent | be23c86db27b7f66ca16a10ae3ef3f043d1d4b27 (diff) | |
| download | pki-2408bec41a56378fcf942a68a1ab290464c001d7.tar.gz pki-2408bec41a56378fcf942a68a1ab290464c001d7.tar.xz pki-2408bec41a56378fcf942a68a1ab290464c001d7.zip | |
Patch for fixes for Review Comments
Diffstat (limited to 'base/common/src/com/netscape/cmscore/security/JssSubsystem.java')
| -rw-r--r-- | base/common/src/com/netscape/cmscore/security/JssSubsystem.java | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/base/common/src/com/netscape/cmscore/security/JssSubsystem.java b/base/common/src/com/netscape/cmscore/security/JssSubsystem.java index 2d6fe7d88..12bf8d95a 100644 --- a/base/common/src/com/netscape/cmscore/security/JssSubsystem.java +++ b/base/common/src/com/netscape/cmscore/security/JssSubsystem.java @@ -539,7 +539,7 @@ public final class JssSubsystem implements ICryptoSubsystem { if (tokenList.length()==0) return Constants.PR_INTERNAL_TOKEN; else - return tokenList.append(",").append(Constants.PR_INTERNAL_TOKEN).toString(); + return tokenList.append("," + Constants.PR_INTERNAL_TOKEN).toString(); } public boolean isTokenLoggedIn(String name) throws EBaseException { @@ -610,9 +610,11 @@ public final class JssSubsystem implements ICryptoSubsystem { for (int i = 0; i < list.length; i++) { String nickname = list[i].getNickname(); - if (i > 0) - certNames.append(","); - certNames.append(nickname); + if (certNames.length() < 1) { + certNames.append(nickname); + } else { + certNames.append("," + nickname); + } } } } catch (TokenException e) { |