| 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
| ||
|
Message-ID: <20080229144437.GD28849@shadowen.org> Date: Fri, 29 Feb 2008 14:44:37 +0000 From: Andy Whitcroft <apw@...dowen.org> To: Andrew Morton <akpm@...ux-foundation.org> Cc: Dirk Eibach <eibach@...ys.de>, linux-kernel@...r.kernel.org, greg@...ah.com Subject: Re: [PATCH] usb: add sysfs configuration interface for CP2101 On Fri, Feb 29, 2008 at 02:02:45AM -0800, Andrew Morton wrote: > On Fri, 29 Feb 2008 10:02:18 +0100 Dirk Eibach <eibach@...ys.de> wrote: > We have a new strict_strtoul() (and related functions) which will perform > proper checking for a valid number. Please use that interface. > > Andy, this is going to happen so much that a "should you have used > strict_strtoul?" warning in checkpatch would reduce my email output. Sure, will add something. I wonder if this new interface is documented. I note that this interface is new in -mm at this time. So that brings up an interesting question as to how one would integrate this check with checkpatch. As checking patches for mainline, this would be an incorrect check until that patch merges. I guess the right thing to do is provide a separate patch for checkpatch which adds this check which should sit with the patch in your tree which adds the functionality. Make sense? -apw -- 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