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

Commit 10c57f4f authored by Jakub Pawlowski's avatar Jakub Pawlowski
Browse files

Security Manager: free data queue when channel stays unused 

Not freeing the channel causes test to crash

Bug: 155399771
Test: cert/run --host  --test_filter=LeSecurityTest
Change-Id: I850fabcc31cbef8d5682bc2c796f614a016acafb
parent 3ed1f736

system/gd/security/internal/security_manager_impl.cc

+3 −0
Original line number Diff line number Diff line
@@ -423,6 +423,9 @@ SecurityManagerImpl::SecurityManagerImpl(os::Handler* security_handler, l2cap::l 
void SecurityManagerImpl::OnPairingFinished(security::PairingResultOrFailure pairing_result) {

  LOG_INFO(" ■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ Received pairing result");



  pending_le_pairing_.channel_->GetQueueUpEnd()->UnregisterDequeue();

  pending_le_pairing_.enqueue_buffer_.reset();



  if (std::holds_alternative<PairingFailure>(pairing_result)) {

    PairingFailure failure = std::get<PairingFailure>(pairing_result);

    LOG_INFO(" ■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■ failure message: %s",