[prev in list] [next in list] [prev in thread] [next in thread] 

List:       squirrelmail-cvs
Subject:    [SM-CVS] SF.net SVN: squirrelmail:[14965] trunk/squirrelmail/class/deliver/ Deliver_SMTP.class.php
From:       pdontthink--- via squirrelmail-cvs <squirrelmail-cvs () lists ! sourceforge ! net>
Date:       2022-05-24 8:26:46
Message-ID: 1653380806.104467.30912 () sfp-scm-1 ! v30 ! lw ! sourceforge ! com
[Download RAW message or body]

Revision: 14965
          http://sourceforge.net/p/squirrelmail/code/14965
Author:   pdontthink
Date:     2022-05-24 08:26:45 +0000 (Tue, 24 May 2022)
Log Message:
-----------
Fix PHP notice when property doesn't exist

Modified Paths:
--------------
    trunk/squirrelmail/class/deliver/Deliver_SMTP.class.php

Modified: trunk/squirrelmail/class/deliver/Deliver_SMTP.class.php
===================================================================
--- trunk/squirrelmail/class/deliver/Deliver_SMTP.class.php	2022-05-24 04:43:39 UTC \
                (rev 14964)
+++ trunk/squirrelmail/class/deliver/Deliver_SMTP.class.php	2022-05-24 08:26:45 UTC \
(rev 14965) @@ -347,7 +347,8 @@
             if (!$to[$i]->host) $to[$i]->host = $domain;
             if (strlen($to[$i]->mailbox)) {
                 // Ask for DSN if user has requested such and remote server supports \
                it
-                if ($rfc822_header->dsn && array_key_exists('DSN',$this->ehlo)) {
+                if (isset($rfc822_header->dsn) && $rfc822_header->dsn
+                 && array_key_exists('DSN',$this->ehlo)) {
                     // TODO: Make the DSN parameters configurable by admin? user?
                     fputs($stream, 'RCPT TO:<'.$to[$i]->mailbox.'@'.$to[$i]->host."> \
NOTIFY=SUCCESS,DELAY,FAILURE\r\n");  // Retry without DSN fields for cranky MTAs
@@ -368,7 +369,8 @@
             if (!$cc[$i]->host) $cc[$i]->host = $domain;
             if (strlen($cc[$i]->mailbox)) {
                 // Ask for DSN if user has requested such and remote server supports \
                it
-                if ($rfc822_header->dsn && array_key_exists('DSN',$this->ehlo)) {
+                if (isset($rfc822_header->dsn) && $rfc822_header->dsn
+                 && array_key_exists('DSN',$this->ehlo)) {
                     // TODO: Make the DSN parameters configurable by admin? user?
                     fputs($stream, 'RCPT TO:<'.$cc[$i]->mailbox.'@'.$cc[$i]->host."> \
NOTIFY=SUCCESS,DELAY,FAILURE\r\n");  // Retry without DSN fields for cranky MTAs
@@ -389,7 +391,8 @@
             if (!$bcc[$i]->host) $bcc[$i]->host = $domain;
             if (strlen($bcc[$i]->mailbox)) {
                 // Ask for DSN if user has requested such and remote server supports \
                it
-                if ($rfc822_header->dsn && array_key_exists('DSN',$this->ehlo)) {
+                if (isset($rfc822_header->dsn) && $rfc822_header->dsn
+                 && array_key_exists('DSN',$this->ehlo)) {
                     // TODO: Make the DSN parameters configurable by admin? user?
                     fputs($stream, 'RCPT \
TO:<'.$bcc[$i]->mailbox.'@'.$bcc[$i]->host."> NOTIFY=SUCCESS,DELAY,FAILURE\r\n");  // \
Retry without DSN fields for cranky MTAs

This was sent by the SourceForge.net collaborative development platform, the world's \
largest Open Source development site.



-----
squirrelmail-cvs mailing list
List address: squirrelmail-cvs@lists.sourceforge.net
List info (subscribe/unsubscribe/change options): \
                https://lists.sourceforge.net/lists/listinfo/squirrelmail-cvs
Repository: http://squirrelmail.org/svn


[prev in list] [next in list] [prev in thread] [next in thread]