Mesa (master): radv: fix memory leaks in radv_load_meta_pipeline()
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Wed Aug 15 14:21:41 UTC 2018
Module: Mesa
Branch: master
Commit: 85113c4d0507f91dd773b555b8e3f75423275010
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=85113c4d0507f91dd773b555b8e3f75423275010
Author: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Date: Wed Aug 15 15:28:43 2018 +0200
radv: fix memory leaks in radv_load_meta_pipeline()
Reported by Coverity.
Fixes: fbcd167314 ("radv: Add on-demand compilation of built-in shaders.")
Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
---
src/amd/vulkan/radv_meta.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_meta.c b/src/amd/vulkan/radv_meta.c
index ccdcb9ceb0..b8d0a7cfc7 100644
--- a/src/amd/vulkan/radv_meta.c
+++ b/src/amd/vulkan/radv_meta.c
@@ -263,6 +263,7 @@ radv_load_meta_pipeline(struct radv_device *device)
char path[PATH_MAX + 1];
struct stat st;
void *data = NULL;
+ bool ret = false;
if (!radv_builtin_cache_path(path))
return false;
@@ -278,11 +279,11 @@ radv_load_meta_pipeline(struct radv_device *device)
if(read(fd, data, st.st_size) == -1)
goto fail;
- return radv_pipeline_cache_load(&device->meta_state.cache, data, st.st_size);
+ ret = radv_pipeline_cache_load(&device->meta_state.cache, data, st.st_size);
fail:
free(data);
close(fd);
- return false;
+ return ret;
}
static void
More information about the mesa-commit
mailing list