From: Marc-André Lureau <marcandre.lureau@redhat.com>
Date: Thu, 17 Feb 2022 07:36:45 +0000 (+0400)
Subject: ui/shader: fix potential leak of shader on error
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=8fe496adacc03fe37fba6765d34618a390fabdc8;p=qemu.git

ui/shader: fix potential leak of shader on error

Value of 0 for program and shaders are silently ignored and indicate error.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Acked-by: Gerd Hoffmann <kraxel@redhat.com>
---

diff --git a/ui/shader.c b/ui/shader.c
index e8b8d321b7..4c80fc831f 100644
--- a/ui/shader.c
+++ b/ui/shader.c
@@ -130,15 +130,17 @@ static GLuint qemu_gl_create_link_program(GLuint vert, GLuint frag)
 static GLuint qemu_gl_create_compile_link_program(const GLchar *vert_src,
                                                   const GLchar *frag_src)
 {
-    GLuint vert_shader, frag_shader, program;
+    GLuint vert_shader, frag_shader, program = 0;
 
     vert_shader = qemu_gl_create_compile_shader(GL_VERTEX_SHADER, vert_src);
     frag_shader = qemu_gl_create_compile_shader(GL_FRAGMENT_SHADER, frag_src);
     if (!vert_shader || !frag_shader) {
-        return 0;
+        goto end;
     }
 
     program = qemu_gl_create_link_program(vert_shader, frag_shader);
+
+end:
     glDeleteShader(vert_shader);
     glDeleteShader(frag_shader);