You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2003/06/24 23:09:15 UTC

cvs commit: jakarta-james/src/java/org/apache/james James.java

noel        2003/06/24 14:09:15

  Modified:    src/java/org/apache/james James.java
  Log:
  Handle null return from getAllRecipients()
  
  Revision  Changes    Path
  1.60      +15 -11    jakarta-james/src/java/org/apache/james/James.java
  
  Index: James.java
  ===================================================================
  RCS file: /home/cvs/jakarta-james/src/java/org/apache/james/James.java,v
  retrieving revision 1.59
  retrieving revision 1.60
  diff -u -r1.59 -r1.60
  --- James.java	23 May 2003 17:22:01 -0000	1.59
  +++ James.java	24 Jun 2003 21:09:15 -0000	1.60
  @@ -416,11 +416,13 @@
           MailAddress sender = new MailAddress((InternetAddress)message.getFrom()[0]);
           Collection recipients = new HashSet();
           Address addresses[] = message.getAllRecipients();
  -        for (int i = 0; i < addresses.length; i++) {
  -            // Javamail treats the "newsgroups:" header field as a
  -            // recipient, so we want to filter those out.
  -            if ( addresses[i] instanceof InternetAddress ) {
  -                recipients.add(new MailAddress((InternetAddress)addresses[i]));
  +        if (addresses != null) {
  +            for (int i = 0; i < addresses.length; i++) {
  +                // Javamail treats the "newsgroups:" header field as a
  +                // recipient, so we want to filter those out.
  +                if ( addresses[i] instanceof InternetAddress ) {
  +                    recipients.add(new MailAddress((InternetAddress)addresses[i]));
  +                }
               }
           }
           sendMail(sender, recipients, message);
  @@ -673,11 +675,13 @@
           //Create the list of recipients in our MailAddress format
           Collection recipients = new HashSet();
           Address addresses[] = reply.getAllRecipients();
  -        for (int i = 0; i < addresses.length; i++) {
  -            // Javamail treats the "newsgroups:" header field as a
  -            // recipient, so we want to filter those out.
  -            if ( addresses[i] instanceof InternetAddress ) {
  -                recipients.add(new MailAddress((InternetAddress)addresses[i]));
  +        if (addresses != null) {
  +            for (int i = 0; i < addresses.length; i++) {
  +                // Javamail treats the "newsgroups:" header field as a
  +                // recipient, so we want to filter those out.
  +                if ( addresses[i] instanceof InternetAddress ) {
  +                    recipients.add(new MailAddress((InternetAddress)addresses[i]));
  +                }
               }
           }
           //Change the sender...
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: james-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: james-dev-help@jakarta.apache.org