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: Fri, 15 Apr 2022 08:31:16 +0300 From: Dan Carpenter <dan.carpenter@...cle.com> To: "Fabio M. De Francesco" <fmdefrancesco@...il.com> Cc: gregkh@...uxfoundation.org, davem@...emloft.net, len.baker@....com, dave@...olabs.net, edumazet@...gle.com, Haowen Bai <baihaowen@...zu.com>, linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org Subject: Re: [PATCH V2] staging: rtl8192e: Fix signedness bug in rtllib_rx_assoc_resp() On Fri, Apr 15, 2022 at 07:22:36AM +0200, Fabio M. De Francesco wrote: > On venerdì 15 aprile 2022 04:09:31 CEST Haowen Bai wrote: > > function rtllib_rx_assoc_resp () unsigned errcode receive auth_parse()'s > > errcode -ENOMEM. > > > > Signed-off-by: Haowen Bai <baihaowen@...zu.com> > > --- > > V1->V2: reduce return random value; print its own error message. > > > > drivers/staging/rtl8192e/rtllib_softmac.c | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > > [snip] > > It looks like you are doing too many things and that those aren't even > discussed in your commit message. > Nope. The patch is one thing, but maybe it could be described better. Here is my proposed commit message: "The rtllib_rx_assoc_resp() function has a signedness bug because it's a declared as a u16 but it return -ENOMEM. When you look at it more closely it returns a mix of error codes including 0xcafe, -ENOMEM, and a->status which is WLAN_STATUS_NOT_SUPPORTED_AUTH_ALG. This is a mess. Clean it up to just return standard kernel error codes. We can print out the a->status before returning a regular error code. The printks in the caller need to be adjusted as well." regards, dan carpenter
Powered by blists - more mailing lists