php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #47983 schizophrenic behaviour of adding EOL indicator in mail()
Submitted: 2009-04-16 09:05 UTC Modified: 2011-01-23 21:12 UTC
Votes:11
Avg. Score:4.8 ± 0.6
Reproduced:9 of 9 (100.0%)
Same Version:1 (11.1%)
Same OS:8 (88.9%)
From: hannibal at astral dot lodz dot pl Assigned:
Status: Open Package: Mail related
PHP Version: 5.2.9 OS: Linux
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
MUST BE VALID
Solve the problem:
21 + 23 = ?
Subscribe to this entry?

 
 [2009-04-16 09:05 UTC] hannibal at astral dot lodz dot pl
Description:
------------
Diffrent parts of mail function use LF some CRLF. This cause headers
to have mixed LF & CRLF endings. This can casue in some situation
broken mail delivery. PLEASE do not tell this is MTA problem. This is
also your concer to PROVIDE MTA standard compilant headers or at
least do this in consistent way.

Details:
ext/standard/mail.c (PHP 5.2.9) line 147
SKIP_LONG_HEADER_SEP(to_r, i);

This code assume that user provide CRLF line ending. If user provide
LF only, it is replace by spaces - so it cannot be used.


ext/standard/mail.c (PHP 5.2.9) line 273-278
fprintf(sendmail, "To: %s\n", to);
fprintf(sendmail, "Subject: %s\n", subject);
if (headers != NULL) {
   fprintf(sendmail, "%s\n", headers);
}
fprintf(sendmail, "\n%s\n", message);

This cleanly show that headers added automatially by PHP are always
ended by LF only.

In case, where subject is multline, headers always have mixed CRLF
and LF. This cause Postfix to left CR in header lines that have it.
It can cause probles where buggy Qmail is recipent, it replaces alone
CR to empty line, which causes premature end of headers in recipent
mail program. Again this is also YOUR PROBLEM, MTA must only properly
handling RFC compilant data - when other is sended, behaviour can be
undefined.

There is a couple methods to fix it, more or less properly.
1. Add "\r" to fprintf - RFC compilant but probably broke many apps
which use LF only header endings.
2. Add support to LF ending to SKIP_LONG_HEADER_SEP - also quite
easy, but not necessery best.
3. Add prameter to mail function in which user provide desired EOL -
best some enum (LEGACY - current buggy, CRLF - RFC compilant, LF -
unix compilant)
4. Configuration parameter whith same values as above.


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-01-23 21:12 UTC] jani@php.net
-Package: Feature/Change Request +Package: Mail related
 [2016-08-02 19:08 UTC] anrdaemon at freemail dot ru
Using a wrapper along the lines of

/usr/bin/tee -a "/tmp/sendmail.log" | /usr/sbin/sendmail "$@"

it is easy to prove that PHP, indeed, inconsistent in its documentation and internal use of mail handling.
In other words, contrary to the http://php.net/manual/en/function.mail.php additional headers must be provided with LF separation. Even on Windows.
 
PHP Copyright © 2001-2019 The PHP Group
All rights reserved.
Last updated: Thu Sep 19 23:01:27 2019 UTC