gh: fix issue #12;
authorMichael Wallner <mike@php.net>
Fri, 25 Sep 2015 06:36:22 +0000 (08:36 +0200)
committerMichael Wallner <mike@php.net>
Fri, 25 Sep 2015 06:36:22 +0000 (08:36 +0200)
Trying to set an invalid URL on an http\Message instance might cause a
crash.

php_http_message.c

index 3b19a8c8f2dc1ba94fe6812542ac04df4b4f4429..c6b03ffe597b773ec93dea1b6c5f5f81ce473f8a 100644 (file)
@@ -1619,10 +1619,10 @@ static PHP_METHOD(HttpMessage, setRequestUrl)
        url = php_http_url_from_zval(zurl, ~0 TSRMLS_CC);
        zend_restore_error_handling(&zeh TSRMLS_CC);
 
-       if (php_http_url_is_empty(url)) {
+       if (url && php_http_url_is_empty(url)) {
                php_http_url_free(&url);
                php_http_throw(invalid_arg, "Cannot set http\\Message's request url to an empty string", NULL);
-       } else {
+       } else if (url) {
                PTR_SET(obj->message->http.info.request.url, url);
        }