diff options
| author | Michalis Spyrou <michalis.spyrou@arm.com> | 2019-07-05 16:03:36 +0100 |
|---|---|---|
| committer | Georgios Pinitas <georgios.pinitas@arm.com> | 2019-07-08 10:48:05 +0000 |
| commit | 74cd112addebc7678cb763cc1cb173feb0e61bb2 (patch) | |
| tree | 40524e9fab917bad9dea263e461af856b4f3d97b /src/core | |
| parent | ed51650a604e4dc7cc87fdeeefb2b01ad414f699 (diff) | |
| download | ComputeLibrary-74cd112addebc7678cb763cc1cb173feb0e61bb2.tar.gz | |
COMPMID-2425: Fix memory leak reported by coverity
Change-Id: Ia41626a59148978ea72bee9c5358f2748a29144c
Signed-off-by: Michalis Spyrou <michalis.spyrou@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1482
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Diffstat (limited to 'src/core')
| -rw-r--r-- | src/core/CL/OpenCL.cpp | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/core/CL/OpenCL.cpp b/src/core/CL/OpenCL.cpp index ef03a5a302..0212774161 100644 --- a/src/core/CL/OpenCL.cpp +++ b/src/core/CL/OpenCL.cpp @@ -35,6 +35,14 @@ CLSymbols &CLSymbols::get() return symbols; } +CLSymbols::~CLSymbols() +{ + if(handle) + { + dlclose(handle); + } +} + bool CLSymbols::load_default() { static const std::vector<std::string> libraries{ "libOpenCL.so", "libGLES_mali.so", "libmali.so" }; @@ -61,7 +69,7 @@ bool CLSymbols::load_default() bool CLSymbols::load(const std::string &library) { - void *handle = dlopen(library.c_str(), RTLD_LAZY | RTLD_LOCAL); + handle = dlopen(library.c_str(), RTLD_LAZY | RTLD_LOCAL); if(handle == nullptr) { |