511kinderheim/device_xiaomi_sm6150-common
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

sm6150-common: gps: Add NULL check before object access

Browse Source 
Correct the NULL check code block and add NULL check before
object access.

Change-Id: Ic41b781b41fb4e21bbff8801d500a41a6d7219d0
CRs-fixed: 3084543
This commit is contained in:
Madhanraj Chelladurai 2023-09-01 18:22:06 +02:00 committed by basamaryan
parent 29be33dd64
commit 0c0ae20c68
No known key found for this signature in database
GPG Key ID: 0A28C6D8648B81DD
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gps/android/utils/battery_listener.cpp
View File

@ -184,7 +184,7 @@ BatteryListenerImpl::~BatteryListenerImpl()
{ {
{ {
std::lock_guard<std::mutex> _l(mLock); std::lock_guard<std::mutex> _l(mLock);
if (mHealth != NULL) if (mHealth != NULL) {
mHealth->unregisterCallback(this); mHealth->unregisterCallback(this);
auto r = mHealth->unlinkToDeath(this); auto r = mHealth->unlinkToDeath(this);
if (!r.isOk() || r == false) { if (!r.isOk() || r == false) {
@ -192,9 +192,12 @@ BatteryListenerImpl::~BatteryListenerImpl()
r.description().c_str()); r.description().c_str());
} }
} }
}
mDone = true; mDone = true;
if (NULL != mThread) {
mThread->join(); mThread->join();
} }
}
void BatteryListenerImpl::serviceDied(uint64_t cookie __unused, void BatteryListenerImpl::serviceDied(uint64_t cookie __unused,
const wp<hidl::base::V1_0::IBase>& who) const wp<hidl::base::V1_0::IBase>& who)
@ -210,7 +213,9 @@ void BatteryListenerImpl::serviceDied(uint64_t cookie __unused,
} }
mHealth = NULL; mHealth = NULL;
mCond.notify_one(); mCond.notify_one();
if (NULL != mThread) {
mThread->join(); mThread->join();
}
std::lock_guard<std::mutex> _l(mLock); std::lock_guard<std::mutex> _l(mLock);
init(); init();
} }