fix condition when invalid format is set 14/73514/1 accepted/tizen/common/20160609.145228 accepted/tizen/ivi/20160609.091749 accepted/tizen/mobile/20160609.092007 accepted/tizen/tv/20160609.091632 accepted/tizen/wearable/20160609.091723 submit/tizen/20160609.040222
authorSejun Park <sejun79.park@samsung.com>
Wed, 8 Jun 2016 10:08:04 +0000 (19:08 +0900)
committerSejun Park <sejun79.park@samsung.com>
Wed, 8 Jun 2016 10:08:04 +0000 (19:08 +0900)
Change-Id: Iddfb84ed5594fb9a0fdb2328ece9acb0bc43af77

src/media_packet_pool.c

index 4ca5f560c9db5bcfb2a23730d5bff66e9929eecc..fce91de8d9c2c67d6479ba56015efad346b4bb55 100755 (executable)
@@ -61,22 +61,23 @@ int media_packet_pool_set_media_format(media_packet_pool_h pool, media_format_h
 {
        int ret = MEDIA_PACKET_ERROR_NONE;
        media_packet_pool_s *pool_handle = NULL;
-       gboolean is_writable = false;
+       media_format_type_e type;
 
        MEDIA_PACKET_POOL_INSTANCE_CHECK(pool);
        MEDIA_PACKET_POOL_NULL_ARG_CHECK(fmt);
 
-       media_format_is_writable(fmt, &is_writable);
-       if (!is_writable) {
-               LOGE("The media format handle is corrupted or Not set media info");
-               return MEDIA_PACKET_ERROR_INVALID_PARAMETER;
+       ret = media_format_get_type(fmt, &type);
+       if (ret == MEDIA_FORMAT_ERROR_INVALID_OPERATION || type == MEDIA_FORMAT_CONTAINER || type == MEDIA_FORMAT_TEXT) {
+               LOGE("Invaild media format");
+               return MEDIA_PACKET_ERROR_INVALID_OPERATION;
        }
+
        pool_handle = (media_packet_pool_s *) pool;
 
        /* increase format reference count */
        media_format_ref(fmt);
-
        pool_handle->fmt_h = fmt;
+
        return ret;
 }