You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ff...@apache.org on 2009/09/27 06:51:04 UTC
svn commit: r819257 -
/felix/trunk/karaf/features/core/src/main/java/org/apache/felix/karaf/features/internal/FeaturesServiceImpl.java
Author: ffang
Date: Sun Sep 27 04:51:04 2009
New Revision: 819257
URL: http://svn.apache.org/viewvc?rev=819257&view=rev
Log:
[FELIX-1645] add null and empty check for FeaturesServiceImpl.readValue(String val)
Modified:
felix/trunk/karaf/features/core/src/main/java/org/apache/felix/karaf/features/internal/FeaturesServiceImpl.java
Modified: felix/trunk/karaf/features/core/src/main/java/org/apache/felix/karaf/features/internal/FeaturesServiceImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/karaf/features/core/src/main/java/org/apache/felix/karaf/features/internal/FeaturesServiceImpl.java?rev=819257&r1=819256&r2=819257&view=diff
==============================================================================
--- felix/trunk/karaf/features/core/src/main/java/org/apache/felix/karaf/features/internal/FeaturesServiceImpl.java (original)
+++ felix/trunk/karaf/features/core/src/main/java/org/apache/felix/karaf/features/internal/FeaturesServiceImpl.java Sun Sep 27 04:51:04 2009
@@ -561,8 +561,10 @@
protected Set<Long> readValue(String val) {
Set<Long> set = new HashSet<Long>();
- for (String str : val.split(",")) {
- set.add(Long.parseLong(str));
+ if (val != null && val.length() != 0) {
+ for (String str : val.split(",")) {
+ set.add(Long.parseLong(str));
+ }
}
return set;
}