diff options
| author | jitiphil <jitiphil@codeaurora.org> | 2018-04-03 16:52:52 +0530 |
|---|---|---|
| committer | nshrivas <nshrivas@codeaurora.org> | 2018-04-04 05:43:47 -0700 |
| commit | cfa4e0a33421a7b28a2e57ca1dedbbe50f45ebaf (patch) | |
| tree | eb733a05cc161db71ba58a19e6fe61cfb3ce946b | |
| parent | deedaf35e1a6f925e37b05ad64ee0acd482164cf (diff) | |
qcacld-3.0: Fix check condition after taking lock
In IPA clean interface function a NULL check was
added to prevent race condition between supplicant
and MC thread after acquiring lock, which
otherwise would have lead to a NULL access. The
check condition was performing check for valid
value instead of NULL.
To address the issue the check condition was
corrected to check for NULL.
Change-Id: I1c7205ac78f31a0d31a2d07eccbe11b64578ab76
CRS-Fixed: 2217450
| -rw-r--r-- | core/hdd/src/wlan_hdd_ipa.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/hdd/src/wlan_hdd_ipa.c b/core/hdd/src/wlan_hdd_ipa.c index 1938d8b27bee..00022a92d97c 100644 --- a/core/hdd/src/wlan_hdd_ipa.c +++ b/core/hdd/src/wlan_hdd_ipa.c @@ -6254,7 +6254,7 @@ static void hdd_ipa_cleanup_iface(struct hdd_ipa_iface_context *iface_context) * and check if the address has been already cleared by the * other thread */ - if (iface_context->adapter) { + if (!iface_context->adapter) { qdf_spin_unlock_bh(&iface_context->interface_lock); HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "Already cleared"); goto end; |