source: fix unintended deallocaton problem 55/66455/2
authorPark Kyoung Won <kw0712.park@samsung.com>
Tue, 19 Apr 2016 04:33:40 +0000 (13:33 +0900)
committerPark Kyoung Won <kw0712.park@samsung.com>
Wed, 27 Apr 2016 05:14:14 +0000 (14:14 +0900)
- CodecElements were deallocated after registering element data, but
  being used while encoding or decoding.

Change-Id: Ibc8f666232421d6f8017216adeb983a53c6339b5
Signed-off-by: Park Kyoung Won <kw0712.park@samsung.com>
src/gstmaru.c
src/gstmaruinterface3.c

index 9ae2c45..c744d7b 100644 (file)
@@ -137,19 +137,15 @@ plugin_init (GstPlugin *plugin)
   g_mutex_unlock (&gst_maru_mutex);
   if (!gst_maruviddec_register (plugin, elements)) {
     GST_ERROR ("failed to register decoder elements");
-    return FALSE;
   }
   if (!gst_maruauddec_register (plugin, elements)) {
     GST_ERROR ("failed to register decoder elements");
-    return FALSE;
   }
   if (!gst_maruvidenc_register (plugin, elements)) {
     GST_ERROR ("failed to register encoder elements");
-    return FALSE;
   }
   if (!gst_maruaudenc_register (plugin, elements)) {
     GST_ERROR ("failed to register encoder elements");
-    return FALSE;
   }
   return TRUE;
 }
@@ -164,7 +160,7 @@ GST_PLUGIN_DEFINE (
   tizen-emul,
   "Codecs for Tizen Emulator",
   plugin_init,
-  "1.2.0",
+  "1.2.2",
   "LGPL",
   "gst-plugins-emulator",
   "http://www.tizen.org"
index 63db30c..b108515 100644 (file)
@@ -670,6 +670,8 @@ prepare_elements (int fd)
   }
 
   elem = g_malloc(size);
+  //FIXME
+  //An elem variable should be deallocated, will be fixed.
 
   ret = ioctl (fd, IOCTL_RW(IOCTL_CMD_GET_ELEMENTS), elem);
   if (ret < 0) {
@@ -683,7 +685,6 @@ prepare_elements (int fd)
     elements = g_list_append (elements, &elem[i]);
   }
 
-  g_free (elem);
   return elements;
 }