You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@river.apache.org by pe...@apache.org on 2013/02/20 10:25:11 UTC
svn commit: r1448054 -
/river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/share/BaseQATest.java
Author: peter_firmstone
Date: Wed Feb 20 09:25:10 2013
New Revision: 1448054
URL: http://svn.apache.org/r1448054
Log:
Fix NPE
Modified:
river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/share/BaseQATest.java
Modified: river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/share/BaseQATest.java
URL: http://svn.apache.org/viewvc/river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/share/BaseQATest.java?rev=1448054&r1=1448053&r2=1448054&view=diff
==============================================================================
--- river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/share/BaseQATest.java (original)
+++ river/jtsk/skunk/qa_refactor/trunk/qa/src/com/sun/jini/test/share/BaseQATest.java Wed Feb 20 09:25:10 2013
@@ -1551,7 +1551,9 @@ abstract public class BaseQATest extends
String[] groups = (String[])pair.getValue();
logger.log(Level.FINE,
" discoveredMap.locator = "+loc);
- if( groups.length <= 0 ) {
+ if (groups == null) {
+ logger.log(Level.FINE, " discoveredMap.groups is null");
+ } else if( groups.length <= 0 ) {
logger.log(Level.FINE,
" discoveredMap.groups == NO_GROUPS");
} else {