diff options
| author | Christoph M. Becker <cmbecker69@gmx.de> | 2020-06-12 14:07:16 +0200 |
|---|---|---|
| committer | Christoph M. Becker <cmbecker69@gmx.de> | 2020-06-12 14:07:43 +0200 |
| commit | ff7fd3dc858eb71234ee2f38f2c2f58050927b8b (patch) | |
| tree | 0edc663a0b8cc67cf0f4994b57a11015479eabde /ext/gd | |
| parent | 09532a12a6afb28e5a956547bce8b07572a26ba5 (diff) | |
| parent | 86e1f0ea1197e878a657288562d0a11d6f427a18 (diff) | |
| download | php-git-ff7fd3dc858eb71234ee2f38f2c2f58050927b8b.tar.gz | |
Merge branch 'PHP-7.3' into PHP-7.4
* PHP-7.3:
Fix #79676: imagescale adds black border with IMG_BICUBIC
Diffstat (limited to 'ext/gd')
| -rw-r--r-- | ext/gd/libgd/gd_interpolation.c | 8 | ||||
| -rw-r--r-- | ext/gd/tests/bug79676.phpt | 31 |
2 files changed, 35 insertions, 4 deletions
diff --git a/ext/gd/libgd/gd_interpolation.c b/ext/gd/libgd/gd_interpolation.c index 698e407fbf..886b3b023b 100644 --- a/ext/gd/libgd/gd_interpolation.c +++ b/ext/gd/libgd/gd_interpolation.c @@ -936,7 +936,7 @@ static inline void _gdScaleRow(gdImagePtr pSrc, unsigned int src_width, gdImage int *p_dst_row = dst->tpixels[row]; unsigned int x; - for (x = 0; x < dst_width - 1; x++) { + for (x = 0; x < dst_width; x++) { register unsigned char r = 0, g = 0, b = 0, a = 0; const int left = contrib->ContribRow[x].Left; const int right = contrib->ContribRow[x].Right; @@ -972,7 +972,7 @@ static inline int _gdScaleHoriz(gdImagePtr pSrc, unsigned int src_width, unsigne return 0; } /* Scale each row */ - for (u = 0; u < dst_height - 1; u++) { + for (u = 0; u < dst_height; u++) { _gdScaleRow(pSrc, src_width, pDst, dst_width, u, contrib); } _gdContributionsFree (contrib); @@ -982,7 +982,7 @@ static inline int _gdScaleHoriz(gdImagePtr pSrc, unsigned int src_width, unsigne static inline void _gdScaleCol (gdImagePtr pSrc, unsigned int src_width, gdImagePtr pRes, unsigned int dst_width, unsigned int dst_height, unsigned int uCol, LineContribType *contrib) { unsigned int y; - for (y = 0; y < dst_height - 1; y++) { + for (y = 0; y < dst_height; y++) { register unsigned char r = 0, g = 0, b = 0, a = 0; const int iLeft = contrib->ContribRow[y].Left; const int iRight = contrib->ContribRow[y].Right; @@ -1019,7 +1019,7 @@ static inline int _gdScaleVert (const gdImagePtr pSrc, const unsigned int src_wi return 0; } /* scale each column */ - for (u = 0; u < dst_width - 1; u++) { + for (u = 0; u < dst_width; u++) { _gdScaleCol(pSrc, src_width, pDst, dst_width, dst_height, u, contrib); } _gdContributionsFree(contrib); diff --git a/ext/gd/tests/bug79676.phpt b/ext/gd/tests/bug79676.phpt new file mode 100644 index 0000000000..1187bfa1df --- /dev/null +++ b/ext/gd/tests/bug79676.phpt @@ -0,0 +1,31 @@ +--TEST-- +Bug #79676 (imagescale adds black border with IMG_BICUBIC) +--SKIPIF-- +<?php +if (!extension_loaded('gd')) die('skip gd extension not available'); +?> +--FILE-- +<?php +function test($image, $desc) +{ + echo "$desc - Test Result: ", + (imagecolorat($image, imagesx($image) - 1 , imagesy($image) - 1) != 0x000000 ? 'pass' : 'fail'), + PHP_EOL; +} + +$size = 32; +$src = imagecreatetruecolor($size, $size); +imagefilledrectangle($src, 0, 0, $size - 1 , $size - 1, 0xff00ff); + +test($src, 'No scaling'); +test(imagescale($src, $size * 2, $size * 2), 'Scale 200%, default mode'); +test(imagescale($src, $size / 2, $size / 2), 'Scale 50%, default mode'); +test(imagescale($src, $size * 2, $size * 2, IMG_BICUBIC), 'Scale 200%, IMG_BICUBIC mode'); +test(imagescale($src, $size / 2, $size / 2, IMG_BICUBIC), 'Scale 50%, IMG_BICUBIC mode'); +?> +--EXPECT-- +No scaling - Test Result: pass +Scale 200%, default mode - Test Result: pass +Scale 50%, default mode - Test Result: pass +Scale 200%, IMG_BICUBIC mode - Test Result: pass +Scale 50%, IMG_BICUBIC mode - Test Result: pass |