From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 00087C2BBC7 for ; Sat, 11 Apr 2020 23:44:12 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id CED75206A1 for ; Sat, 11 Apr 2020 23:44:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586648652; bh=um6CFsKJ8fcYt2IepK3FugjmiB8jjVP+9kV045Z58m0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:List-ID:From; b=2bEtRYZLHwSUnSGP6/Temv3awz2ldYl9Z965GYMxe+f6uf4Wa+Ep7PjJVCm3qOQ05 f7814j8cD6NR1wchJoj0YdhJ9A7fBTd2VBaa7ovDuwP1lDNQb0Z+Ef4GVueEe1xebl C42U/oNJvEoT16JG8sfG9Vcx/Z5DW92PA82SYENY= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731055AbgDKXoJ (ORCPT ); Sat, 11 Apr 2020 19:44:09 -0400 Received: from mail.kernel.org ([198.145.29.99]:41274 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728279AbgDKXGK (ORCPT ); Sat, 11 Apr 2020 19:06:10 -0400 Received: from sasha-vm.mshome.net (c-73-47-72-35.hsd1.nh.comcast.net [73.47.72.35]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id B235B217D8; Sat, 11 Apr 2020 23:06:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586646370; bh=um6CFsKJ8fcYt2IepK3FugjmiB8jjVP+9kV045Z58m0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=qqW4TUcEZVro6+TIVi5Zbv9VPwbhyngmQxWmBILZWHDX8z7HPnBsTxQQGTE8+/Yn6 nXLZKxZCqb3/uqzSgrYPWUaQ86NoopKSrBEHenE7We1fhzqqmhbbC2VK09n07CheNB /Wt5eOL1HV66Ek7s4VutPOBwykpmNwjQmkj4lU5g= From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Raveendran Somu , Arend van Spriel , Chi-hsien Lin , Kalle Valo , Sasha Levin , linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com, brcm80211-dev-list@cypress.com, netdev@vger.kernel.org Subject: [PATCH AUTOSEL 5.6 113/149] brcmfmac: fix the incorrect return value in brcmf_inform_single_bss(). Date: Sat, 11 Apr 2020 19:03:10 -0400 Message-Id: <20200411230347.22371-113-sashal@kernel.org> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20200411230347.22371-1-sashal@kernel.org> References: <20200411230347.22371-1-sashal@kernel.org> MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: 8bit Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Raveendran Somu [ Upstream commit bd9944918ceb28ede97f715d209e220db5e92c09 ] The function brcmf_inform_single_bss returns the value as success, even when the length exceeds the maximum value. The fix is to send appropriate code on this error. This issue is observed when Cypress test group reported random fmac crashes when running their tests and the path was identified from the crash logs. With this fix the random failure issue in Cypress test group was resolved. Reviewed-by: Arend van Spriel Signed-off-by: Raveendran Somu Signed-off-by: Chi-hsien Lin Signed-off-by: Kalle Valo Link: https://lore.kernel.org/r/1585124429-97371-4-git-send-email-chi-hsien.lin@cypress.com Signed-off-by: Sasha Levin --- drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c index a2328d3eee036..2ba165330038e 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c @@ -2953,7 +2953,7 @@ static s32 brcmf_inform_single_bss(struct brcmf_cfg80211_info *cfg, if (le32_to_cpu(bi->length) > WL_BSS_INFO_MAX) { bphy_err(drvr, "Bss info is larger than buffer. Discarding\n"); - return 0; + return -EINVAL; } if (!bi->ctl_ch) { -- 2.20.1