Merge pull request Samsung#2570 from zhouxinhe/fix-svace-issue

Taejun-Kwon · web-flow · commit 2422401e3111 · 2018-12-27T17:39:14.000+09:00
framework/media: Fix issues svace reported
diff --git a/framework/src/media/HttpStream.cpp b/framework/src/media/HttpStream.cpp
@@ -48,7 +48,7 @@ std::shared_ptr<HttpStream> HttpStream::create()
 }
 
 HttpStream::HttpStream() :
-	mCurl(nullptr), mHttpHeaders(nullptr)
+	mCurl(nullptr), mHttpHeaders(nullptr), mInitializeFlag(false)
 {
 }
 
diff --git a/framework/src/media/voice/HardwareEndPointDetector.cpp b/framework/src/media/voice/HardwareEndPointDetector.cpp
@@ -184,7 +184,7 @@ bool HardwareEndPointDetector::waitEndPoint(int timeout)
 		ret = sem_timedwait(&mSem, &waketime);
 	}
 
-	stop_stream_in_device_process_type(mSdCard, mSdDevice, AUDIO_DEVICE_SPEECH_DETECT_EPD);
+	result = stop_stream_in_device_process_type(mSdCard, mSdDevice, AUDIO_DEVICE_SPEECH_DETECT_EPD);
 	if (result != AUDIO_MANAGER_SUCCESS) {
 		meddbg("Error: stop_stream_in_device_process_type(%d, %d) failed!\n", mSdCard, mSdDevice);
 		return false;

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ std::shared_ptr<HttpStream> HttpStream::create()`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`HttpStream::HttpStream() :`
`51`		`- mCurl(nullptr), mHttpHeaders(nullptr)`
	`51`	`+ mCurl(nullptr), mHttpHeaders(nullptr), mInitializeFlag(false)`
`52`	`52`	`{`
`53`	`53`	`}`
`54`	`54`
Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,7 @@ bool HardwareEndPointDetector::waitEndPoint(int timeout)`
`184`	`184`	`ret = sem_timedwait(&mSem, &waketime);`
`185`	`185`	`}`
`186`	`186`
`187`		`- stop_stream_in_device_process_type(mSdCard, mSdDevice, AUDIO_DEVICE_SPEECH_DETECT_EPD);`
	`187`	`+ result = stop_stream_in_device_process_type(mSdCard, mSdDevice, AUDIO_DEVICE_SPEECH_DETECT_EPD);`
`188`	`188`	`if (result != AUDIO_MANAGER_SUCCESS) {`
`189`	`189`	`meddbg("Error: stop_stream_in_device_process_type(%d, %d) failed!\n", mSdCard, mSdDevice);`
`190`	`190`	`return false;`