From 635437f5fdcbd8274f1505ae0e2750ceca3060c5 Mon Sep 17 00:00:00 2001
From: Gil Pitney <gil.pitney@linaro.org>
Date: Wed, 17 Jun 2015 00:49:55 +0000
Subject: clCreateProgramWithBinary(): fix memory leak.

Memory for a temporary array of context devices was not getting freed.

Signed-off-by: Gil Pitney <gil.pitney@linaro.org>
---
 src/api/api_program.cpp | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/api/api_program.cpp b/src/api/api_program.cpp
index 5dd6350..4afd603 100644
--- a/src/api/api_program.cpp
+++ b/src/api/api_program.cpp
@@ -119,9 +119,10 @@ clCreateProgramWithBinary(cl_context            context,
                                  context_num_devices * sizeof(cl_device_id),
                                  context_devices, 0);
 
-    if (*errcode_ret != CL_SUCCESS)
+    if (*errcode_ret != CL_SUCCESS) {
+        std::free(context_devices);
         return 0;
-
+    }
     for (cl_uint i=0; i<num_devices; ++i)
     {
         bool found = false;
@@ -132,6 +133,7 @@ clCreateProgramWithBinary(cl_context            context,
                 binary_status[i] = CL_INVALID_VALUE;
 
             *errcode_ret = CL_INVALID_VALUE;
+            std::free(context_devices);
             return 0;
         }
 
@@ -147,6 +149,7 @@ clCreateProgramWithBinary(cl_context            context,
         if (!found)
         {
             *errcode_ret = CL_INVALID_DEVICE;
+            std::free(context_devices);
             return 0;
         }
     }
@@ -163,9 +166,11 @@ clCreateProgramWithBinary(cl_context            context,
     if (*errcode_ret != CL_SUCCESS)
     {
         delete program;
+        std::free(context_devices);
         return 0;
     }
 
+    std::free(context_devices);
     return (cl_program)program;
 }
 
-- 
cgit v1.2.3