diff options
author | Uli Schlachter <psychon@znc.in> | 2017-12-23 13:04:59 +0100 |
---|---|---|
committer | Uli Schlachter <psychon@znc.in> | 2017-12-23 13:04:59 +0100 |
commit | 82f4028532c11152a0110f1b277e3358dfca6545 (patch) | |
tree | 277efc232a8cca5187f5119dda5416c471415c15 /src/cairo-png.c | |
parent | 3e07efd21e91ab90969aa2e4fc8b66adebac01ed (diff) | |
download | cairo-82f4028532c11152a0110f1b277e3358dfca6545.tar.gz |
Revert "fix warning: variable X might be clobbered by 'longjmp'"
This reverts commit b092b63119cbfe3cb4bc786eee81630998996acf which
introduced a wrapper function around setjmp(). To quote from man setjmp:
If the function which called setjmp() returns before longjmp() is
called, the behavior is undefined. Some kind of subtle or unsubtle
chaos is sure to result.
Since after the above commit setjmp() is called from the wrapper
function, the result might or might not work, depending on compiler
settings. If the setjmp() wrapper is not inlined, then the state of the
stack after longjmp() will likely be garbage.
Diffstat (limited to 'src/cairo-png.c')
-rw-r--r-- | src/cairo-png.c | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/src/cairo-png.c b/src/cairo-png.c index fe23fcf9d..596b506ab 100644 --- a/src/cairo-png.c +++ b/src/cairo-png.c @@ -158,14 +158,6 @@ png_simple_warning_callback (png_structp png, */ } -static int -png_setjmp (png_struct *png) -{ -#ifdef PNG_SETJMP_SUPPORTED - return setjmp (png_jmpbuf (png)); -#endif - return 0; -} /* Starting with libpng-1.2.30, we must explicitly specify an output_flush_fn. * Otherwise, we will segfault if we are writing to a stream. */ @@ -237,8 +229,10 @@ write_png (cairo_surface_t *surface, goto BAIL4; } - if (png_setjmp (png)) +#ifdef PNG_SETJMP_SUPPORTED + if (setjmp (png_jmpbuf (png))) goto BAIL4; +#endif png_set_write_fn (png, closure, write_func, png_simple_output_flush_fn); @@ -584,11 +578,12 @@ read_png (struct png_read_closure_t *png_closure) png_set_read_fn (png, png_closure, stream_read_func); status = CAIRO_STATUS_SUCCESS; - - if (png_setjmp (png)) { +#ifdef PNG_SETJMP_SUPPORTED + if (setjmp (png_jmpbuf (png))) { surface = _cairo_surface_create_in_error (status); goto BAIL; } +#endif png_read_info (png, info); |