Author: kstam
Date: Mon Dec 13 21:39:46 2010
New Revision: 1045383

URL: http://svn.apache.org/viewvc?rev=1045383&view=rev
Log:
JUDDI-437, fixing potential NPE if no custom config files are found.

Modified:
    juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java

Modified: 
juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java
URL: 
http://svn.apache.org/viewvc/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java?rev=1045383&r1=1045382&r2=1045383&view=diff
==============================================================================
--- juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java 
(original)
+++ juddi/trunk/juddi-core/src/main/java/org/apache/juddi/config/Install.java 
Mon Dec 13 21:39:46 2010
@@ -539,17 +539,20 @@ public class Install {
                                                en = new JarFile(new 
File(paths[0])).entries();
                                        }
                                }
-                       
-                               log.debug("Discovering the Publisher XML data 
files in jar: " + paths[0]);
-                               while (en.hasMoreElements()) {
-                                       String name = 
en.nextElement().getName();
-                                       if (name.endsWith(FILE_PUBLISHER)) {
-                                               log.debug("Found publisher 
file=" + name);
-                                               String publisher = 
name.substring(basePath.length(),name.indexOf(FILE_PUBLISHER));
-                                       if (! 
rootPublisherStr.equalsIgnoreCase(publisher)) {
-                                               publishers.add(publisher);
-                                       } 
+                               if (paths.length > 0) {
+                                       log.debug("Discovering the Publisher 
XML data files in jar: " + paths[0]);
+                                       while (en.hasMoreElements()) {
+                                               String name = 
en.nextElement().getName();
+                                               if 
(name.endsWith(FILE_PUBLISHER)) {
+                                                       log.debug("Found 
publisher file=" + name);
+                                                       String publisher = 
name.substring(basePath.length(),name.indexOf(FILE_PUBLISHER));
+                                               if (! 
rootPublisherStr.equalsIgnoreCase(publisher)) {
+                                                       
publishers.add(publisher);
+                                               } 
+                                               }
                                        }
+                               } else {
+                                       log.info("No custom configuration files 
where found in " + path);
                                }
                        } catch (IOException e) {
                                throw new ConfigurationException(e);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to