[Fixed Bug] Fixed alloc-dealloc-mismatch (new vs. free)

[cause]
 - allocate memory using operator new[]
 - deallocate the memory using free()
 - it may cause memory corruption or undefined behaviors

[solve]
 - replace operator new[] with malloc()

Change-Id: Icc844a98fda28556b071e62ad2599bacba441974
Signed-off-by: Tae-Young Chung <ty83.chung@samsung.com>

diff --git a/mv_common/src/mv_common_c.cpp b/mv_common/src/mv_common_c.cpp
index 9242a56..51e0dbb 100644 (file)
--- a/mv_common/src/mv_common_c.cpp
+++ b/mv_common/src/mv_common_c.cpp
@@ -22,6 +22,7 @@
 #include <mv_private.h>
 
 #include <string.h>
+#include <stdlib.h>
 #include <media_packet.h>
 
 int mv_create_source_c(
@@ -512,11 +513,11 @@ int mv_engine_config_get_string_attribute_c(
 
        LOGD("Convert string to char*");
        int stringSize = attributeValue.size();
-       (*value) = new char[stringSize + 1];
+       (*value) = (char *)malloc(sizeof(char) * (stringSize + 1));
 
        if (attributeValue.copy(*value, stringSize) != attributeValue.size()) {
                LOGE("Conversion from string to char* failed");
-               delete[] (*value);
+               free(*value);
                (*value) = NULL;
                return MEDIA_VISION_ERROR_INVALID_OPERATION;
        }

diff --git a/packaging/capi-media-vision.spec b/packaging/capi-media-vision.spec
index 3f33e5e..c9c241a 100644 (file)
--- a/packaging/capi-media-vision.spec
+++ b/packaging/capi-media-vision.spec
@@ -1,6 +1,6 @@
 Name:        capi-media-vision
 Summary:     Media Vision library for Tizen Native API
-Version:     0.3.16
+Version:     0.3.17
 Release:     1
 Group:       Multimedia/Framework
 License:     Apache-2.0 and BSD-2.0