From nobody Mon Jun 22 14:27:25 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DA532C433FE for ; Tue, 22 Mar 2022 13:30:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234355AbiCVNbx (ORCPT ); Tue, 22 Mar 2022 09:31:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47038 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233715AbiCVNbs (ORCPT ); Tue, 22 Mar 2022 09:31:48 -0400 Received: from mail-lj1-x22e.google.com (mail-lj1-x22e.google.com [IPv6:2a00:1450:4864:20::22e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4FB142DD4F for ; Tue, 22 Mar 2022 06:30:20 -0700 (PDT) Received: by mail-lj1-x22e.google.com with SMTP id u3so24031423ljd.0 for ; Tue, 22 Mar 2022 06:30:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=waldekranz-com.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version:organization :content-transfer-encoding; bh=9HKooFSzF8EEsT2BEZgeunYRwvAjSyxG2IXK+iRrPEs=; b=29htvhMmpTh4PKGEPvfVlPAUZrleKez8uBO1KBF0exKnC1WyZp6jRTQwDHy/q9a/if R4OwsReo7NGIThGIV7Cil5RrsIpD6Gq6vC1sQxSVfEfj+mIzlK0nKfNpZ9QbNiwlgS8C IAU6K1e3RWdb5GSt7vgRoCkvtWl8/5rV2tcGNYmKwgvSq1+bnrrTogQhTcAQrJIYuQV4 c2i+5SZEbLgLKdrjjNhmiO9KDuvbzRn9mfT3oshKO442uXOdu1FxRcxyAka7Iz174Xak A40spqBYtTqd5MC1/e/YiixXhFlC5r4oiPSg7YS+KHOQSiu7YXIEt7T90404opxzbomi bUvw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :organization:content-transfer-encoding; bh=9HKooFSzF8EEsT2BEZgeunYRwvAjSyxG2IXK+iRrPEs=; b=ggUtFyTxW3JZj4RisKtGp4TyOEh+hp4f0f3NmI9+sbCBwa+QNBIqts1/SFCacLsLYI mpAQsY4YXcdYNT/ovotEERI/TtfLwkE3W5RTwawx04l3Tcl6JaV4GXAuazse7NQgzZ82 j7Bfh1/hVeu3S00fICOkfma3kOUPZx3Qxfr7k82kGQ8lQpjAq7T7OOwJssjXc0ex8lRW KnzNwrD8jY9mSVG8zCv+8t1mZ4HOy9xo24xIqQ13D/nm/7GVHc4yWB/IV5/sk4xL9Wgy olrw3jj7PYG/G9N2bciDIhKysnOpnVYmm+ZX6d/pTyGAxXyRPMVAd2J560qil+NbITgf FTHg== X-Gm-Message-State: AOAM531lTLYhOQ5qbZm5vpuWGXF8TAtrgl9tdiRURVtgfEv0bn0yUZ9E /cAT+MTd6U/AjDg+NPnJLOYhyg== X-Google-Smtp-Source: ABdhPJxkwrq7ZJvbNzKo/Kac7gQ286RAOZTZyDPHLItZIJZkOra75RUq9+xsw1H+JJrRwUndrhihDg== X-Received: by 2002:a2e:a4d1:0:b0:249:8c9c:5ae0 with SMTP id p17-20020a2ea4d1000000b002498c9c5ae0mr4441949ljm.283.1647955817198; Tue, 22 Mar 2022 06:30:17 -0700 (PDT) Received: from veiron.westermo.com (static-193-12-47-89.cust.tele2.se. [193.12.47.89]) by smtp.gmail.com with ESMTPSA id y26-20020a19915a000000b0044a0356abc4sm1779904lfj.220.2022.03.22.06.30.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Mar 2022 06:30:16 -0700 (PDT) From: Tobias Waldekranz To: davem@davemloft.net, kuba@kernel.org Cc: Roopa Prabhu , Nikolay Aleksandrov , Paolo Abeni , bridge@lists.linux-foundation.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next] net: bridge: mst: Restrict info size queries to bridge ports Date: Tue, 22 Mar 2022 14:30:01 +0100 Message-Id: <20220322133001.16181-1-tobias@waldekranz.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Organization: Westermo Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Ensure that no bridge masters are ever considered for MST info dumping. MST states are only supported on bridge ports, not bridge masters - which br_mst_info_size relies on. Fixes: 122c29486e1f ("net: bridge: mst: Support setting and reporting MST p= ort states") Signed-off-by: Tobias Waldekranz --- It turns out that even with Eric's fix, the guard was not restrictive enough. Sorry about all the noise around this. net/bridge/br_netlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c index 204472449ec9..200ad05b296f 100644 --- a/net/bridge/br_netlink.c +++ b/net/bridge/br_netlink.c @@ -119,7 +119,7 @@ static size_t br_get_link_af_size_filtered(const struct= net_device *dev, /* Each VLAN is returned in bridge_vlan_info along with flags */ vinfo_sz +=3D num_vlan_infos * nla_total_size(sizeof(struct bridge_vlan_i= nfo)); =20 - if (vg && (filter_mask & RTEXT_FILTER_MST)) + if (p && vg && (filter_mask & RTEXT_FILTER_MST)) vinfo_sz +=3D br_mst_info_size(vg); =20 if (!(filter_mask & RTEXT_FILTER_CFM_STATUS)) --=20 2.25.1