Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f951958e authored by Hsin-yu Chao's avatar Hsin-yu Chao
Browse files

blueooth/audio/aidl/default: Fix return code in startSession 

BluetoothAudioProvider::startSession() currently doesn't use the
return code from onSessionReady() and that basically hides all
exceptions return by the child implementation, e.g
A2dpSoftwareAudioProvider::onSessionReady(). This commit fixed
that so the errors are propagated to the binder service.

Bug: 399554337
Test: m android.hardware.bluetooth.audio-V1-impl
Change-Id: I37be56769ceb00caacf0085eef3033469be2a9a8
parent 63c25994

bluetooth/audio/aidl/default/BluetoothAudioProvider.cpp

+1 −2
Original line number Original line Diff line number Diff line
@@ -78,8 +78,7 @@ ndk::ScopedAStatus BluetoothAudioProvider::startSession( 
                       cookie);

                       cookie);





  LOG(INFO) << __func__ << " - SessionType=" << toString(session_type_);

  LOG(INFO) << __func__ << " - SessionType=" << toString(session_type_);

  onSessionReady(_aidl_return);

  return onSessionReady(_aidl_return);

  return ndk::ScopedAStatus::ok();

}

}





ndk::ScopedAStatus BluetoothAudioProvider::endSession() {

ndk::ScopedAStatus BluetoothAudioProvider::endSession() {