linux-kernel - Re: [PATCH v5] wifi: mt76: Fix NULL Dereference caused by mt76_connac_get_he_phy

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <48c3da06-fe59-4902-a74f-a4f3e4488160@web.de>
Date: Sat, 5 Oct 2024 19:28:47 +0200
From: Markus Elfring <Markus.Elfring@....de>
To: Zichen Xie <zichenxie0106@...il.com>, linux-mediatek@...ts.infradead.org,
 linux-arm-kernel@...ts.infradead.org, linux-wireless@...r.kernel.org,
 Aditya Kumar Singh <quic_adisi@...cinc.com>,
 Allen Ye (葉芷勳) <Allen.Ye@...iatek.com>,
 Angelo Gioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
 Benjamin Lin <benjamin-jw.lin@...iatek.com>, Bo Jiao <bo.jiao@...iatek.com>,
 Deren Wu <deren.wu@...iatek.com>,
 Evelyn Tsai (蔡珊鈺) <Evelyn.Tsai@...iatek.com>,
 Felix Fietkau <nbd@....name>, Howard Hsu <howard-yh.hsu@...iatek.com>,
 Johannes Berg <johannes.berg@...el.com>, Kalle Valo <kvalo@...nel.org>,
 Krzysztof Kozlowski <krzk@...nel.org>, Lorenzo Bianconi
 <lorenzo@...nel.org>, Matthias Brugger <matthias.bgg@...il.com>,
 MeiChia Chiu <MeiChia.Chiu@...iatek.com>,
 Ming Yen Hsieh <mingyen.hsieh@...iatek.com>,
 Peter Chiu <chui-hao.chiu@...iatek.com>, Ryder Lee <ryder.lee@...iatek.com>,
 Sean Wang <sean.wang@...iatek.com>, Shayne Chen <shayne.chen@...iatek.com>,
 StanleyYP Wang <StanleyYP.Wang@...iatek.com>
Cc: LKML <linux-kernel@...r.kernel.org>, kernel-janitors@...r.kernel.org,
 Chenyuan Yang <chenyuan0y@...il.com>, Zijie Zhao <zzjas98@...il.com>
Subject: Re: [PATCH v5] wifi: mt76: Fix NULL Dereference caused by
 mt76_connac_get_he_phy_cap()

> mt76_connac_get_he_phy_cap() may return a NULL pointer,
> leading to NULL Pointer Dereference.
…

             multiple questionable accesses?

How do you think about to use the term “null pointer dereference”
for the final commit message (including the summary phrase)?


Were any static source code analysis tools involved in the detection
of presented change possibilities?


…
> ---
>  drivers/net/wireless/mediatek/mt76/mt7915/mcu.c | 5 +++++
…

Did you overlook to add patch version descriptions behind the marker line?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.12-rc1#n723

Regards,
Markus