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

Commit fa4c6c7c authored by Łukasz Rymanowski's avatar Łukasz Rymanowski Committed by Automerger Merge Worker
Browse files

Merge "vc: Fix null pointer dereference" into main am: fa5781a4 am: f7bf63c1 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/3057904



Change-Id: I4ed5519ce449cbb667b78628538bea1c64c92bb5
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 26d8b3f9 f7bf63c1

system/bta/vc/types.h

+6 −0
Original line number Diff line number Diff line
@@ -89,6 +89,12 @@ struct VolumeOperation { 
  };



  ~VolumeOperation() {

    if (operation_timeout_ == nullptr) {

      log::warn("operation_timeout_ should not be null, id {}, device cnt {}",

                operation_id_, devices_.size());

      return;

    }



    if (alarm_is_scheduled(operation_timeout_))

      alarm_cancel(operation_timeout_);

system/bta/vc/vc.cc

+2 −0
Original line number Diff line number Diff line
@@ -681,6 +681,8 @@ class VolumeControlImpl : public VolumeControl { 
      return;

    }



    log::info("{}", remote_bda);



    bool notify = device->IsReady() || device->connecting_actively;

    device_cleanup_helper(device, notify);