You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jb...@apache.org on 2006/08/01 14:43:55 UTC

svn commit: r427562 - /geronimo/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java

Author: jbohn
Date: Tue Aug  1 05:43:55 2006
New Revision: 427562

URL: http://svn.apache.org/viewvc?rev=427562&view=rev
Log:
GERONIMO-2218 fix bug in prior commit with delete processing

Modified:
    geronimo/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java

Modified: geronimo/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java
URL: http://svn.apache.org/viewvc/geronimo/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java?rev=427562&r1=427561&r2=427562&view=diff
==============================================================================
--- geronimo/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java (original)
+++ geronimo/branches/1.1/applications/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java Tue Aug  1 05:43:55 2006
@@ -212,7 +212,7 @@
             for(int i = 0; i < keys.length; ++i) {
                 if(keys[i].equals(alias)) {
                     String[] temp = new String[keys.length-1];
-                    for(int j = 0; j < i-1; ++j) {
+                    for(int j = 0; j < i; ++j) {
                         temp[j] = keys[j];
                     }
                     for(int j = i+1; j < keys.length; ++j) {
@@ -226,13 +226,14 @@
             for(int i = 0; i < certificates.length; ++i) {
                 if(certificates[i].equals(alias)) {
                     String[] temp = new String[certificates.length-1];
-                    for(int j = 0; j < i-1; ++j) {
+                    for(int j = 0; j < i; ++j) {
                         temp[j] = certificates[j];
                     }
                     for(int j = i+1; j < certificates.length; ++j) {
                         temp[j-1] = certificates[j];
                     }
                     certificates = temp;
+                    break;
                 }
             }
             instance.deleteEntry(alias);