From 74cd112addebc7678cb763cc1cb173feb0e61bb2 Mon Sep 17 00:00:00 2001 From: Michalis Spyrou Date: Fri, 5 Jul 2019 16:03:36 +0100 Subject: COMPMID-2425: Fix memory leak reported by coverity Change-Id: Ia41626a59148978ea72bee9c5358f2748a29144c Signed-off-by: Michalis Spyrou Reviewed-on: https://review.mlplatform.org/c/1482 Tested-by: Arm Jenkins Reviewed-by: Michele Di Giorgio Comments-Addressed: Arm Jenkins Reviewed-by: Georgios Pinitas --- arm_compute/core/CL/OpenCL.h | 12 ++++++++++++ src/core/CL/OpenCL.cpp | 10 +++++++++- 2 files changed, 21 insertions(+), 1 deletion(-) diff --git a/arm_compute/core/CL/OpenCL.h b/arm_compute/core/CL/OpenCL.h index fc7083d276..fbf603ea00 100644 --- a/arm_compute/core/CL/OpenCL.h +++ b/arm_compute/core/CL/OpenCL.h @@ -64,6 +64,17 @@ private: void load_symbols(void *handle); public: + /** Prevent instances of this class from being copied (As this class contains pointers) */ + CLSymbols(const CLSymbols &) = delete; + /** Prevent instances of this class from being copied (As this class contains pointers) */ + CLSymbols &operator=(const CLSymbols &) = delete; + /** Allow instances of this class to be move constructed */ + CLSymbols(CLSymbols &&) = default; + /** Allow instances of this class to be moved */ + CLSymbols &operator=(CLSymbols &&) = default; + /** Destructor */ + ~CLSymbols(); + /** Get the static instance of CLSymbols. * * @return The static instance of CLSymbols. @@ -138,6 +149,7 @@ public: private: std::pair _loaded{ false, false }; + void *handle{ nullptr }; }; } // namespace arm_compute #endif /* __ARM_COMPUTE_OPENCL_H__ */ 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 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) { -- cgit v1.2.1