From e5b563685603e6bb80fd4c665ff14bf480e0cba3 Mon Sep 17 00:00:00 2001 From: Wu Gao Date: Tue, 14 Aug 2018 16:04:16 +0800 Subject: [PATCH] qcacmn: Report err status if idr_find return NULL It won't report err status since if idr_find return NULL, which cause unnecessary process with NULL pointer. Change-Id: I5813ddca11172f75fb73c9530e003c50e55beba2 CRs-Fixed: 2275848 --- qdf/linux/src/qdf_idr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qdf/linux/src/qdf_idr.c b/qdf/linux/src/qdf_idr.c index 9749083746b8..b6341e2cb72f 100644 --- a/qdf/linux/src/qdf_idr.c +++ b/qdf/linux/src/qdf_idr.c @@ -133,7 +133,7 @@ QDF_STATUS qdf_idr_find(qdf_idr *idp, int32_t id, void **ptr) qdf_spinlock_acquire(&idp->lock); *ptr = idr_find(&idp->idr, id); qdf_spinlock_release(&idp->lock); - if (!ptr) + if (!(*ptr)) return QDF_STATUS_E_INVAL; else return QDF_STATUS_SUCCESS;