- #45905, imagefilledrectangle() clipping error

diff --git a/ext/gd/libgd/gd.c b/ext/gd/libgd/gd.c
index d66bbbfad8541ac92bd2b9577c2eb9a707403598..f01a6904ee606f76bd4733578f057fe39433bd6c 100644 (file)
--- a/ext/gd/libgd/gd.c
+++ b/ext/gd/libgd/gd.c
@@ -2114,33 +2114,40 @@ void gdImageFilledRectangle (gdImagePtr im, int x1, int y1, int x2, int y2, int
 {
        int x, y;
 
-       /* Nick Atty: limit the points at the edge.  Note that this also
-        * nicely kills any plotting for rectangles completely outside the
-        * window as it makes the tests in the for loops fail
-        */
-       if (x1 < 0) {
-               x1 = 0;
-       }
-       if (x1 > gdImageSX(im)) {
-               x1 = gdImageSX(im);
-       }
-       if(y1 < 0) {
-               y1 = 0;
-       }
-       if (y1 > gdImageSY(im)) {
-               y1 = gdImageSY(im);
+
+       if (x1 == x2 && y1 == y2) {
+               gdImageSetPixel(im, x1, y1, color);
+               return;
        }
+
        if (x1 > x2) {
                x = x1;
                x1 = x2;
                x2 = x;
        }
+
        if (y1 > y2) {
                y = y1;
                y1 = y2;
                y2 = y;
        }
 
+       if (x1 < 0) {
+               x1 = 0;
+       }
+
+       if (x2 >= gdImageSX(im)) {
+               x2 = gdImageSX(im) - 1;
+       }
+
+       if (y1 < 0) {
+               y1 = 0;
+       }
+
+       if (y2 >= gdImageSY(im)) {
+               y2 = gdImageSY(im) - 1;
+       }
+
        for (y = y1; (y <= y2); y++) {
                for (x = x1; (x <= x2); x++) {
                        gdImageSetPixel (im, x, y, color);