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

Commit d7540f74 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "csis: Fix possible assert" into main

parents 28e5177e e6c62c59

system/bta/csis/csis_client.cc

+3 −1
Original line number Diff line number Diff line
@@ -2327,7 +2327,9 @@ void CsisClient::Initialize(bluetooth::csis::CsisClientCallbacks* callbacks, Clo 
bool CsisClient::IsCsisClientRunning() { return instance; }



CsisClient* CsisClient::Get(void) {

  log::assert_that(instance != nullptr, "assert failed: instance != nullptr");

  if (instance == nullptr) {

    log::error("instance not available");

  }

  return instance;

}

system/bta/csis/csis_client_test.cc

+1 −1
Original line number Diff line number Diff line
@@ -630,7 +630,7 @@ protected: 
  const RawAddress test_address2 = GetTestAddress(1);

};



TEST_F(CsisClientTest, test_get_uninitialized) { ASSERT_DEATH(CsisClient::Get(), ""); }

TEST_F(CsisClientTest, test_get_uninitialized) { ASSERT_EQ(CsisClient::Get(), nullptr); }



TEST_F(CsisClientTest, test_initialize) {

  CsisClient::Initialize(callbacks.get(), base::DoNothing());