You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by fm...@apache.org on 2009/05/18 08:09:02 UTC
svn commit: r775823 -
/felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java
Author: fmeschbe
Date: Mon May 18 06:09:02 2009
New Revision: 775823
URL: http://svn.apache.org/viewvc?rev=775823&view=rev
Log:
FELIX-1164 ensure maxSize is not negative before resizing the vector
Modified:
felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java
Modified: felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java?rev=775823&r1=775822&r2=775823&view=diff
==============================================================================
--- felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java (original)
+++ felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java Mon May 18 06:09:02 2009
@@ -814,9 +814,10 @@
}
}
- // but ensure size
+ // but ensure size (check for positive value since
+ // abs(Integer.MIN_VALUE) is still INTEGER.MIN_VALUE)
int maxSize = Math.abs( ad.getCardinality() );
- if ( vec.size() > maxSize )
+ if ( vec.size() > maxSize && maxSize > 0 )
{
vec.setSize( maxSize );
}