Throw an exception when an invalid comparison operator is passed to version_compare()

Author: kocsismate

ext/opcache/Optimizer/zend_func_info.c

@@ -503,7 +503,7 @@ static const func_info_t func_infos[] = {
 	FN("array_key_last",               MAY_BE_NULL | MAY_BE_LONG | MAY_BE_STRING),
 	F1("pos",                          UNKNOWN_INFO),
 	F1("assert_options",               MAY_BE_NULL | MAY_BE_FALSE | MAY_BE_LONG | MAY_BE_STRING | MAY_BE_ARRAY | MAY_BE_ARRAY_KEY_LONG | MAY_BE_ARRAY_OF_STRING | MAY_BE_ARRAY_OF_OBJECT | MAY_BE_OBJECT),
-	F0("version_compare",              MAY_BE_NULL | MAY_BE_FALSE | MAY_BE_TRUE | MAY_BE_LONG),
+	F0("version_compare",              MAY_BE_FALSE | MAY_BE_TRUE | MAY_BE_LONG),
 	F1("str_rot13",                    MAY_BE_STRING),
 	F1("stream_get_filters",           MAY_BE_ARRAY | MAY_BE_ARRAY_KEY_LONG | MAY_BE_ARRAY_OF_STRING),
 	F0("stream_filter_register",       MAY_BE_FALSE | MAY_BE_TRUE),

ext/standard/tests/versioning/version_compare_invalid_operator.phpt

@@ -1,10 +1,14 @@
 --TEST--
-Ensures null is returned if versions are compared with invalid operator
+Ensures an exception is thrown if versions are compared with an invalid operator
 --CREDITS--
 David Stockton - <[email protected]> - i3logix PHP Testfest 2017
 --FILE--
 <?php
-var_dump(version_compare('1.2', '2.1', '??'));
+try {
+    version_compare('1.2', '2.1', '??');
+} catch (ValueError $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
 --EXPECT--
-NULL
+Invalid comparison operator

ext/standard/versioning.c

@@ -237,7 +237,8 @@ PHP_FUNCTION(version_compare)
 	if (!strncmp(op, "!=", op_len) || !strncmp(op, "<>", op_len) || !strncmp(op, "ne", op_len)) {
 		RETURN_BOOL(compare != 0);
 	}
-	RETURN_NULL();
+
+	zend_value_error("Invalid comparison operator");
 }
 
 /* }}} */