Bug #14407: Win32 SendMail function doesn't use "From:" in SMTP MAIL FROM command
| From: | msisolak at yahoo dot com | Date: | Mon, 10 Dec 2001 17:27:13 +0000 |
| Subject: | Bug #14407: Win32 SendMail function doesn't use "From:" in SMTP MAIL FROM command | ||
| Groups: | php.dev | ||
| Request: | Send a blank email to php-dev+get-73085@lists.php.net to get a copy of this message | ||
From: msisolak@yahoo.com
Operating system: Windows 2000
PHP version: 4.0CVS-2001-12-10
PHP Bug Type: Mail related
Bug description: Win32 SendMail function doesn't use "From:" in SMTP MAIL FROM
command
The SendMail function that is used for Win32 platforms doesn't extract the
"From:" header address to use as for the SMTP MAIL FROM command. As a
result any mail bounces go to the "sendmail_from" php.ini option instead of
to the sender. This is handled correctly in the PostHeader function (which
doesn't add "From:" if it is already there), but not in TSendMail. The
patch for win32/sendmail.c below allows TSendMail to use the e-mail from
address if it is available, and default to the sendmail_from setting
otherwise.
124,128c124,142
<
< if (INI_STR("sendmail_from")){
< RPath = estrdup(INI_STR("sendmail_from"));
< } else {
< return 19;
---
>
> if (headers && strstr(headers, "From:")) {
> char *pos;
> pos = strstr(headers, "From:") + 5;
> while (pos && (*pos == ' '))
> pos++;
>
> if (pos && strlen(pos)) {
> RPath = estrdup(pos);
> RPath[strcspn(RPath, "\r\n")] = '\x0';
> }
> }
>
> if (!RPath) {
> if (INI_STR("sendmail_from")) {
> RPath = estrdup(INI_STR("sendmail_from"));
> } else {
> return 19;
> }
--
Edit bug report at: http://bugs.php.net/?id=14407&edit=1