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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 20 Mar 2022 09:53:02 +0800 From: Xiaomeng Tong <xiam0nd.tong@...il.com> To: jakobkoschel@...il.com Cc: davem@...emloft.net, kuba@...nel.org, kvalo@...nel.org, linux-kernel@...r.kernel.org, linux-wireless@...r.kernel.org, netdev@...r.kernel.org, pizza@...ftnet.org, xiam0nd.tong@...il.com Subject: Re: [PATCH] cw1200: remove an unneeded NULL check on list iterator On Sun, 20 Mar 2022 01:47:26 +0100, Jakob Koschel <jakobkoschel@...il.com> wrote: > I don't think this is fixing anything here. You are basically just removing > a check that was always true. Yes. > > I'm pretty sure that this check is here to check if either the list is empty or no > element was found. If I'm not wrong, some time ago, lists where not circular but > actually pointed to NULL (or the head was NULL) so this check made sense but doesn't > anymore. > > The appropriate fix would be only setting 'item' when a break is hit and keep > the original check. You are right if that is the author's original intention. I will fix it in PATCH v2. > > > unsigned long tmo = item->queue_timestamp + queue->ttl; > > mod_timer(&queue->gc, tmo); > > cw1200_pm_stay_awake(&stats->priv->pm_state, > > -- > > 2.17.1 > > > > > > I've made those changes already and I'm in the process of upstreaming them in an organized > way, so maybe it would make sense to synchronize, so we don't post duplicate patches. Ok, I will cc you when sending related patches to avoid duplication. I hope you can do the same, thank you. Here are the 9 patches I have sent, so you don't have to reinvent th wheel: https://lore.kernel.org/all/20220319102222.3079-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220319073143.30184-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220319063800.28791-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220319053742.27443-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220319052350.26535-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220319044416.24242-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220319043606.23292-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220319042657.21835-1-xiam0nd.tong@gmail.com/ https://lore.kernel.org/all/20220316075153.3708-1-xiam0nd.tong@gmail.com/ -- Xiaomeng Tong
Powered by blists - more mailing lists