Return-path: Received: from wolverine01.qualcomm.com ([199.106.114.254]:54456 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750928Ab2ITGVd (ORCPT ); Thu, 20 Sep 2012 02:21:33 -0400 From: To: CC: , Raja Mani Subject: [PATCH 1/3] ath6kl: Avoid null ptr dereference while printing reg domain pair Date: Thu, 20 Sep 2012 11:51:06 +0530 Message-ID: <1348122068-24648-1-git-send-email-rmani@qca.qualcomm.com> (sfid-20120920_082151_423049_DB649138) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-wireless-owner@vger.kernel.org List-ID: From: Raja Mani Return value of ath6kl_get_regpair() is stored in 'regpair' in ath6kl_wmi_regdomain_event() func and it's directly accessed in the debug prints without checking for NULL value. There are situation to get NULL pointer as a return value from ath6kl_get_regpair() func. Fix this. Found this on code review. Signed-off-by: Raja Mani --- drivers/net/wireless/ath/ath6kl/wmi.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath6kl/wmi.c b/drivers/net/wireless/ath/ath6kl/wmi.c index 68b46bd..d5263ff 100644 --- a/drivers/net/wireless/ath/ath6kl/wmi.c +++ b/drivers/net/wireless/ath/ath6kl/wmi.c @@ -936,8 +936,9 @@ static void ath6kl_wmi_regdomain_event(struct wmi *wmi, u8 *datap, int len) regpair = ath6kl_get_regpair((u16) reg_code); country = ath6kl_regd_find_country_by_rd((u16) reg_code); - ath6kl_dbg(ATH6KL_DBG_WMI, "Regpair used: 0x%0x\n", - regpair->regDmnEnum); + if (regpair) + ath6kl_dbg(ATH6KL_DBG_WMI, "Regpair used: 0x%0x\n", + regpair->regDmnEnum); } if (country && wmi->parent_dev->wiphy_registered) { -- 1.7.1