commit 208e8af1160c871a98547aa02a2be048a8d18530
author Sebastian Bazley <sebb@apache.org> Sat Apr 30 20:14:08 2016 +0000
committer Sebastian Bazley <sebb@apache.org> Sat Apr 30 20:14:08 2016 +0000
tree 59b3d8c3a2f8672749fafcfb05ffbe7dd14f00d3
parent f42e49c710e732f9be1efe22568f72d6db699192

Findbugs: possible NPE

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/net/trunk@1741808 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/examples/nntp/PostMessage.java

diff --git a/src/main/java/examples/nntp/PostMessage.java b/src/main/java/examples/nntp/PostMessage.java
index 260adb5..c9583c5 100644
--- a/src/main/java/examples/nntp/PostMessage.java
+++ b/src/main/java/examples/nntp/PostMessage.java
@@ -85,8 +85,12 @@
                 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;