Author: sebb Date: Sat Apr 30 20:14:08 2016 New Revision: 1741808 URL: http://svn.apache.org/viewvc?rev=1741808&view=rev Log: Findbugs: possible NPE
Modified: commons/proper/net/trunk/src/main/java/examples/nntp/PostMessage.java Modified: commons/proper/net/trunk/src/main/java/examples/nntp/PostMessage.java URL: http://svn.apache.org/viewvc/commons/proper/net/trunk/src/main/java/examples/nntp/PostMessage.java?rev=1741808&r1=1741807&r2=1741808&view=diff ============================================================================== --- commons/proper/net/trunk/src/main/java/examples/nntp/PostMessage.java (original) +++ commons/proper/net/trunk/src/main/java/examples/nntp/PostMessage.java Sat Apr 30 20:14:08 2016 @@ -85,8 +85,12 @@ public final class PostMessage System.out.print("Additional Newsgroup <Hit enter to end>: "); System.out.flush(); - // Of course you don't want to do this because readLine() may be null - newsgroup = stdin.readLine().trim(); + newsgroup = stdin.readLine(); + if (newsgroup == null) { + break; + } + + newsgroup = newsgroup.trim(); if (newsgroup.length() == 0) { break;