diff options
author | Uli Schlachter <psychon@znc.in> | 2021-07-23 17:22:16 +0200 |
---|---|---|
committer | Uli Schlachter <psychon@znc.in> | 2021-07-23 17:22:16 +0200 |
commit | 2fbd53a6b33a90a08ef6eda9e5f88e8fb3b97c1e (patch) | |
tree | b99bcdf4741e6473d77ddc0e2ea1e71e76381086 /src/cairo-pdf-surface.c | |
parent | b408352d450bb8c1b1cc17b14e0d605dc40daaed (diff) | |
download | cairo-2fbd53a6b33a90a08ef6eda9e5f88e8fb3b97c1e.tar.gz |
pdf: Properly pass on stdio write errors
cairo-pdf was silently ignoring write errors in
_cairo_pdf_surface_finish(). Any write errors that happened here ended
up setting a "status" variable, but the value in this variable was then
unused.
This commit fixes this bug by passing this error on to the caller.
Additionally, this also adds a test case for this behaviour based on
writing to /dev/full. This file is non-standard and thus the test first
checks that this file exists and is writable before trying to write to
it.
This bug was found based on a report from Knut Petersen [0].
[0]: https://lists.cairographics.org/archives/cairo/2021-July/029281.html
Signed-off-by: Uli Schlachter <psychon@znc.in>
Diffstat (limited to 'src/cairo-pdf-surface.c')
-rw-r--r-- | src/cairo-pdf-surface.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/cairo-pdf-surface.c b/src/cairo-pdf-surface.c index b033fdf07..f2463a8c3 100644 --- a/src/cairo-pdf-surface.c +++ b/src/cairo-pdf-surface.c @@ -2329,7 +2329,9 @@ _cairo_pdf_surface_finish (void *abstract_surface) _cairo_surface_clipper_reset (&surface->clipper); - return _cairo_pdf_interchange_fini (surface); + _cairo_pdf_interchange_fini (surface); + + return status; } static cairo_int_status_t |