From: InHong Han <inhong1.han@samsung.com>
Date: Fri, 17 Jun 2022 06:21:32 +0000 (+0900)
Subject: Update API descriptions
X-Git-Tag: submit/tizen/20220628.044215~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bbff0fc80e2542c92e18ca9978bb8396f4dd9c87;p=platform%2Fcore%2Fuifw%2Fmulti-assistant.git

Update API descriptions

Change-Id: Ibff44e843a9e6773d05f607dfb2c2724adc1fcb0
---

diff --git a/client/ma.c b/client/ma.c
index 85046fe..0308d9f 100644
--- a/client/ma.c
+++ b/client/ma.c
@@ -2460,7 +2460,7 @@ int ma_remove_wake_word(const char* wake_word, const char *language) {
 int ma_get_wakeup_info_wake_word(char** wake_word)
 {
 	if (wake_word) {
-		*wake_word = g_wakeup_info.wake_word;
+		*wake_word = strdup(g_wakeup_info.wake_word);
 	}
 	return 0;
 }
@@ -2468,7 +2468,9 @@ int ma_get_wakeup_info_wake_word(char** wake_word)
 int ma_get_wakeup_info_extra_data(unsigned char** extra_data, int* extra_data_length, char **extra_data_desc)
 {
 	if (extra_data) {
-		*extra_data = g_wakeup_info.extra_data;
+		*extra_data = (unsigned char*)malloc(g_wakeup_info.extra_data_length);
+		if (*extra_data)
+			memcpy(*extra_data, g_wakeup_info.extra_data, g_wakeup_info.extra_data_length);
 	}
 
 	if (extra_data_length) {
@@ -2476,7 +2478,7 @@ int ma_get_wakeup_info_extra_data(unsigned char** extra_data, int* extra_data_le
 	}
 
 	if (extra_data_desc) {
-		*extra_data_desc = g_wakeup_info.extra_data_desc;
+		*extra_data_desc = strdup(g_wakeup_info.extra_data_desc);
 	}
 
 	return 0;
diff --git a/include/multi_assistant.h b/include/multi_assistant.h
index 80c4caf..f5a2e1e 100644
--- a/include/multi_assistant.h
+++ b/include/multi_assistant.h
@@ -522,7 +522,7 @@ int ma_assistant_info_get_app_id(ma_assistant_info_h handle, char** app_id);
  * @since_tizen 5.5
  *
  * @param[in] handle The handle to the assistant's information
- * @param[in] status The enable status of the given assistant handle
+ * @param[out] status The enable status of the given assistant handle
  *
  * @return @c 0 on success, otherwise a negative error value
  * @retval #MA_ERROR_NONE Successful
@@ -844,8 +844,8 @@ int ma_remove_wake_word(const char* wake_word, const char *language);
  * @brief Retrieves wake word information of the last wakeup event.
  * @since_tizen 7.0
  *
- * @remarks You must not release @a wake_word using free().
- *          @a wake_word can be changed whenever @a ma_active_state_changed_cb is called.
+ * @remarks You must release @a wake_word using free().
+ *          @a wake_word can be changed whenever @c ma_active_state_changed_cb is called.
  * @param[out] wake_word The wake word information, if exists.
  *
  * @return @c 0 on success, otherwise a negative error value
@@ -867,8 +867,8 @@ int ma_get_wakeup_info_wake_word(char** wake_word);
  *          The information MUST be defined by both wakeup engine and voice assistant.
  * @since_tizen 7.0
  *
- * @remarks You must not release @a extra_data and @a extra_data_desc using free().
- *          @a extra_data can be changed whenever @a ma_active_state_changed_cb is called.
+ * @remarks You must release @a extra_data and @a extra_data_desc using free().
+ *          @a extra_data can be changed whenever @c ma_active_state_changed_cb is called.
  * @param[out] extra_data The extra data information, if exists.
  * @param[out] extra_data_length The length of extra data.
  * @param[out] extra_data_desc The string value describing the type of extra data.