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
| ||
|
Date: Tue, 22 Nov 2022 19:09:11 +0900 From: Max Staudt <max@...as.org> To: <wg@...ndegger.com>, <mkl@...gutronix.de> Cc: "Ziyang Xuan (William)" <william.xuanziyang@...wei.com>, <davem@...emloft.net>, <edumazet@...gle.com>, <kuba@...nel.org>, <pabeni@...hat.com>, <linux-can@...r.kernel.org>, <netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>, Vincent Mailhol <mailhol.vincent@...adoo.fr> Subject: Re: [PATCH] can: can327: fix potential skb leak when netdev is down Marc, Wolfgang, Could you please include William's patch to can327, provided that you see no issue with it? Thanks :) Max On Tue, 22 Nov 2022 10:10:50 +0800 "Ziyang Xuan (William)" <william.xuanziyang@...wei.com> wrote: > Hello, > > Gently ask. > > Is there any other problem? And can it be applied? > > Thanks. > > > (CC Vincent, he may be interested) > > > > > > On Thu, 10 Nov 2022 14:14:37 +0800 > > Ziyang Xuan <william.xuanziyang@...wei.com> wrote: > > > >> Fix it by adding kfree_skb() in can327_feed_frame_to_netdev() when netdev > >> is down. Not tested, just compiled. > > > > Looks correct to me, so: > > > > Reviewed-by: Max Staudt <max@...as.org> > > > > > > Thank you very much for finding and fixing this! > > > > Max > > > > . > >
Powered by blists - more mailing lists