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 8B89FC2BA19 for ; Sat, 11 Apr 2020 23:35:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5CDF2206E9 for ; Sat, 11 Apr 2020 23:35:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586648139; bh=GnH4lKvQ8fu8Ol/BddxK6xESVKV6pEYeeVYiwJHu0MQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:List-ID:From; b=ZJ5qPNB8LZ1N3BBVIA/PLjiAzcRdxSSvFk+VIHywP/AF2eLZ1/SZA/t6nqqvsFn48 2bN8Kv0HMn90MS5iVhU0UPKvf047AGDko4Jm2s/5K4yh8Mr2eb6o+eDg7/Rzyuu16D An7663v421gysbaN8QPWGxYemUQpLkH5M5jH9qIE= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729060AbgDKXI7 (ORCPT ); Sat, 11 Apr 2020 19:08:59 -0400 Received: from mail.kernel.org ([198.145.29.99]:46434 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728000AbgDKXI7 (ORCPT ); Sat, 11 Apr 2020 19:08:59 -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 1C9FE20708; Sat, 11 Apr 2020 23:08:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586646539; bh=GnH4lKvQ8fu8Ol/BddxK6xESVKV6pEYeeVYiwJHu0MQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=pw5IIWI9BlbJzl18NOd5Zvlh2dmaWmLBuDC46Lv2O8VPKQ88ydxcKMV/W/0cWbigr jO0RBYC09yExEcojzC0UjQ0qww1VTCqqQcJXcZn//9QOzqeaXXS285Ghc0EcoKA6lC FRLFNy9OxhAkbhd0som8SDsFxrkAAiSeVEgYj76s= 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.5 091/121] brcmfmac: fix the incorrect return value in brcmf_inform_single_bss(). Date: Sat, 11 Apr 2020 19:06:36 -0400 Message-Id: <20200411230706.23855-91-sashal@kernel.org> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20200411230706.23855-1-sashal@kernel.org> References: <20200411230706.23855-1-sashal@kernel.org> MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@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 5598bbd09b627..999c36cf52a6c 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c @@ -2874,7 +2874,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