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:44:30 UTC
svn commit: r427563 -
/geronimo/trunk/applications/console/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java
Author: jbohn
Date: Tue Aug 1 05:44:29 2006
New Revision: 427563
URL: http://svn.apache.org/viewvc?rev=427563&view=rev
Log:
GERONIMO-2218 fix bug in prior commit with delete processing
Modified:
geronimo/trunk/applications/console/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java
Modified: geronimo/trunk/applications/console/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java
URL: http://svn.apache.org/viewvc/geronimo/trunk/applications/console/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java?rev=427563&r1=427562&r2=427563&view=diff
==============================================================================
--- geronimo/trunk/applications/console/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java (original)
+++ geronimo/trunk/applications/console/console-standard/src/java/org/apache/geronimo/console/keystores/BaseKeystoreHandler.java Tue Aug 1 05:44:29 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);