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;


Reply via email to