phplist

NOTE:: Before reporting an issue, make sure you are running the latest version, currently 3.3.1


View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0016784phplist applicationAll Otherpublic02-05-13 17:0402-05-13 22:10
Reporterduncanc 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version2.11.9 
Target Version2.11.10Fixed in Version2.11.10 
Summary0016784: sendAdminCopy() function tries to send to an empty email address
DescriptionThe function sendAdminCopy() tries to send to an empty email address when the configuration item admin_addresses is empty, i.e. there are no other admins to copy on the email. This error gets caught and logged in the event log.

The cause is the php explode() function which, for an empty string, returns an array of one element, not an empty array.

See file admin/lib.php line 433

    if (!sizeof($mails)) {
      $admin_mail = getConfig("admin_address");
      $mails = explode(",",getConfig("admin_addresses"));
      array_push($mails,$admin_mail);
    }

Can be changed to

    if (!sizeof($mails)) {
      $admin_mail = getConfig("admin_address");

      if ($c = getConfig("admin_addresses")) {
        $mails = explode(",", $c);
      }
      array_push($mails,$admin_mail);
    }
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
(0052025)
michiel (manager)
02-05-13 22:10

changed as suggested


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker