[UTC][capi-media-streamer][fix UTC SVACE issue]
authorHyunil <hyunil46.park@samsung.com>
Mon, 2 Apr 2018 01:03:53 +0000 (10:03 +0900)
committerHyunil <hyunil46.park@samsung.com>
Tue, 3 Apr 2018 00:01:19 +0000 (09:01 +0900)
Change-Id: I5aae5846738d479b39433e3f120d9b0c5815c73e
Signed-off-by: Hyunil <hyunil46.park@samsung.com>
src/utc/capi-media-streamer/utc-capi-media-streamer.c

index 1a53703..4c98429 100755 (executable)
@@ -80,6 +80,7 @@ void utc_capi_media_streamer_startup(void)
 
        char *test_file = FILE_NAME;
        char *test_playlist = FILE_PATH_PLAYLIST;
+       int ret = MEDIA_STREAMER_ERROR_NONE;
 
        char pszValue[CONFIG_VALUE_LEN_MAX] = {0,};
        if ( true == GetValueForTCTSetting("DEVICE_SUITE_TARGET_30", pszValue, API_NAMESPACE ))
@@ -87,15 +88,31 @@ void utc_capi_media_streamer_startup(void)
                PRINT_UTC_LOG("[Line : %d][%s] 'DEVICE_SUITE_TARGET_30' Values Received = %s\\n", __LINE__, API_NAMESPACE,  pszValue);
                unsigned int size_of_path = strlen(pszValue) + strlen(test_file) + 6;
                file_path = (char*)malloc(size_of_path);
+               if (!file_path) {
+                       PRINT_UTC_LOG("memory is not alloced");
+                       return;
+               }
                snprintf(file_path, size_of_path, "%s/res/%s", pszValue, test_file);
                unsigned int size_of_path_playlist = strlen(pszValue) + strlen(test_playlist) + 6;
                file_path_playlist = (char*)malloc(size_of_path_playlist);
+               if (!file_path_playlist) {
+                       PRINT_UTC_LOG("memory is not alloced");
+                       if (file_path) {
+                               free (file_path);
+                               file_path = NULL;
+                       }
+                       return;
+               }
                snprintf(file_path_playlist, size_of_path_playlist, "%s/res/%s", pszValue, test_playlist);
        } else {
                PRINT_UTC_LOG("[Line : %d][%s] GetValueForTCTSetting returned error for 'DEVICE_SUITE_TARGET_30'\\n", __LINE__, API_NAMESPACE);
        }
 
-       media_streamer_create(&streamer);
+       ret = media_streamer_create(&streamer);
+       if (ret != MEDIA_STREAMER_ERROR_NONE) {
+               PRINT_UTC_LOG("fail to create mediastreamer");
+               return;
+       }
 
        /* Define video raw format */
        media_format_create(&vfmt_raw);
@@ -172,12 +189,20 @@ void utc_capi_media_streamer_startup(void)
  */
 void utc_capi_media_streamer_cleanup(void)
 {
+       int ret = MEDIA_STREAMER_ERROR_NONE;
+
        media_format_unref(vfmt_encoded);
        media_format_unref(vfmt_raw);
        media_format_unref(afmt_encoded);
        media_format_unref(afmt_raw);
-       media_streamer_unprepare(streamer);
-       media_streamer_destroy(streamer);
+
+       ret = media_streamer_unprepare(streamer);
+       if (ret != MEDIA_STREAMER_ERROR_NONE)
+               PRINT_UTC_LOG("fail to unprepare mediastreamer");
+
+       ret = media_streamer_destroy(streamer);
+       if (ret != MEDIA_STREAMER_ERROR_NONE)
+               PRINT_UTC_LOG("fail to destroy mediastreamer");
 
        /* Clean Up Nodes */
        int i = g_node_counter - 1;
@@ -186,6 +211,11 @@ void utc_capi_media_streamer_cleanup(void)
                g_nodes[i] = NULL;
        }
        g_node_counter = 0;
+
+       if (file_path)
+               free (file_path);
+       if (file_path_playlist)
+               free (file_path_playlist);
 }
 
 /* Application source callback */
@@ -478,8 +508,10 @@ int utc_media_streamer_play_p2(void)
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
        assert_neq(rtp_bin, NULL);
        APPEND_NODE(rtp_bin);
-       media_streamer_node_set_param(rtp_bin, MEDIA_STREAMER_PARAM_VIDEO_OUT_PORT, RTP_VIDEO_PORT);
-       media_streamer_node_set_param(rtp_bin, MEDIA_STREAMER_PARAM_HOST, "localhost");
+       ret =media_streamer_node_set_param(rtp_bin, MEDIA_STREAMER_PARAM_VIDEO_OUT_PORT, RTP_VIDEO_PORT);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
+       ret = media_streamer_node_set_param(rtp_bin, MEDIA_STREAMER_PARAM_HOST, "localhost");
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
        ret = media_streamer_node_add(streamer, rtp_bin);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
@@ -532,8 +564,10 @@ int utc_media_streamer_play_p2(void)
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
        assert_neq(client_rtp_bin, NULL);
        APPEND_NODE(client_rtp_bin);
-       media_streamer_node_set_param(client_rtp_bin, MEDIA_STREAMER_PARAM_VIDEO_IN_PORT, RTP_VIDEO_PORT);
-       media_streamer_node_set_pad_format(client_rtp_bin, "video_in_rtp", vfmt_encoded);
+       ret = media_streamer_node_set_param(client_rtp_bin, MEDIA_STREAMER_PARAM_VIDEO_IN_PORT, RTP_VIDEO_PORT);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
+       ret = media_streamer_node_set_pad_format(client_rtp_bin, "video_in_rtp", vfmt_encoded);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
        ret = media_streamer_node_add(client, client_rtp_bin);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
@@ -617,8 +651,8 @@ int utc_media_streamer_play_n2(void)
        assert_neq(src, NULL);
        APPEND_NODE(src);
 
-       media_streamer_node_set_param(src, MEDIA_STREAMER_PARAM_URI, "/some/invalid/path");
-       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
+       ret = media_streamer_node_set_param(src, MEDIA_STREAMER_PARAM_URI, "/some/invalid/path");
+       assert_eq(ret, MEDIA_STREAMER_ERROR_INVALID_PARAMETER);
 
        media_streamer_node_h sink = NULL;
        ret = media_streamer_node_create_sink(MEDIA_STREAMER_NODE_SINK_TYPE_FAKE, &sink);
@@ -723,7 +757,8 @@ int utc_media_streamer_set_position_p(void)
        assert_neq(video_src, NULL);
        APPEND_NODE(video_src);
 
-       media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       ret = media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
        ret = media_streamer_node_add(streamer, video_src);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
@@ -769,7 +804,8 @@ int utc_media_streamer_set_position_n1(void)
        assert_neq(video_src, NULL);
        APPEND_NODE(video_src);
 
-       media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       ret = media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
        ret = media_streamer_node_add(streamer, video_src);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
@@ -863,7 +899,8 @@ int utc_media_streamer_get_position_p(void)
        assert_neq(video_src, NULL);
        APPEND_NODE(video_src);
 
-       media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       ret = media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
        ret = media_streamer_node_add(streamer, video_src);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
@@ -909,7 +946,8 @@ int utc_media_streamer_get_position_n(void)
        assert_neq(video_src, NULL);
        APPEND_NODE(video_src);
 
-       media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       ret = media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
        ret = media_streamer_node_add(streamer, video_src);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
@@ -955,7 +993,8 @@ int utc_media_streamer_get_duration_p1(void)
        assert_neq(video_src, NULL);
        APPEND_NODE(video_src);
 
-       media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       ret = media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
        ret = media_streamer_node_add(streamer, video_src);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
@@ -1050,7 +1089,8 @@ int utc_media_streamer_get_duration_n(void)
        assert_neq(video_src, NULL);
        APPEND_NODE(video_src);
 
-       media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       ret = media_streamer_node_set_param(video_src,MEDIA_STREAMER_PARAM_URI, file_path);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
 
        ret = media_streamer_node_add(streamer, video_src);
        assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
@@ -1393,6 +1433,7 @@ int utc_media_streamer_node_push_packet_n1(void)
 
        media_streamer_node_h custom_src = NULL;
        ret = media_streamer_node_create_src(MEDIA_STREAMER_NODE_SRC_TYPE_CUSTOM, &custom_src);
+       assert_eq(ret, MEDIA_STREAMER_ERROR_NONE);
        assert_neq(custom_src, NULL);
        APPEND_NODE(custom_src);