[0.3.62] fix error of heap-buffer-overflow 09/133509/1
authorEunhae Choi <eunhae1.choi@samsung.com>
Mon, 12 Jun 2017 12:14:38 +0000 (21:14 +0900)
committerEunhae Choi <eunhae1.choi@samsung.com>
Mon, 12 Jun 2017 12:14:38 +0000 (21:14 +0900)
Change-Id: Ifb163fd87fc1cd79038d14d5b1ca2dc283ea2aa6

include/player_private.h
packaging/capi-media-player.spec
src/player.c

index acec3d0ab465a056c658d36283bff5c986f03f5f..01b2f286a9b095c1ccf0f9cc33954b917af40cd3 100644 (file)
@@ -35,7 +35,7 @@ extern "C" {
 #define PLAYER_CHECK_CONDITION(condition, error, msg)     \
 do {   \
        if (condition) {} else \
-       { LOGE("[%s] %s(0x%08x)", __FUNCTION__, msg, error); return error; }; \
+       { LOGE("[%s] %s(0x%08x)", __FUNCTION__, msg, error); return error; } \
 } while (0)
 
 #define PLAYER_INSTANCE_CHECK(player)   \
index 8d3a982b8119cf63d2316cc604621a0ef6239610..dd35593d462516616515272c4de246deaef05ba7 100644 (file)
@@ -1,6 +1,6 @@
 Name:       capi-media-player
 Summary:    A Media Player API
-Version:    0.3.61
+Version:    0.3.62
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
index 12e00f7f29dff8c50cabc8aec01738ef2ebe1bb7..04be22e1d976e0d651ad11bffdc0a02ac3114344 100644 (file)
@@ -266,6 +266,10 @@ static int player_recv_msg(callback_cb_info_s * cb_info, tbm_fd *tfd)
 
        memset(buff->recvMsg, 0x00, sizeof(char)*buff->bufLen);
        recvLen = muse_core_ipc_recv_fd_msg(cb_info->fd, buff->recvMsg, tfd);
+       if (recvLen <= 0) {
+               LOGE("failed to recv msg %d", recvLen);
+               return 0;
+       }
 
        /* check the first msg */
        if (buff->part_of_msg && buff->recvMsg[0] != '{')