diff options
author | Robin Watts <robin.watts@artifex.com> | 2019-02-04 16:56:51 +0000 |
---|---|---|
committer | Robin Watts <robin.watts@artifex.com> | 2019-02-04 19:35:54 +0000 |
commit | c5fd7dc0f3c706eddd59488a33fc334a2860da49 (patch) | |
tree | fdf283f615acf61cba9672781784c501371a8a1e /base/gxp1fill.c | |
parent | 2d5b86f0acf37fcb6176817580c2d3bb745e27d2 (diff) | |
download | ghostpdl-c5fd7dc0f3c706eddd59488a33fc334a2860da49.tar.gz |
Ensure return code of gs_bbox_transform_inverse always checked.
Spotted this while debugging something else.
Diffstat (limited to 'base/gxp1fill.c')
-rw-r--r-- | base/gxp1fill.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/base/gxp1fill.c b/base/gxp1fill.c index 7570e7b7d..406dd169c 100644 --- a/base/gxp1fill.c +++ b/base/gxp1fill.c @@ -175,7 +175,9 @@ tile_by_steps(tile_fill_state_t * ptfs, int x0, int y0, int w0, int h0, bbox.p.x = x0, bbox.p.y = y0; bbox.q.x = x1, bbox.q.y = y1; - gs_bbox_transform_inverse(&bbox, &step_matrix, &ibbox); + code = gs_bbox_transform_inverse(&bbox, &step_matrix, &ibbox); + if (code < 0) + return code; if_debug10m('T', mem, "[T]x,y=(%d,%d) w,h=(%d,%d) => (%g,%g),(%g,%g), offset=(%g,%g)\n", x0, y0, w0, h0, @@ -616,6 +618,7 @@ tile_by_steps_trans(tile_fill_trans_state_t * ptfs, int x0, int y0, int w0, int #ifdef DEBUG const gs_memory_t *mem = ptile->ttrans->mem; #endif + int code; ptfs->x0 = x0, ptfs->w0 = w0; ptfs->y0 = y0, ptfs->h0 = h0; @@ -631,7 +634,9 @@ tile_by_steps_trans(tile_fill_trans_state_t * ptfs, int x0, int y0, int w0, int bbox.p.x = x0, bbox.p.y = y0; bbox.q.x = x1, bbox.q.y = y1; - gs_bbox_transform_inverse(&bbox, &step_matrix, &ibbox); + code = gs_bbox_transform_inverse(&bbox, &step_matrix, &ibbox); + if (code < 0) + return code; if_debug10m('T', mem, "[T]x,y=(%d,%d) w,h=(%d,%d) => (%g,%g),(%g,%g), offset=(%g,%g)\n", x0, y0, w0, h0, |