php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #42505 new sendmail default breaks NetWare platform
Submitted: 2007-08-31 22:55 UTC Modified: 2008-08-26 23:27 UTC
Votes:5
Avg. Score:4.6 ± 0.8
Reproduced:5 of 5 (100.0%)
Same Version:5 (100.0%)
Same OS:5 (100.0%)
From: gk at gknw dot de Assigned: pajoye
Status: Closed Package: Mail related
PHP Version: 5.3CVS (?) OS: NetWare
Private report: No CVE-ID:
 [2007-08-31 22:55 UTC] gk at gknw dot de
Description:
------------
The recent commit to ./main/main.c by johannes (Revision 1.640.2.23.2.43 and 1.640.2.23.2.50) breaks NetWare internal mail functionality; IMO the preprocessor ifdefs are totally bogus;
I think this patch is a more accurate way to express it, and restores previous default for NetWare platform (+ adds a comment):

--- main.c.orig	Fri Aug 03 03:30:22 2007
+++ main.c	Fri Aug 31 23:46:21 2007
@@ -333,10 +333,10 @@
 #	define PHP_SAFE_MODE_EXEC_DIR ""
 #endif
 
-#if defined(PHP_PROG_SENDMAIL) && !defined(NETWARE)
-#	define DEFAULT_SENDMAIL_PATH PHP_PROG_SENDMAIL " -t -i "
-#elif defined(PHP_WIN32)
+#if defined(PHP_WIN32) || defined(NETWARE) /* Win32 and NetWare use internal mail */
 #	define DEFAULT_SENDMAIL_PATH NULL
+#elif defined(PHP_PROG_SENDMAIL)
+#	define DEFAULT_SENDMAIL_PATH PHP_PROG_SENDMAIL " -t -i "
 #else
 #	define DEFAULT_SENDMAIL_PATH "/usr/sbin/sendmail -t -i" 
 #endif

please fix that soon! Thanks!


Reproduce code:
---------------
<?php
phpinfo()
?>

Expected result:
----------------
sendmail_path = no value

Actual result:
--------------
sendmail_path = sendmail -t -i

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2007-09-03 13:06 UTC] jani@php.net
Johannes, you broke it, you fix it..
 [2008-03-05 21:21 UTC] pajoye@php.net
Applied to 5.2 and HEAD, remains 5.3. I will do it later, no 5.3 tree at hand and low bandwidth here :)
 [2008-08-15 19:08 UTC] jani@php.net
Pierre, have you forgot this or is this already done..?
 [2008-08-26 23:27 UTC] jani@php.net
This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.


 
PHP Copyright © 2001-2014 The PHP Group
All rights reserved.
Last updated: Sat Apr 19 19:02:15 2014 UTC