pan/mdg: Fix memory leak on error path.
authorVinson Lee <vlee@freedesktop.org>
Fri, 25 Sep 2020 01:32:49 +0000 (18:32 -0700)
committerMarge Bot <eric+marge@anholt.net>
Mon, 28 Sep 2020 23:14:18 +0000 (23:14 +0000)
Fix defect reported by Coverity Scan.

Resource leak (RESOURCE_LEAK)
leaked_storage: Variable l going out of scope leaks the storage it
points to.

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Kristian H. Kristensen <hoegsberg@google.com>
Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6858>

src/panfrost/midgard/midgard_ra.c

index 6eb7368..1e511ad 100644 (file)
@@ -971,6 +971,7 @@ mir_ra(compiler_context *ctx)
 
                         if (spill_node == -1) {
                                 fprintf(stderr, "ERROR: Failed to choose spill node\n");
+                                lcra_free(l);
                                 return;
                         }