ext/gd: Fix comparison with result of php_stream_can_cast()

Girgias · Girgias · commit 83b8d2c290d2 · 2025-07-28T11:54:19.000+01:00
Closes GH-19107
diff --git a/NEWS b/NEWS
@@ -11,6 +11,9 @@ PHP                                                                        NEWS
 - FTP:
   . Fix theoretical issues with hrtime() not being available. (nielsdos)
 
+- GD:
+  . Fix incorrect comparison with result of php_stream_can_cast(). (Girgias)
+
 - Hash:
   . Fix crash on clone failure. (nielsdos)
 
diff --git a/ext/gd/gd.c b/ext/gd/gd.c
@@ -1566,7 +1566,7 @@ static void _php_image_create_from(INTERNAL_FUNCTION_PARAMETERS, int image_type,
 		pefree(pstr, 1);
 		zend_string_release_ex(buff, 0);
 	}
-	else if (php_stream_can_cast(stream, PHP_STREAM_AS_STDIO)) {
+	else if (php_stream_can_cast(stream, PHP_STREAM_AS_STDIO) == SUCCESS) {
 		/* try and force the stream to be FILE* */
 		if (FAILURE == php_stream_cast(stream, PHP_STREAM_AS_STDIO | PHP_STREAM_CAST_TRY_HARD, (void **) &fp, REPORT_ERRORS)) {
 			goto out_err;

Original file line number	Diff line number	Diff line change
`@@ -1566,7 +1566,7 @@ static void _php_image_create_from(INTERNAL_FUNCTION_PARAMETERS, int image_type,`
`1566`	`1566`	`pefree(pstr, 1);`
`1567`	`1567`	`zend_string_release_ex(buff, 0);`
`1568`	`1568`	`}`
`1569`		`- else if (php_stream_can_cast(stream, PHP_STREAM_AS_STDIO)) {`
	`1569`	`+ else if (php_stream_can_cast(stream, PHP_STREAM_AS_STDIO) == SUCCESS) {`
`1570`	`1570`	`/* try and force the stream to be FILE* */`
`1571`	`1571`	`if (FAILURE == php_stream_cast(stream, PHP_STREAM_AS_STDIO \| PHP_STREAM_CAST_TRY_HARD, (void **) &fp, REPORT_ERRORS)) {`
`1572`	`1572`	`goto out_err;`