diff options
author | Sebastian Rasmussen <sebras@gmail.com> | 2018-08-22 22:39:56 +0800 |
---|---|---|
committer | Sebastian Rasmussen <sebras@gmail.com> | 2018-08-30 21:20:55 +0800 |
commit | 985fdcfc117a3bd4bc097cdcae8347b3787fbab2 (patch) | |
tree | 85fab11a009c7d68e43781abe2e8be5e8a5ea65a /platform/gl/gl-form.c | |
parent | 984887ee8fb431e5c5d243c40dcb73d5149b033f (diff) | |
download | mupdf-985fdcfc117a3bd4bc097cdcae8347b3787fbab2.tar.xz |
Bug 699695: Remember to end groups/softmasks even upon exception.
fz_fill_path() may throw an exception halfway through
pdf_show_path(), which in this case would not attempt to end any
begun groups or softmasks. This led to e.g. leaks of pixmaps held
by a group that was never ended.
Moving the cleanup to the always block is not foolproof because
the cleanup code itself may also throw exceptions, hence
preventing the end of the fz_always block from being executed.
This commit does put pdf_show_path() in the same situation as
pdf_run_xobject() that has the same problem with its cleanup
code.
Thanks to oss-fuzz for reporting.
Diffstat (limited to 'platform/gl/gl-form.c')
0 files changed, 0 insertions, 0 deletions