Skip to content

Commit

Permalink
Merge branch 'PHP-8.4'
Browse files Browse the repository at this point in the history
  • Loading branch information
devnexen committed Feb 5, 2025
2 parents dd66bb9 + dc7b661 commit 65a20d5
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 0 deletions.
5 changes: 5 additions & 0 deletions ext/gd/gd.c
Original file line number Diff line number Diff line change
Expand Up @@ -3956,6 +3956,11 @@ PHP_FUNCTION(imagescale)

im = php_gd_libgdimageptr_from_zval_p(IM);

if (tmp_h < 0 && tmp_w < 0) {
zend_value_error("Argument #2 ($width) and argument #3 ($height) cannot be both negative");
RETURN_THROWS();
}

if (tmp_h < 0 || tmp_w < 0) {
/* preserve ratio */
long src_x, src_y;
Expand Down
17 changes: 17 additions & 0 deletions ext/gd/tests/gh17703.phpt
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
--TEST--
GH-17703 both width and height value being negative triggers ValueError on width.
--EXTENSIONS--
gd
--FILE--
<?php

$img = imagecreatetruecolor ( 256, 1);

try {
imagescale($img, -1, -1, 0);
} catch (\ValueError $e) {
echo $e->getMessage();
}
?>
--EXPECT--
Argument #2 ($width) and argument #3 ($height) cannot be both negative

0 comments on commit 65a20d5

Please sign in to comment.