| 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: Wed, 27 Apr 2011 10:29:46 +0200
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Steven Rostedt <rostedt@...dmis.org>
Cc: Thiago Farina <tfransosi@...il.com>, Joe Perches <joe@...ches.com>,
Alexey Dobriyan <adobriyan@...il.com>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] linux/string.h: Introduce streq macro.
On Tue, Apr 26, 2011 at 9:58 PM, Steven Rostedt <rostedt@...dmis.org> wrote:
> On Tue, 2011-04-26 at 16:47 -0300, Thiago Farina wrote:
>
>> > $ grep -rP --include=*.[ch] "\bstrcmp.*==\s*0\b" * | wc -l
>> > 1143
>> > $ grep -rP --include=*.[ch] "\!\s*strcmp\s*\(" * | wc -l
>> > 1663
>> >
>> > Can you count how many misuses of strcmp have been
>> > corrected?
>>
>> > Do you plan to convert the existing 2800?
>>
>> I'd work on this without any problem.
>
> Nothing a perl script can't do either.
If you are really going to do that, please use a coccinelle's semantic
patch (which is designed precisely for that purpose) and document it
so that it can be included in the standard set of semantic patches
applied to the kernel regularly, i.e. something like:
@@ expression E1, E2; @@
- strcmp(E1,E2) == 0
+ streq(E1,E2)
@@ expression E1, E2; @@
- strcmp(E1,E2) != 0
+ !streq(E1,E2)
@@ expression E1, E2; @@
- !strcmp(E1,E2)
+ streq(E1,E2)
@@ expression E1, E2; @@
- strcmp(E1,E2)
+ !streq(E1,E2)
This produces a patch ~1.1 MB in size, which is probably going to
produce some headaches when merging other patches...
$ diffstat streq.patch | tail -1
1073 files changed, 3429 insertions(+), 3419 deletions(-)
$ diffstat streq.patch | sort -nk 3 | tail -20
drivers/sbus/char/envctrl.c | 30 -
fs/reiserfs/bitmap.c | 30 -
sound/pci/hda/hda_eld.c | 30 -
drivers/video/matrox/matroxfb_base.c | 32 -
arch/blackfin/kernel/module.c | 34 +-
tools/perf/util/symbol.c | 35 +-
scripts/mod/modpost.c | 38 +-
net/irda/irlan/irlan_client.c | 40 +-
drivers/staging/hv/vmbus_drv.c | 42 +-
drivers/usb/gadget/gadget_chips.h | 42 +-
drivers/s390/net/qeth_l3_sys.c | 44 +-
drivers/macintosh/windfarm_smu_controls.c | 48 +-
security/selinux/hooks.c | 48 +-
security/smack/smack_lsm.c | 48 +-
arch/x86/pci/common.c | 50 +--
drivers/net/niu.c | 54 +--
fs/xfs/linux-2.6/xfs_super.c | 92 ++---
tools/perf/util/trace-event-parse.c | 108 +++---
drivers/staging/wlags49_h2/wl_profile.c | 121 +++----
net/core/pktgen.c |
164 +++++-----
Miguel Ojeda
>
> -- Steve
>
>
> --
> 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/
>
--
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