Re: svn: /php/php-src/branches/PHP_5_4/Zend/ zend_signal.c
| From: | Stas Malyshev | Date: | Tue, 08 Nov 2011 20:35:19 +0000 |
| Subject: | Re: svn: /php/php-src/branches/PHP_5_4/Zend/ zend_signal.c | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-66915@lists.php.net to get a copy of this message | ||
Hi!
IIRC there's one more place where we do sigaction and E_ERROR in the same file. -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/ (408)454-6900 ext. 227rasmus Tue, 08 Nov 2011 20:32:29 +0000Revision: http://svn.php.net/viewvc?view=revision&revision=318921 Log: This really shouldn't fail, but if it does, like on OSX under Valgrind, let's just issue a warning. Changed paths:U php/php-src/branches/PHP_5_4/Zend/zend_signal.cModified: php/php-src/branches/PHP_5_4/Zend/zend_signal.c =================================================================== --- php/php-src/branches/PHP_5_4/Zend/zend_signal.c 2011-11-08 19:07:23 UTC (rev 318920) +++ php/php-src/branches/PHP_5_4/Zend/zend_signal.c 2011-11-08 20:32:29 UTC (rev 318921) @@ -215,7 +215,7 @@sa.sa_mask = global_sigmask;if (sigaction(signo,&sa, NULL)< 0) { - zend_error(E_ERROR, "Error installing signal handler for %d", signo); + zend_error(E_WARNING, "Error installing signal handler for %d", signo); }