Throw invalid operation error when unprepare() is called at inside of pulse-thread
[platform/core/api/audio-io.git] / src / cpp / CAudioInput.cpp
index 59eeb40..903ee7d 100644 (file)
@@ -84,12 +84,14 @@ void CAudioInput::onInterrupt(CAudioSessionHandler* pHandler, int id, mm_sound_f
     assert(pHandler);
     AUDIO_IO_LOGD("[pHandler:%p], [focus_type:%d], [state:%d], [reason_for_change:%s], [additional_info:%s]",
                    pHandler, focus_type, state, reason_for_change, additional_info);
+
     CAudioIO::onInterrupt(pHandler, id, focus_type, state, reason_for_change, additional_info);
 }
 
 void CAudioInput::onSignal(CAudioSessionHandler* pHandler, mm_sound_signal_name_t signal, int value) {
     assert(pHandler);
     AUDIO_IO_LOGD("[pHandler:%p], [signal:%d], [value:%d]", pHandler, signal, value);
+
     CAudioIO::onSignal(pHandler, signal, value);
 }
 
@@ -150,6 +152,7 @@ static bool __IsPrivilegeAllowed() {
     prData.paMainloop = pa_threaded_mainloop_new();
     if (prData.paMainloop == NULL)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_OUT_OF_MEMORY, "Failed pa_threaded_mainloop_new()");
+
     c = pa_context_new(pa_threaded_mainloop_get_api(prData.paMainloop), CLIENT_NAME);
     if (c == NULL)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_OUT_OF_MEMORY, "Failed pa_context_new()");
@@ -203,31 +206,36 @@ static bool __IsPrivilegeAllowed() {
 }
 
 void CAudioInput::initialize() {
-    if (__IsInit() == true) {
+    if (__IsInit() == true)
         return;
-    }
 
     try {
         CAudioIO::initialize();
-        if (__IsPrivilegeAllowed() == false) {
-            THROW_ERROR_MSG(CAudioError::EError::ERROR_PERMISSION_DENIED, "No privilege for record");
-        }
+    } catch (CAudioError& e) {
+        finalize();
+        throw;
+    }
 
-        // Create ASM Handler
-        mpAudioSessionHandler = new CAudioSessionHandler(CAudioSessionHandler::EAudioSessionType::AUDIO_SESSION_TYPE_CAPTURE, mAudioInfo, this);
+    if (__IsPrivilegeAllowed() == false)
+        THROW_ERROR_MSG(CAudioError::EError::ERROR_PERMISSION_DENIED, "No privilege for record");
 
-        // Initialize ASM Handler
+    try {
+        internalSessionLock();
+        mpAudioSessionHandler = new CAudioSessionHandler(CAudioSessionHandler::EAudioSessionType::AUDIO_SESSION_TYPE_CAPTURE, mAudioInfo, this);
         mpAudioSessionHandler->initialize();
-
-        __setInit(true);
-        CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
+        internalSessionUnlock();
     } catch (CAudioError& e) {
+        internalSessionUnlock();
         finalize();
         throw;
     } catch (const std::bad_alloc&) {
+        internalSessionUnlock();
         finalize();
         THROW_ERROR_MSG(CAudioError::EError::ERROR_OUT_OF_MEMORY, "Failed to allocate CAudioSessionHandler object");
     }
+
+    __setInit(true);
+    CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
 }
 
 void CAudioInput::finalize() {
@@ -236,8 +244,10 @@ void CAudioInput::finalize() {
         return;
     }
 
+    internalSessionLock();
     SAFE_FINALIZE(mpAudioSessionHandler);
     SAFE_DELETE(mpAudioSessionHandler);
+    internalSessionUnlock();
 
     CAudioIO::finalize();
 
@@ -245,9 +255,8 @@ void CAudioInput::finalize() {
 }
 
 void CAudioInput::prepare() {
-    if (__IsInit() == false) {
+    if (__IsInit() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput");
-    }
 
     if (__IsReady() == true) {
         AUDIO_IO_LOGD("Already prepared CAudioInput");
@@ -257,25 +266,36 @@ void CAudioInput::prepare() {
 
     /* Check invalid AudioType */
     CAudioInfo::EAudioType audioType = mAudioInfo.getAudioType();
-    if (audioType < CAudioInfo::EAudioType::AUDIO_IN_TYPE_MEDIA || audioType >= CAudioInfo::EAudioType::AUDIO_OUT_TYPE_MEDIA) {
+    if (audioType < CAudioInfo::EAudioType::AUDIO_IN_TYPE_MEDIA ||
+        audioType >= CAudioInfo::EAudioType::AUDIO_OUT_TYPE_MEDIA)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
                                "The audioType is invalid [type:%d]", static_cast<int>(audioType));
-    }
 
     try {
-        if (mpAudioSessionHandler->getId() < 0) {
-            if (isForceIgnore() == false && mpAudioSessionHandler->isSkipSession() == false) {
-                /* Register ASM Listener */
+        internalSessionLock();
+        if (mpAudioSessionHandler &&
+            mpAudioSessionHandler->getId() < 0) {  // Did not registerSound()
+            if (isForceIgnore() == false &&
+                mpAudioSessionHandler->isSkipSession() == false) {
                 AUDIO_IO_LOGD("Register ASM Listener");
                 mpAudioSessionHandler->registerSound();
             }
         }
+        internalSessionUnlock();
+    } catch (CAudioError& e) {
+        internalSessionUnlock();
+        throw;
+    }
 
-        CAudioIO::setInternalStreamInfo();
+    CAudioIO::setInternalStreamInfo();
 
-        if (isForceIgnore() == false && mpAudioSessionHandler->isSkipSession() == false)
+    try {
+        internalSessionLock();
+        if (isSessionEnabled() && isForceIgnore() == false)
             mpAudioSessionHandler->updatePlaying();
+        internalSessionUnlock();
     } catch (CAudioError& e) {
+        internalSessionUnlock();
         throw;
     }
 
@@ -286,17 +306,12 @@ void CAudioInput::prepare() {
         CPulseStreamSpec spec(streamSpec, mAudioInfo);
 
         internalLock();
-
-        /* Create PulseAudio Handler */
         mpPulseAudioClient = new CPulseAudioClient(CPulseAudioClient::EStreamDirection::STREAM_DIRECTION_RECORD, spec, this);
-
-        /* Initialize PulseAudio Handler */
         mpPulseAudioClient->initialize();
 #ifndef DISABLE_MOBILE_BACK_COMP
         /* Uncork stream which is created with CORKED flag */
         mpPulseAudioClient->cork(false);
 #endif
-
         internalUnlock();
 
         CAudioIO::prepare();
@@ -312,24 +327,23 @@ void CAudioInput::prepare() {
 }
 
 void CAudioInput::unprepare() {
-    if (__IsInit() == false) {
+    if (__IsInit() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize CAudioInput");
-    }
 
     if (__IsReady() == false) {
         AUDIO_IO_LOGD("Already unprepared");
         return;
     }
 
-    try {
-        CAudioIO::unprepare();
+    CAudioIO::unprepare();
 
+    try {
         internalLock();
-
+        if (mpPulseAudioClient && mpPulseAudioClient->isInThread())
+            THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't unprepare inside pulseaudio thread");
         SAFE_FINALIZE(mpPulseAudioClient);
         SAFE_DELETE(mpPulseAudioClient);
-
         internalUnlock();
     } catch (CAudioError& e) {
         internalUnlock();
@@ -337,64 +351,48 @@ void CAudioInput::unprepare() {
     }
 
     try {
-        if (mpAudioSessionHandler->getId() >= 0) {
-            if (isForceIgnore() == false && mpAudioSessionHandler->isSkipSession() == false)
-                mpAudioSessionHandler->updateStop();
-        }
+        internalSessionLock();
+        if (isSessionEnabled() && isForceIgnore() == false)
+            mpAudioSessionHandler->updateStop();
+        internalSessionUnlock();
 
         CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
     } catch (CAudioError& e) {
-        internalUnlock();
+        internalSessionUnlock();
         throw;
     }
 }
 
 void CAudioInput::pause() {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioInput");
-    }
 
-    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING) {
+    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_STATE,
                         "Can't pause if not in Running state");
-    }
 
-    if (mpPulseAudioClient->isInThread() == true) {
+    if (mpPulseAudioClient->isInThread() == true)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't pause in thread");
-    }
-
-    try {
-        CAudioIO::pause();
 
-        CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED);
-    } catch (CAudioError& e) {
-        throw;
-    }
+    CAudioIO::pause();
+    CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED);
 }
 
 void CAudioInput::resume() {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioInput");
-    }
 
-    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED) {
+    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_STATE,
                         "Can't resume if not in Paused state");
-    }
 
-    if (mpPulseAudioClient->isInThread() == true) {
+    if (mpPulseAudioClient->isInThread() == true)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't resume in thread");
-    }
-
-    try {
-        CAudioIO::resume();
 
-        CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING);
-    } catch (CAudioError& e) {
-        throw;
-    }
+    CAudioIO::resume();
+    CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING);
 }
 
 void CAudioInput::drain() {
@@ -402,69 +400,58 @@ void CAudioInput::drain() {
 }
 
 void CAudioInput::flush() {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioInput");
-    }
 
-    try {
-        CAudioIO::flush();
-    } catch (CAudioError& e) {
-        throw;
-    }
+    CAudioIO::flush();
 }
 
 int CAudioInput::getBufferSize() {
-    if (__IsInit() == false) {
+    if (__IsInit() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput");
-    }
 
     /* FIXME : return calculated size here to satisfy backward compatibility */
     return (mAudioInfo.getSampleRate() * DEFAULT_PERIOD_SIZE) / 1000 * mAudioInfo.getSampleSize();
 }
 
 void CAudioInput::setStreamCallback(SStreamCallback callback) {
-    if (__IsInit() == false) {
+    if (__IsInit() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioInput");
-    }
 
-    if (callback.onStream == NULL) {
-        AUDIO_IO_LOGD("__mIsUsedSyncRead = true");
+    if (callback.onStream == NULL)
         __mIsUsedSyncRead = true;
-    } else {
-        AUDIO_IO_LOGD("__mIsUsedSyncRead = false");
+    else
         __mIsUsedSyncRead = false;
-    }
+    AUDIO_IO_LOGD("__mIsUsedSyncRead = %d", __mIsUsedSyncRead);
 
     CAudioIO::setStreamCallback(callback);
 }
 
 size_t CAudioInput::read(void* buffer, size_t length) {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioInput");
-    }
-    if (buffer == NULL) {
+
+    if (buffer == NULL)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
                                "Parameters are NULL buffer:%p", buffer);
-    }
-    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING) {
+
+    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION,
                         "Can't read if not in Running state");
-    }
+
     /* Checks synchronous flag */
-    if (__mIsUsedSyncRead == false) {
+    if (__mIsUsedSyncRead == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION,
                         "Invalid operation of read() if receive stream callback");
-    }
 
     int ret = 0;
 
     try {
         internalLock();
-        if (mIsInterrupted) {
+        if (mIsInterrupted)
             THROW_ERROR_MSG(CAudioError::EError::ERROR_POLICY_BLOCKED, "audio-io is interrupted");
-        }
 
         // If another thread did call unprepare, do not read
         if (mpPulseAudioClient == NULL)
@@ -473,8 +460,8 @@ size_t CAudioInput::read(void* buffer, size_t length) {
 
         // Block until read done
         ret = mpPulseAudioClient->read(buffer, length);
-
         internalUnlock();
+
         sched_yield();
     } catch (CAudioError& e) {
         internalUnlock();
@@ -485,52 +472,31 @@ size_t CAudioInput::read(void* buffer, size_t length) {
 }
 
 int CAudioInput::peek(const void** buffer, size_t* length) {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioInput");
-    }
 
-    if (buffer == NULL || length == NULL) {
+    if (buffer == NULL || length == NULL)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
                                "Parameters are NULL buffer:%p, length:%p", buffer, length);
-    }
 
     /* Checks synchronous flag */
-    if (__mIsUsedSyncRead == true) {
+    if (__mIsUsedSyncRead == true)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION,
                         "Invalid operation of peek() if does not receive a stream callback");
-    }
-
-    int ret = 0;
 
-    try {
-        ret = mpPulseAudioClient->peek(buffer, length);
-    } catch (CAudioError& e) {
-        throw;
-    }
-
-    return ret;
+    return mpPulseAudioClient->peek(buffer, length);
 }
 
 int CAudioInput::drop() {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioInput");
-    }
 
     /* Checks synchronous flag */
-    if (__mIsUsedSyncRead == true) {
+    if (__mIsUsedSyncRead == true)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION,
                         "Invalid operation of drop() if does not receive a stream callback");
-    }
 
-    int ret = 0;
-
-    try {
-        ret = mpPulseAudioClient->drop();
-    } catch (CAudioError& e) {
-        throw;
-    }
-
-    return ret;
+    return mpPulseAudioClient->drop();
 }