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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7F6B7C35FFC for ; Sat, 22 Mar 2025 10:03:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:Subject:References:Cc:To:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=iAEpjrSv507710jqRh6zdUdBI0yMAqXScOhii8Wavfo=; b=gr/cX+IX+bDsIXkwlvvMIpNyDj 4GK+/NIAcCLxKldLKDtyGhyGm8hgAyluYpx/VezCtNSjUS4KIJjQc+66MB4ma3JciOLgStQotOu12 Qr1MMDiAX8ETqwY55O3wpXQEJeOrwyVFhDt0oHhNKOtpTfJeP6cLw8eB6MRb0OX0JSn4tZzyIJz3F Y0zM22pmI+u59f5emhT63EFYXUwJw+Br3BgwWduzn4c1ziw4JjD4YEPY0djMMfp/ZOz1X1+4v7VOR yfZDduVaxcpBLftu0TF0ITQkOSs+OYKLff2NoIciV0V+06c0W9UHZZtWF8PozhVQeoTdt2jGaLjbY +R7j5dog==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tvvhX-0000000HCyP-067i; Sat, 22 Mar 2025 10:03:35 +0000 Received: from mout.web.de ([217.72.192.78]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tvvfp-0000000HCq0-1oSU; Sat, 22 Mar 2025 10:01:50 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=web.de; s=s29768273; t=1742637688; x=1743242488; i=markus.elfring@web.de; bh=iAEpjrSv507710jqRh6zdUdBI0yMAqXScOhii8Wavfo=; h=X-UI-Sender-Class:Message-ID:Date:MIME-Version:To:Cc:References: Subject:From:In-Reply-To:Content-Type:Content-Transfer-Encoding: cc:content-transfer-encoding:content-type:date:from:message-id: mime-version:reply-to:subject:to; b=uETpm1M9BwO3VTF7XVDXSoT97UulAaZRSAUWy0X0UNVwA6AQLxBxrsH8KBZjRXN7 r5q5aYxbryyuGI3PuMLc6M686mh32BmfGjuK78FZuryjuzYVEV/56Zt96+wG/nx8B Hv2cv5X+K0crz0ucVDOzWncRsYRT0BaTSkS0iBuo5ugKx4UYB1w44LZScCVRL4VYe 3UU6Z/5BDIwneUrw7oFy0uoLoMed5G9lILnLpdOVCRwyGckREmOmKdmkm0KYeA9SF qtGNoJwL/NmIPf0agCdgw1EkbrvQPIMtLV+SMg09vb2Ueh1ogzBaTpoDNGe1zfV8A 1FKMltss2bKOxAWwLA== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.29] ([94.31.70.73]) by smtp.web.de (mrweb105 [213.165.67.124]) with ESMTPSA (Nemesis) id 1MFJnX-1tyRpB1EOv-001Fe1; Sat, 22 Mar 2025 11:01:28 +0100 Message-ID: <061ebbe3-557b-46d7-acb8-308ae87105dd@web.de> Date: Sat, 22 Mar 2025 11:01:18 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: Qasim Ijaz , linux-wireless@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org Cc: LKML , Angelo Gioacchino Del Regno , Bo Jiao , Dan Carpenter , Felix Fietkau , Lorenzo Bianconi , Matthias Brugger , Peter Chiu , Ryder Lee , Sean Wang , Shayne Chen References: <20250320201914.48159-1-qasdev00@gmail.com> Subject: Re: [PATCH] wifi: mt76: mt7996: prevent uninit return in mt7996_mac_sta_add_links Content-Language: en-GB From: Markus Elfring In-Reply-To: <20250320201914.48159-1-qasdev00@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:2McX4CekldO+dCShfVp3nTTN6ebmKzksWZyPGWUnWPaMGAAp0mb VYfQxcDdow9Nxrcn84qoj1thGy581LDB1hSfhFBfIC6mnjGy9/KPgMp0zmaqR6xxkb84TSO Xj41wDMvD+Sn3RoJLTrMBiy/03IFIMX5XORul19XLAW/RWdmf6M2Zcpt9diAo8M/+o+o2Vw TB4yqtQemP28jUy1MHX8w== UI-OutboundReport: notjunk:1;M01:P0:ZLMBFJ+Ezhg=;XdhfVc17+G6mZHDb0rrzWwQSBPY JyGwU2AIeGvJHa5/dvsgMUoXpLSoLtiCWRM4en4UXlVptT75+W1Ui2QqZwyFso53cMT7rD90T oJuRdQsIdBFcC7xyf/ohVt+IhTG00r/IcXTkdFHJlpkgVeAeqlIk8SZUr9C8K8R1VUCZv1Vmw 8Vmneq1nS/ORSlKGADbeIPuR5n+3cXqlCTgiOYrY27vYYshGG3osluR21qktOFVd8uBtsCWYZ Ylw6F3Z1onm5syJIMoCkQpGer4+e0uSQT633n3vjf08UnGQkIV88Tm5h/hw0GgQZw0/56uDUI ioeokjERcO3kUQuGCzy7i5Mv2PkhZg4I++luxREzEaA2whvAv/o3sh7yRHZmoSZBU9+DzHPsa 1jV3invEdwYD9eVV6zl9RgZoubyxzUHEJniO2N8mlIrx2GOTdVvBbiJ/mHwb0YUTY6x1eVSBQ KH3prczrDY0KGAb13xdEPQ2x0mN6DXuYLdrD/ZQlB0Re8VouvMH7tDyABDWQwq4beOJ1qcFVn 35zHeHCQM46TFeP4RYqvnFB4lsRm+P/30G9Z6zsnysQVYqa/4Yj0T8tJIea9xYNhCnEcZLsMp qanxmGs7aViFmyfPS6t/fkm+DnLCnr543u0V3Iq3jc2TguXFRpVY0W6RwjhWpK19G5e0qH0QY Yy+3ZhUPd2KK6jv6JgTHlceXNEfk9d8b9wWTae9aSfoAetAxuDIFbPcJGjx2wdoi8eiulmk6O xDEqlWoNfIdySi2MMNKIP/pfR8+kP1l1O+N7C9y3WKa9xt+m3CE5/6aT+fPsl7ekOTMb66+0R iK5XdX2qOqe/YvNWTmgX0E5fiuVpORMAU691Hs1soguFgcMMSrybRuWiuaSP9rPXAi2AbKazl wBKfo8YM5ggWisD2to4YsnXOGIGS2ZsZ3+I8rIFSBqRKj9ifQW2bj7DLIEz+4tGZv3EpITQGY J528ccLeE/XOvAmT8BgpdwBU7+5dd+1SwbYa7VFaxjOfELQESC9qfQCNGAhQXi7IRyggKYD7/ c4nAW1KSktL66W1vZbRf87BzhhYl9lDBLKsdHvSFXkYRrxs7zXgtdfPOGesak4+HiYjUrMibJ ii9N7VCM2EX4YaOCmHTX6g5E7Bvt3nRW8No7bJo73ZBk0wtVp/jL9kk75Luw6lNrSFM6Yjq+g YmebnzZmiJkNCIlDQwf83GkFaIZthU0JRfTu3DeQWd0004V1kInrWUz+rFlaaL0MOWfNqfdRu v6gPULtj2VcxcEQ4B/RuERfZMteYbPZsYG/IYy57Z5GIQHPsGTtNVsZ+/oJLnqGr4Hi3Qx4Ex PJrmiVJyE9fymePVetNSQupM4Tc5a6iq0H3Uwto0c+NZ2PS+Sf2ysR5EZ+fALpvqKAobQHQsd vMp94R9NWq3CfvulUpn+DsUOF3cgxiMLtlfpWQtHlAIQfGArlwoE6TxX7M9jHDPWyCHODXBFf sPRD9WCAbgCiYQe92s5M7ip/AjevWgwCoJufjCn/GoW3ky57i1we1wE5GMH+VQXFFhJRevQ== X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250322_030149_854443_96BFB5B4 X-CRM114-Status: UNSURE ( 5.99 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org =E2=80=A6 > +++ b/drivers/net/wireless/mediatek/mt76/mt7996/main.c > @@ -998,16 +998,22 @@ mt7996_mac_sta_add_links(struct mt7996_dev *dev, s= truct ieee80211_vif *vif, > continue; > > link_conf =3D link_conf_dereference_protected(vif, link_id); > - if (!link_conf) > + if (!link_conf) { > + err =3D -EINVAL; > goto error_unlink; > + } > > link =3D mt7996_vif_link(dev, vif, link_id); > - if (!link) > + if (!link) { > + err =3D -EINVAL; > goto error_unlink; > + } =E2=80=A6 I suggest to avoid such repeated error code assignments. Can an additional label be applied instead for this purpose? Regards, Markus