| 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, 11 Aug 2015 20:37:59 +0530 From: Chandra Gorentla <csgorentla@...il.com> To: Dan Carpenter <dan.carpenter@...cle.com> Cc: rachel.kim@...el.com, dean.lee@...el.com, chris.park@...el.com, gregkh@...uxfoundation.org, devel@...verdev.osuosl.org, linux-wireless@...r.kernel.org, johnny.kim@...el.com, linux-kernel@...r.kernel.org, sudipm.mukherjee@...il.com, csgorentla@...il.com Subject: Re: [PATCH] staging: wilc100: Remove pointer and integer comparision On Mon, Aug 10, 2015 at 09:38:56PM +0300, Dan Carpenter wrote: > Warnings are not a bad thing they are a valuable marker to let us know > which code is broken. If we just silence the warning in the laziest > possible way then we are throwing away valuable information. It's > better to leave the warning there so that the next person can fix it > properly. > > If you want to leave the code as-is that is absolutely fine with me, but > don't remove the warning until someone can look at this a bit carefully. > > regards, > dan carpenter > Thank you. It looks like it needs more knowledge of wireless networking to conclude if there was any problem and to fix if any problem exists. I will come back to this problem in near future, if the error was still observed then. chandra -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists