From c780f4a5624f27a6cbf7829e10e8cd3544ae4f38 Mon Sep 17 00:00:00 2001 From: Carl Worth Date: Mon, 13 Mar 2006 12:05:13 -0800 Subject: [PATCH] cairo-output-stream: Don't dereference a NULL pointer due to OOM. This close bug #6176: Null pointer dereference on OOM in _cairo_output_stream_create_for_file() https://bugs.freedesktop.org/show_bug.cgi?id=6176 --- src/cairo-output-stream.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/cairo-output-stream.c b/src/cairo-output-stream.c index a6db09198..b07423c10 100644 --- a/src/cairo-output-stream.c +++ b/src/cairo-output-stream.c @@ -305,9 +305,11 @@ _cairo_output_stream_create_for_file (const char *filename) return NULL; stream = _cairo_output_stream_create (stdio_write, fp); - if (stream == NULL) + + if (stream) + stream->owns_closure_is_file = TRUE; + else fclose (fp); - stream->owns_closure_is_file = TRUE; return stream; }