- fix #49372, segfault in php_curl_option_url

diff --git a/NEWS b/NEWS
index 1457f234f9cee3a93c2e3796155b2067fc4721e4..d3788b590b4883a7542d3a7b5ca489722f8dd5fd 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -20,6 +20,7 @@ PHP                                                                        NEWS
   defined as a file handle. (Ilia)
 - Fixed memory leak in stream_is_local(). (Felipe, Tony)
 
+- Fixed bug #49372 (segfault in php_curl_option_curl). (Pierre)
 - Fixed bug #49132 (posix_times returns false without error).
   (phpbugs at gunnu dot us)
 - Fixed bug #49125 (Error in dba_exists C code). (jdornan at stanford dot edu)

diff --git a/ext/curl/interface.c b/ext/curl/interface.c
index 3ee3435db27cc535b599e35c622a83c299604c13..3f7db13a3eb84cbbb7dbce1f93ecee58f73bbf83 100644 (file)
--- a/ext/curl/interface.c
+++ b/ext/curl/interface.c
@@ -183,7 +183,7 @@ static int php_curl_option_url(php_curl *ch, const char *url, const int len) /*
                        return 0;
                }
 
-               if (!strncasecmp("file", uri->scheme, sizeof("file"))) {
+               if (uri->scheme && !strncasecmp("file", uri->scheme, sizeof("file"))) {
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "Protocol 'file' disabled in cURL");
                        php_url_free(uri);
                        return 0;