diff options
author | Remi Collet <fedora@famillecollet.com> | 2015-03-23 18:33:56 +0100 |
---|---|---|
committer | Remi Collet <fedora@famillecollet.com> | 2015-03-23 18:33:56 +0100 |
commit | bf10ddd51b76640cce8e3fee6d1cc7fdc9e2b33a (patch) | |
tree | 6b4a6b62caad0c08acaafb856c0f0cd5fc9d4655 /gd-sa1.patch | |
parent | 027cdd97196d4e1d1b7ec0ee5d650992554cde6a (diff) |
gd: fix version in gdlib.pc
Diffstat (limited to 'gd-sa1.patch')
-rw-r--r-- | gd-sa1.patch | 59 |
1 files changed, 0 insertions, 59 deletions
diff --git a/gd-sa1.patch b/gd-sa1.patch deleted file mode 100644 index dc2b1b9..0000000 --- a/gd-sa1.patch +++ /dev/null @@ -1,59 +0,0 @@ -These issues were found by Coverity static analysis tool - -Error: DEADCODE (CWE-561): [#def3] -gd-2.0.35/gd.c:3494: cond_cannot_single: Condition "dx == 0L", taking false branch. Now the value of "dx" cannot be equal to 0. -gd-2.0.35/gd.c:3502: cannot_single: At condition "dx == 0L", the value of "dx" cannot be equal to 0. -gd-2.0.35/gd.c:3502: dead_error_condition: The condition "dx == 0L" cannot be true. -gd-2.0.35/gd.c:3502: dead_error_line: Execution cannot reach this expression "dy == 0L" inside statement "if (dx == 0L && dy == 0L){ -...". - -Error: DEADCODE (CWE-561): [#def4] -gd-2.0.35/gd.c:1085: cond_cannot_single: Condition "dx == 0", taking false branch. Now the value of "dx" cannot be equal to 0. -gd-2.0.35/gd.c:1097: cannot_single: At condition "dx == 0", the value of "dx" cannot be equal to 0. -gd-2.0.35/gd.c:1097: dead_error_condition: The condition "dx == 0" cannot be true. -gd-2.0.35/gd.c:1097: dead_error_line: Execution cannot reach this expression "dy == 0" inside statement "if (dx == 0 && dy == 0){ - ...". - - -diff -up gd-2.0.35/gd.c.sa1 gd-2.0.35/gd.c ---- gd-2.0.35/gd.c.sa1 2012-12-05 16:23:09.289667430 +0100 -+++ gd-2.0.35/gd.c 2012-12-05 16:30:41.634854587 +0100 -@@ -1094,11 +1094,6 @@ BGD_DECLARE(void) gdImageLine (gdImagePt - { - /* More-or-less horizontal. use wid for vertical stroke */ - /* Doug Claar: watch out for NaN in atan2 (2.0.5) */ -- if ((dx == 0) && (dy == 0)) -- { -- wid = 1; -- } -- else - { - /* 2.0.12: Michael Schwartz: divide rather than multiply; - TBB: but watch out for /0! */ -@@ -3490,6 +3485,12 @@ static void gdImageAALine (gdImagePtr im - dx = x2 - x1; - dy = y2 - y1; - -+ if (dx == 0 && dy == 0) { -+ /* TBB: allow setting points */ -+ gdImageSetAAPixelColor(im, x1, y1, col, 0xFF); -+ return; -+ } -+ - /* Axis aligned lines */ - if (dx == 0) { - gdImageVLine(im, x1, y1, y2, col); -@@ -3499,12 +3500,7 @@ static void gdImageAALine (gdImagePtr im - return; - } - -- if (dx == 0 && dy == 0) { -- /* TBB: allow setting points */ -- gdImageSetAAPixelColor(im, x1, y1, col, 0xFF); -- return; -- } -- else { -+ { - double ag; - if (abs(dy) < abs(dx)) - ag = cos (atan2 (dy, dx)); |