diff options
author | Nanley Chery <nanley.g.chery@intel.com> | 2019-09-25 12:48:57 -0700 |
---|---|---|
committer | Dylan Baker <dylan@pnwbakers.com> | 2019-10-29 08:15:58 -0700 |
commit | 8cb1070ea3c7ea3b55d3a13b62443b8dbd3f16e2 (patch) | |
tree | 98ac9850f18787a9f9fd8aeee113d45d6e5e5519 | |
parent | 88ca3cf11dbf260c15db43d60df41a2de36d821a (diff) |
iris: Don't leak the resource for unsupported modifier
Make sure the res struct is free'd before returning.
Fixes: 2dce0e94a3d ("iris: Initial commit of a new 'iris' driver for Intel Gen8+ GPUs.")
Reviewed-by: Sagar Ghuge <sagar.ghuge@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
(cherry picked from commit f2fc5dece93019f0bab203247985cf350d541156)
-rw-r--r-- | src/gallium/drivers/iris/iris_resource.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gallium/drivers/iris/iris_resource.c b/src/gallium/drivers/iris/iris_resource.c index 53ecb805840..71f92baab49 100644 --- a/src/gallium/drivers/iris/iris_resource.c +++ b/src/gallium/drivers/iris/iris_resource.c @@ -717,7 +717,7 @@ iris_resource_create_with_modifiers(struct pipe_screen *pscreen, } else { if (modifiers_count > 0) { fprintf(stderr, "Unsupported modifier, resource creation failed.\n"); - return NULL; + goto fail; } /* No modifiers - we can select our own tiling. */ |