Fix for bug #51604 (newline in end of header is shown in start of message).

Patch by Daniel Egeberg.

diff --git a/NEWS b/NEWS
index 8187f2788831a1682ccbca8387587ad8f706680a..92ca9f91d8644fc8a701000d4e10a8e7a2e3e724 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -30,6 +30,8 @@ PHP                                                                        NEWS
   literal). (cbandy at jbandy dot com)
 - Fixed bug #51607 (pg_copy_from does not allow schema in the tablename
   argument). (cbandy at jbandy dot com)
+- Fixed bug #51604 (newline in end of header is shown in start of message).
+  (Daniel Egeberg)
 - Fixed bug #51590 (JSON_ERROR_UTF8 is undefined). (Felipe)
 - Fixed bug #51562 (query timeout in mssql can not be changed per query).
   (ejsmont dot artur at gmail dot com)

diff --git a/ext/standard/mail.c b/ext/standard/mail.c
index ab65f164e70982cb93476e6b79ea0a9bdc25461c..d9deb5c84a71c3711e2dc6ade29809fd8a5ee2f4 100644 (file)
--- a/ext/standard/mail.c
+++ b/ext/standard/mail.c
@@ -97,7 +97,7 @@ PHP_FUNCTION(ezmlm_hash)
    Send an email message */
 PHP_FUNCTION(mail)
 {
-       char *to=NULL, *message=NULL, *headers=NULL;
+       char *to=NULL, *message=NULL, *headers=NULL, *headers_trimmed=NULL;
        char *subject=NULL, *extra_cmd=NULL;
        int to_len, message_len, headers_len = 0;
        int subject_len, extra_cmd_len = 0, i;
@@ -122,6 +122,7 @@ PHP_FUNCTION(mail)
        MAIL_ASCIIZ_CHECK(message, message_len);
        if (headers) {
                MAIL_ASCIIZ_CHECK(headers, headers_len);
+               headers_trimmed = php_trim(headers, headers_len, NULL, 0, NULL, 2 TSRMLS_CC);
        }
        if (extra_cmd) {
                MAIL_ASCIIZ_CHECK(extra_cmd, extra_cmd_len);
@@ -173,12 +174,16 @@ PHP_FUNCTION(mail)
                extra_cmd = php_escape_shell_cmd(extra_cmd);
        }
 
-       if (php_mail(to_r, subject_r, message, headers, extra_cmd TSRMLS_CC)) {
+       if (php_mail(to_r, subject_r, message, headers_trimmed, extra_cmd TSRMLS_CC)) {
                RETVAL_TRUE;
        } else {
                RETVAL_FALSE;
        }
 
+       if (headers_trimmed) {
+               efree(headers_trimmed);
+       }
+
        if (extra_cmd) {
                efree (extra_cmd);
        }

diff --git a/ext/standard/tests/mail/bug51604.phpt b/ext/standard/tests/mail/bug51604.phpt
new file mode 100644 (file)
index 0000000..a657021
--- /dev/null
+++ b/ext/standard/tests/mail/bug51604.phpt
@@ -0,0 +1,35 @@
+--TEST--
+Bug #51604 (newline in end of header is shown in start of message)
+--INI--
+sendmail_path=tee mail_bug51604.out >/dev/null
+mail.add_x_header = Off
+--SKIPIF--
+<?php
+if(substr(PHP_OS, 0, 3) == "WIN")
+  die("skip Won't run on Windows");
+?>
+--FILE--
+<?php
+// Initialise all required variables
+$to = 'user@company.com';
+$subject = 'Test Subject';
+$message = 'A Message';
+$additional_headers = "KHeaders\n\n\n\n\n";
+$outFile = "mail_bug51604.out";
+@unlink($outFile);
+
+// Calling mail() with all additional headers
+var_dump( mail($to, $subject, $message, $additional_headers) );
+echo file_get_contents($outFile);
+unlink($outFile);
+
+?>
+===DONE===
+--EXPECT--
+bool(true)
+To: user@company.com
+Subject: Test Subject
+KHeaders
+
+A Message
+===DONE===