fixed coverity issues 79/155779/2 submit/tizen/20171016.084218
authorSejun Park <sejun79.park@samsung.com>
Mon, 16 Oct 2017 06:50:01 +0000 (15:50 +0900)
committerSejun Park <sejun79.park@samsung.com>
Mon, 16 Oct 2017 07:03:11 +0000 (16:03 +0900)
Change-Id: Iebb50b802a5e286d8ac9480fff34e76b992a3496

packaging/capi-media-tool.spec
src/media_packet_pool.c
test/media_packet_test.c

index 856bc7ac10c019022e0bbff06d1c72c5e5a03102..79f3b212ea4cc6fc604e19e51d3dd4ee4a6a10b8 100755 (executable)
@@ -1,7 +1,7 @@
 Name:       capi-media-tool
 Summary:    A Core API media tool library in Tizen Native API
 Version:    0.1.8
-Release:    2
+Release:    3
 Group:      Multimedia/API
 License:    Apache-2.0
 Source0:    %{name}-%{version}.tar.gz
index 5776fcccd3b97a19edb78cca2a18cc39430a0f9a..ed6e15bf34f28c25b91d80df3af45419f421aee9 100755 (executable)
@@ -235,7 +235,10 @@ int media_packet_pool_acquire_packet(media_packet_pool_h pool, media_packet_h *p
                        break;
 
                } else {
-                       g_cond_wait_until(&pool_handle->queue_cond, &pool_handle->mutex, wait_until);
+                       if (!g_cond_wait_until(&pool_handle->queue_cond, &pool_handle->mutex, wait_until)) {
+                               LOGW("signal not received");
+                               break;
+                       }
                        LOGD("Queue is Empty, waiting for timeout %" G_GSIZE_FORMAT "", wait_until);
                        packet = g_queue_pop_head(pool_handle->queue);
                        if (!packet) {
index fc3f1743cd0497fb2593e34156f8493a879b9dab..89595945f7fb25df10ee170b2676144c1f9d161f 100755 (executable)
@@ -565,12 +565,6 @@ static void _media_packet_get_buffer_data_ptr(void)
 {
        void *ptr;
        media_packet_s *packet = g_media_packet[0];
-       ptr = (void *)malloc(sizeof(packet->size));
-
-       if (ptr)
-               memset(packet, 0, sizeof(packet->size));
-       else
-               g_print("_get_buffer_ptr , malloc failed...");
 
        if (media_packet_get_buffer_data_ptr(g_media_packet[0], &ptr) == MEDIA_PACKET_ERROR_NONE) {
                g_print("media_packet_get_buffer_data_ptr is sucess!!");