ext/opcache: formalize return type of do_validate_timestamps() to zend_result

Girgias · Girgias · commit c136801ddf4e · 2026-03-24T17:46:26.000Z
diff --git a/ext/opcache/ZendAccelerator.c b/ext/opcache/ZendAccelerator.c
@@ -1137,11 +1137,11 @@ accel_time_t zend_get_file_handle_timestamp(zend_file_handle *file_handle, size_
 	return statbuf.st_mtime;
 }
 
-static inline int do_validate_timestamps(zend_persistent_script *persistent_script, zend_file_handle *file_handle)
+static inline zend_result do_validate_timestamps(zend_persistent_script *persistent_script, zend_file_handle *file_handle)
 {
 	zend_file_handle ps_handle;
 	zend_string *full_path_ptr = NULL;
-	int ret;
+	zend_result ret;
 
 	/** check that the persistent script is indeed the same file we cached
 	 * (if part of the path is a symlink than it possible that the user will change it)

Original file line number	Diff line number	Diff line change
`@@ -1137,11 +1137,11 @@ accel_time_t zend_get_file_handle_timestamp(zend_file_handle *file_handle, size_`
`1137`	`1137`	`return statbuf.st_mtime;`
`1138`	`1138`	`}`
`1139`	`1139`
`1140`		`-static inline int do_validate_timestamps(zend_persistent_script persistent_script, zend_file_handle file_handle)`
	`1140`	`+static inline zend_result do_validate_timestamps(zend_persistent_script persistent_script, zend_file_handle file_handle)`
`1141`	`1141`	`{`
`1142`	`1142`	`zend_file_handle ps_handle;`
`1143`	`1143`	`zend_string *full_path_ptr = NULL;`
`1144`		`- int ret;`
	`1144`	`+ zend_result ret;`
`1145`	`1145`
`1146`	`1146`	`/** check that the persistent script is indeed the same file we cached`
`1147`	`1147`	`* (if part of the path is a symlink than it possible that the user will change it)`