diff options
| author | Nanley Chery <nanley.g.chery@intel.com> | 2019-09-25 12:48:57 -0700 |
|---|---|---|
| committer | Nanley Chery <nanley.g.chery@intel.com> | 2019-10-28 10:47:06 -0700 |
| commit | f2fc5dece93019f0bab203247985cf350d541156 (patch) | |
| tree | 3ec8d063d3869584928752d58c309f84fdd38ece | |
| parent | 7a619b5c759e9d1a9795d4d79c7746ec47a00431 (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>
| -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 ecf63f49ade..a45bcdb71ae 100644 --- a/src/gallium/drivers/iris/iris_resource.c +++ b/src/gallium/drivers/iris/iris_resource.c @@ -781,7 +781,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; } /* Use linear for staging buffers */ |