Modify memory leak issue for static analyzer warning

Change-Id: I5c676437319279504ea3303ab081d51104f4853e
Signed-off-by: Jiyong Min <jiyong.min@samsung.com>

diff --git a/gstcs/mm_util_gstcs.c b/gstcs/mm_util_gstcs.c
index cc1ce87..3ff6411 100755 (executable)
--- a/gstcs/mm_util_gstcs.c
+++ b/gstcs/mm_util_gstcs.c
@@ -611,7 +611,6 @@ _mm_imgp_gstcs_processing( gstreamer_s* pGstreamer_s, unsigned char *src, unsign
                gstcs_error("ERROR - g_main_loop_new ");
                gst_object_unref (pGstreamer_s->pipeline);
                g_main_context_unref(pGstreamer_s->context);
-               g_free (pGstreamer_s);
                return GSTCS_ERROR_INVALID_OPERATION;
        }
 
@@ -637,7 +636,6 @@ _mm_imgp_gstcs_processing( gstreamer_s* pGstreamer_s, unsigned char *src, unsign
                gst_object_unref (pGstreamer_s->pipeline);
                g_main_context_unref(pGstreamer_s->context);
                g_main_loop_unref(pGstreamer_s->loop);
-               g_free (pGstreamer_s);
                return ret;
        }
        gstcs_debug("End mm_push_buffer_into_pipeline");
@@ -720,7 +718,6 @@ _mm_imgp_gstcs_processing( gstreamer_s* pGstreamer_s, unsigned char *src, unsign
        pGstreamer_s->output_buffer = NULL;
        g_main_context_unref(pGstreamer_s->context);
        g_main_loop_unref(pGstreamer_s->loop);
-       g_free (pGstreamer_s);
 
        gstcs_debug("End gstreamer processing");
        gstcs_debug("dst: %p", dst);
@@ -813,8 +810,9 @@ _mm_imgp_gstcs(imgp_info_s* pImgp_info, unsigned char *src, unsigned char *dst)
        gstcs_debug("memset argv");
 
        /* add initial */
-       *argc = 1;
-       argv[0] = g_strdup( "mmutil_gstcs" );
+       *argc = 0;
+       argv[*argc] = (gchar *)g_strdup( "mmutil_gstcs" );
+       (*argc)++;
        /* check disable registry scan */
        argv[*argc] = g_strdup("--gst-disable-registry-update");
        (*argc)++;
@@ -836,6 +834,7 @@ _mm_imgp_gstcs(imgp_info_s* pImgp_info, unsigned char *src, unsigned char *dst)
        input_format= _mm_set_input_image_format_s_struct(pImgp_info);
        if (input_format == NULL) {
                gstcs_error("memory allocation failed");
+               g_free (pGstreamer_s);
                return GSTCS_ERROR_OUT_OF_MEMORY;
        }
        output_format= _mm_set_output_image_format_s_struct(pImgp_info, input_format);
@@ -844,6 +843,7 @@ _mm_imgp_gstcs(imgp_info_s* pImgp_info, unsigned char *src, unsigned char *dst)
                GSTCS_FREE(input_format->format_label);
                GSTCS_FREE(input_format->colorspace);
                GSTCS_FREE(input_format);
+               g_free (pGstreamer_s);
                return GSTCS_ERROR_OUT_OF_MEMORY;
        }
 
@@ -864,6 +864,7 @@ _mm_imgp_gstcs(imgp_info_s* pImgp_info, unsigned char *src, unsigned char *dst)
        GSTCS_FREE(output_format->format_label);
        GSTCS_FREE(output_format->colorspace);
        GSTCS_FREE(output_format);
+       g_free (pGstreamer_s);
 
        return ret;
 }

diff --git a/packaging/libmm-imgp-gstcs.spec b/packaging/libmm-imgp-gstcs.spec
index 8169650..dcd664d 100755 (executable)
--- a/packaging/libmm-imgp-gstcs.spec
+++ b/packaging/libmm-imgp-gstcs.spec
@@ -1,7 +1,7 @@
 #sbs-git:slp/pkgs/l/libmm-imgp-gstcs libmm-imgp-gstcs 0.1 62b62e6d483557fc5750d1b4986e9a98323f1194
 Name:       libmm-imgp-gstcs
 Summary:    Multimedia Framework Utility Library
-Version:    0.7
+Version:    0.8
 Release:    16
 Group:      System/Libraries
 License:    Apache-2.0