Use new API for checking if string has NUL bytes

Girgias · jorgsowa · commit b1184ea27031 · 2024-10-13T17:58:47.000+02:00
diff --git a/ext/session/session.c b/ext/session/session.c
@@ -685,7 +685,7 @@ static bool is_session_name_valid(const zend_string *name, int diagnostic_type)
 		return false;
 	}
 	/* NUL bytes are not allowed */
-	if (ZSTR_LEN(name) != strlen(ZSTR_VAL(name))) {
+	if (zend_str_has_nul_byte(name)) {
 		if (diagnostic_type) {
 			php_error_docref(NULL, diagnostic_type, "session.name \"%s\" cannot contain NUL bytes", ZSTR_VAL(name));
 		}

Original file line number	Diff line number	Diff line change
`@@ -685,7 +685,7 @@ static bool is_session_name_valid(const zend_string *name, int diagnostic_type)`
`685`	`685`	`return false;`
`686`	`686`	`}`
`687`	`687`	`/* NUL bytes are not allowed */`
`688`		`- if (ZSTR_LEN(name) != strlen(ZSTR_VAL(name))) {`
	`688`	`+ if (zend_str_has_nul_byte(name)) {`
`689`	`689`	`if (diagnostic_type) {`
`690`	`690`	`php_error_docref(NULL, diagnostic_type, "session.name \"%s\" cannot contain NUL bytes", ZSTR_VAL(name));`
`691`	`691`	`}`