mbeccati Sat Mar 28 03:11:39 2009 UTC
Modified files:
/php-src/ext/pcntl pcntl.c
/php-src/ext/pcntl/tests pcntl_signal.phpt
Log:
- Reverting: pcntl_signal() returning NULL when passing wrong parameters
# Wow, my first commit and I broke something, sorry about that :(
http://cvs.php.net/viewvc.cgi/php-src/ext/pcntl/pcntl.c?r1=1.76&r2=1.77&diff_format=u
Index: php-src/ext/pcntl/pcntl.c
diff -u php-src/ext/pcntl/pcntl.c:1.76 php-src/ext/pcntl/pcntl.c:1.77
--- php-src/ext/pcntl/pcntl.c:1.76 Sat Mar 28 01:31:31 2009
+++ php-src/ext/pcntl/pcntl.c Sat Mar 28 03:11:39 2009
@@ -16,7 +16,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: pcntl.c,v 1.76 2009/03/28 01:31:31 mbeccati Exp $ */
+/* $Id: pcntl.c,v 1.77 2009/03/28 03:11:39 mbeccati Exp $ */
#define PCNTL_DEBUG 0
@@ -757,7 +757,7 @@
zend_bool restart_syscalls = 1;
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "lz|b", &signo,
&handle, &restart_syscalls) == FAILURE) {
- RETURN_FALSE;
+ return;
}
if (!PCNTL_G(spares)) {
http://cvs.php.net/viewvc.cgi/php-src/ext/pcntl/tests/pcntl_signal.phpt?r1=1.2&r2=1.3&diff_format=u
Index: php-src/ext/pcntl/tests/pcntl_signal.phpt
diff -u php-src/ext/pcntl/tests/pcntl_signal.phpt:1.2
php-src/ext/pcntl/tests/pcntl_signal.phpt:1.3
--- php-src/ext/pcntl/tests/pcntl_signal.phpt:1.2 Sat Mar 28 01:31:31 2009
+++ php-src/ext/pcntl/tests/pcntl_signal.phpt Sat Mar 28 03:11:39 2009
@@ -25,7 +25,7 @@
signal dispatched
Warning: pcntl_signal() expects at least 2 parameters, 0 given in %s
-bool(false)
+NULL
bool(true)
Warning: pcntl_signal(): Invalid value for handle argument specified in %s
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php