From f07598eb6f6d5306f0971d391ef0356f7e06e372 Mon Sep 17 00:00:00 2001 From: Daniel van Vugt Date: Thu, 11 Oct 2018 06:49:22 +0000 Subject: [PATCH] cogl-auto-texture: Avoid a double-free crash If texture allocation fails (e.g. on an old GPU with size limit 2048) then `cogl_texture_new_with_size` was trying to use the same CoglError twice. The second time was after it had already been freed. Bug reported and fix provided by Gert van de Kraats. (cherry picked from commit d21478b0f0163b0688abe065a7009b33453d0d00) Bug-Ubuntu: https://launchpad.net/bugs/1790525 Applied-upstream: 3.30.3, commit:1cb21877edd9c7c5bc991527ae70879b1e9fece0 Gbp-Pq: Name cogl-auto-texture-Avoid-a-double-free-crash.patch --- cogl/cogl/deprecated/cogl-auto-texture.c | 1 + 1 file changed, 1 insertion(+) diff --git a/cogl/cogl/deprecated/cogl-auto-texture.c b/cogl/cogl/deprecated/cogl-auto-texture.c index 29a5f73..fe52e15 100644 --- a/cogl/cogl/deprecated/cogl-auto-texture.c +++ b/cogl/cogl/deprecated/cogl-auto-texture.c @@ -94,6 +94,7 @@ cogl_texture_new_with_size (unsigned int width, if (!cogl_texture_allocate (tex, &skip_error)) { cogl_error_free (skip_error); + skip_error = NULL; cogl_object_unref (tex); tex = NULL; } -- 2.30.2