MFB: Fixed bug #43373 (pcntl_fork() should not raise E_ERROR on error)

diff --git a/NEWS b/NEWS
index 19e1ff5ed3900909e8920f91cf48fbafa5a09cd9..342c08f832fdf258d06fe05c4e24d06bc4d709ca 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -3,6 +3,7 @@ PHP                                                                        NEWS
 ?? ??? 2008, PHP 5.2.6
 - Fixed bug #43386 (array_globals not reset to 0 properly on init). (Ilia)
 - Fixed bug #43377 (PHP crashes with invalid argument for DateTimeZone). (Ilia)
+- Fixed bug #43373 (pcntl_fork() should not raise E_ERROR on error). (Ilia)
 - Fixed bug #43092 (curl_copy_handle() crashes with > 32 chars long URL).
   (Jani)
 - Fixed bug #43301 (mb_ereg*_replace() crashes when replacement string is

diff --git a/ext/pcntl/pcntl.c b/ext/pcntl/pcntl.c
index 233e20f6cb2fdc1a0ccdd100815548b1839f8b4e..6188d0e54c4d78b814b28e4fd269b249609d63e5 100755 (executable)
--- a/ext/pcntl/pcntl.c
+++ b/ext/pcntl/pcntl.c
@@ -225,7 +225,7 @@ PHP_FUNCTION(pcntl_fork)
 
        id = fork();
        if (id == -1) {
-               php_error_docref(NULL TSRMLS_CC, E_ERROR, "Error %d", errno);
+               php_error_docref(NULL TSRMLS_CC, E_WARNING, "Error %d", errno);
        }
        
        RETURN_LONG((long) id);