Merge branch 'PHP-7.3'

nikic · nikic · commit 1fbc309e2caf · 2018-12-23T20:21:16.000+01:00
diff --git a/ext/standard/browscap.c b/ext/standard/browscap.c
@@ -757,7 +757,7 @@ PHP_FUNCTION(get_browser)
 			found_entry = zend_hash_str_find_ptr(bdata->htab,
 				DEFAULT_SECTION_NAME, sizeof(DEFAULT_SECTION_NAME)-1);
 			if (found_entry == NULL) {
-				efree(lookup_browser_name);
+				zend_string_release(lookup_browser_name);
 				RETURN_FALSE;
 			}
 		}
diff --git a/ext/standard/tests/misc/browscap_no_default.ini b/ext/standard/tests/misc/browscap_no_default.ini
@@ -0,0 +1,2 @@
+[Firefox]
+Browser="Firefox"
diff --git a/ext/standard/tests/misc/get_browser_no_default.phpt b/ext/standard/tests/misc/get_browser_no_default.phpt
@@ -0,0 +1,12 @@
+--TEST--
+get_browser() without a default
+--INI--
+browscap={PWD}/browscap_no_default.ini
+--FILE--
+<?php
+
+var_dump(get_browser(""));
+
+?>
+--EXPECT--
+bool(false)

Original file line number	Diff line number	Diff line change
`@@ -757,7 +757,7 @@ PHP_FUNCTION(get_browser)`
`757`	`757`	`found_entry = zend_hash_str_find_ptr(bdata->htab,`
`758`	`758`	`DEFAULT_SECTION_NAME, sizeof(DEFAULT_SECTION_NAME)-1);`
`759`	`759`	`if (found_entry == NULL) {`
`760`		`- efree(lookup_browser_name);`
	`760`	`+ zend_string_release(lookup_browser_name);`
`761`	`761`	`RETURN_FALSE;`
`762`	`762`	`}`
`763`	`763`	`}`