efl_ui_selection_manager: free allocated memory after local use
authorStefan Schmidt <s.schmidt@samsung.com>
Fri, 20 Sep 2019 08:46:02 +0000 (10:46 +0200)
committerJongmin Lee <jm105.lee@samsung.com>
Mon, 23 Sep 2019 21:32:19 +0000 (06:32 +0900)
This temporary allocation is only used in this block and goes out of
scope when leaving. Make sure we free the resource before this happens.

Coverity ID: 1402653

Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10045

src/lib/elementary/efl_ui_selection_manager.c

index 2e7be5c..c42394c 100644 (file)
@@ -2454,6 +2454,7 @@ _wl_general_converter(char *target, Sel_Manager_Selection *sel, void *data, int
                   memcpy(tmp, data, size);
                   if (data_ret) *data_ret = tmp;
                   if (size_ret) *size_ret = size;
+                  if (!data_ret) free(tmp);
                }
           }
         else