From: Ji-hoon Lee <dalton.lee@samsung.com>
Date: Mon, 30 Sep 2019 08:28:25 +0000 (+0900)
Subject: Fix crash caused by uninitialized pointer variable
X-Git-Tag: submit/tizen/20190930.111916~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=67c6029364bbe170edea352aaf35d8acced773d8;p=platform%2Fcore%2Fuifw%2Fmulti-assistant-service.git

Fix crash caused by uninitialized pointer variable

Change-Id: I63c6fe9a457085544c013555e169b3cd1882df5e
---

diff --git a/plugins/wakeup-manager/src/wakeup_audio_manager.cpp b/plugins/wakeup-manager/src/wakeup_audio_manager.cpp
index d7ca9b0..8b68362 100644
--- a/plugins/wakeup-manager/src/wakeup_audio_manager.cpp
+++ b/plugins/wakeup-manager/src/wakeup_audio_manager.cpp
@@ -77,7 +77,7 @@ void CAudioManager::sound_focus_changed()
 {
 	sound_stream_focus_change_reason_e acquired_by;
 	int sound_behavior;
-	char* extra_info;
+	char* extra_info = NULL;
 	int focus = sound_manager_get_current_recording_focus(&acquired_by, &sound_behavior, &extra_info);
 	MWR_LOGD("[Recorder] sound focus has changed : %d %d %d %s", focus,
 		(SOUND_MANAGER_ERROR_NO_DATA != focus ? acquired_by : -1),
@@ -95,7 +95,7 @@ void CAudioManager::sound_focus_changed()
 		}
 	}
 	if (extra_info) {
-		free (extra_info);
+		free(extra_info);
 		extra_info = NULL;
 	}
 }
@@ -133,7 +133,7 @@ void CAudioManager::start_recording(bool proactive)
 
 	sound_stream_focus_change_reason_e acquired_by;
 	int sound_behavior;
-	char* extra_info;
+	char* extra_info = NULL;
 	if (SOUND_MANAGER_ERROR_NO_DATA == sound_manager_get_current_recording_focus(&acquired_by, &sound_behavior, &extra_info)) {
 		MWR_LOGD("[Recorder] Currently no other process has acquired sound focus, start recording");
 		dependency_resolver_start_recording();
@@ -142,7 +142,7 @@ void CAudioManager::start_recording(bool proactive)
 		MWR_LOGW("[Recorder] Currently sound focus is acquired by other process, skip recording");
 	}
 	if (extra_info) {
-		free (extra_info);
+		free(extra_info);
 		extra_info = NULL;
 	}
 }