Use strdup() for the out parameter result of ms_node_get_param_value() 32/237632/1
authorSangchul Lee <sc11.lee@samsung.com>
Thu, 2 Jul 2020 04:31:40 +0000 (13:31 +0900)
committerSangchul Lee <sc11.lee@samsung.com>
Thu, 2 Jul 2020 04:42:25 +0000 (13:42 +0900)
It will be passed to the out parameter of media_streamer_node_get_param()
which needs to be released with free() by user.

Missing '()' is also added to @pre description of media_streamer_node_pull_packet().
It will create a link in the HTML documentation.

[Version] 0.1.82
[Issue Type] Improvement

Change-Id: Icc7eea60bb54f9083105b6da876e155613d828f4
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
include/media_streamer.h
packaging/capi-media-streamer.spec
src/media_streamer_node.c

index 6850db7..c328980 100644 (file)
@@ -1053,7 +1053,7 @@ int media_streamer_node_create_sink(media_streamer_node_sink_type_e type, media_
  * @retval #MEDIA_STREAMER_ERROR_INVALID_OPERATION Invalid operation
  * @retval #MEDIA_STREAMER_ERROR_INVALID_STATE Invalid state
  * @pre Create a sink node handle by calling media_streamer_node_create_sink().
- * @pre Set media_streamer_sink_data_ready_cb by calling media_streamer_sink_set_data_ready_cb().
+ * @pre Set media_streamer_sink_data_ready_cb() by calling media_streamer_sink_set_data_ready_cb().
  * @see #media_packet_h
  * @see media_streamer_node_create_sink()
  */
index 3b93ab1..1a52369 100644 (file)
@@ -1,6 +1,6 @@
 Name:       capi-media-streamer
 Summary:    A Media Streamer API
-Version:    0.1.81
+Version:    0.1.82
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
index f179398..11aac47 100644 (file)
@@ -1977,7 +1977,8 @@ int ms_node_get_param_value(media_streamer_node_s *node, param_s *param, char **
                string_val = g_strdup_printf("%d", g_value_get_enum(&value));
        }
 
-       *string_value = string_val;
+       *string_value = strdup(string_val);
+       g_free(string_val);
 
        g_value_reset(&value);
        g_value_unset(&value);