Author: sebb
Date: Fri Mar 11 19:23:04 2011
New Revision: 1080731

URL: http://svn.apache.org/viewvc?rev=1080731&view=rev
Log:
Avoid possible NPE (Findbugs)

Modified:
    commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java

Modified: commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java
URL: 
http://svn.apache.org/viewvc/commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java?rev=1080731&r1=1080730&r2=1080731&view=diff
==============================================================================
--- commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java 
(original)
+++ commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java Fri Mar 
11 19:23:04 2011
@@ -88,14 +88,13 @@ public final class SMTPMail
                 System.out.print("CC <enter one address per line, hit enter to 
end>: ");
                 System.out.flush();
 
-                // Of course you don't want to do this because readLine() may 
be null
-                cc = stdin.readLine().trim();
+                cc = stdin.readLine();
 
-                if (cc.length() == 0)
+                if (cc== null || cc.length() == 0)
                     break;
 
-                header.addCC(cc);
-                ccList.add(cc);
+                header.addCC(cc.trim());
+                ccList.add(cc.trim());
             }
 
             System.out.print("Filename: ");


Reply via email to