From: Ilia Alshanetsky Date: Mon, 8 Sep 2003 20:15:35 +0000 (+0000) Subject: Re-enable sendmail binary check, now with parameter detection. X-Git-Tag: RELEASE_0_7~169 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=80fb9496b3e7a8fd33294d9b49c41c879df2b3a1;p=php Re-enable sendmail binary check, now with parameter detection. --- diff --git a/ext/standard/mail.c b/ext/standard/mail.c index a6bb0bc08b..df95c60266 100644 --- a/ext/standard/mail.c +++ b/ext/standard/mail.c @@ -200,11 +200,24 @@ PHPAPI int php_mail(char *to, char *subject, char *message, char *headers, char #else /* make sure that sendmail_path contains a valid executable, failure to do * would make PHP abruptly exit without a useful error message. */ -/* if (access(sendmail_path, X_OK)) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Permission denied: unable to execute shell to run mail delivery binary '%s'", sendmail_path); - return 0; + { + char *s=NULL, p; + + if ((s = strchr(sendmail_path, ' '))) { + p = *s; + *s = '\0'; + } + if (access(sendmail_path, X_OK)) { + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Permission denied: unable to execute shell to run mail delivery binary '%s'", sendmail_path); + if (s) { + *s = p; + } + return 0; + } + if (s) { + *s = p; + } } -*/ /* Since popen() doesn't indicate if the internal fork() doesn't work * (e.g. the shell can't be executed) we explicitely set it to 0 to be