diff options
author | Jeongmo Yang <jm80.yang@samsung.com> | 2017-10-17 19:24:49 +0900 |
---|---|---|
committer | Jeongmo Yang <jm80.yang@samsung.com> | 2017-10-17 19:27:14 +0900 |
commit | 3430241b93e8bd6e0519a0b9ce40428802872e60 (patch) | |
tree | baeb71e5d156498f947748a08cc95dc036327f36 | |
parent | 86667d8d1a03a193018a4b8b12201b2613695435 (diff) | |
download | recorder-3430241b93e8bd6e0519a0b9ce40428802872e60.tar.gz recorder-3430241b93e8bd6e0519a0b9ce40428802872e60.tar.bz2 recorder-3430241b93e8bd6e0519a0b9ce40428802872e60.zip |
Fix coverity issue(memory leak, return value check, etc)submit/tizen/20171031.024842accepted/tizen/unified/20171101.064718
[Version] 0.3.7
[Profile] Common
[Issue Type] Bug fix
[Dependency module] N/A
[Test] [M(T) - Boot=(OK), sdb=(OK), Home=(OK), Touch=(OK), Version=tizen-4.0-unified_20171013.1]
Change-Id: Idac2dd8ac51abea17349572e7acd28b3501aeb9c
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>
-rw-r--r-- | packaging/capi-media-recorder.spec | 2 | ||||
-rw-r--r-- | src/recorder.c | 5 | ||||
-rw-r--r-- | test/recorder_test.c | 8 |
3 files changed, 11 insertions, 4 deletions
diff --git a/packaging/capi-media-recorder.spec b/packaging/capi-media-recorder.spec index 88d7918..d2e2466 100644 --- a/packaging/capi-media-recorder.spec +++ b/packaging/capi-media-recorder.spec @@ -1,6 +1,6 @@ Name: capi-media-recorder Summary: A Recorder API -Version: 0.3.6 +Version: 0.3.7 Release: 0 Group: Multimedia/API License: Apache-2.0 diff --git a/src/recorder.c b/src/recorder.c index 614ebcc..d6c7936 100644 --- a/src/recorder.c +++ b/src/recorder.c @@ -1029,6 +1029,11 @@ static void *_recorder_msg_recv_func(gpointer data) LOGW("incompleted message [len %d]", remained_length); + if (remained_msg) { + free(remained_msg); + remained_msg = NULL; + } + remained_msg = (char *)malloc(remained_length + 1); if (remained_msg) { strncpy(remained_msg, recv_msg + prev_pos, remained_length); diff --git a/test/recorder_test.c b/test/recorder_test.c index ce48d36..c3d0bab 100644 --- a/test/recorder_test.c +++ b/test/recorder_test.c @@ -325,7 +325,7 @@ static void _recording_muxed_stream_cb(void *stream, int size, unsigned long lon g_print("\tRECORDING MUXED STREAM CB - %p, size %d, offset %llu\n", stream, size, offset); if (stream && size > 0) { - if (access(TARGET_FILENAME_MUXED_CB, F_OK)) + if (offset == 0) fp = fopen(TARGET_FILENAME_MUXED_CB, "w"); else fp = fopen(TARGET_FILENAME_MUXED_CB, "rb+"); @@ -335,9 +335,11 @@ static void _recording_muxed_stream_cb(void *stream, int size, unsigned long lon g_print("\tback to %llu\n", offset); } - fseeko(fp, (off_t)offset, SEEK_SET); + if (fseeko(fp, (off_t)offset, SEEK_SET) == 0) + fwrite(stream, 1, size, fp); + else + g_print("\nSeek Error : offset %llu\n", offset); - fwrite(stream, 1, size, fp); fclose(fp); fp = NULL; } else { |