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: Sun, 22 Jun 2008 09:26:55 -0700 From: Randy Dunlap <randy.dunlap@...cle.com> To: JAILLET <jaillet.christophe@...adoo.fr> Cc: petero2@...ia.com, linux <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 1/1] drivers/block/pktcdvd.c : avoid useless memset On Sun, 22 Jun 2008 15:34:19 +0200 JAILLET wrote: > From: Christophe Jaillet <jaillet.christophe@...adoo.fr> > > Hi, here is a patch against linux/drivers/block/pktcdvd.c which : > > 1) Avoid to 'memset(...,0, ...)' before calling 'init_cdrom_command' > because this function already does it. > > > Signed-off-by: Christophe Jaillet <jaillet.christophe@...adoo.fr> > > --- > diff --git a/drivers/block/pktcdvd.c b/drivers/block/pktcdvd.c > index 3ba1df9..8369ebc 100644 > --- a/drivers/block/pktcdvd.c > +++ b/drivers/block/pktcdvd.c > @@ -2079,7 +2079,6 @@ static noinline_for_stack int > pkt_write_caching(struct pktcdvd_device *pd, > unsigned char buf[64]; > int ret; > > - memset(buf, 0, sizeof(buf)); > init_cdrom_command(&cgc, buf, sizeof(buf), CGC_DATA_READ); > cgc.sense = &sense; > cgc.buflen = pd->mode_offset + 12; > @@ -2126,7 +2125,6 @@ static noinline_for_stack int > pkt_get_max_speed(struct pktcdvd_device *pd, > unsigned char *cap_buf; > int ret, offset; > > - memset(buf, 0, sizeof(buf)); > cap_buf = &buf[sizeof(struct mode_page_header) + pd->mode_offset]; > init_cdrom_command(&cgc, buf, sizeof(buf), CGC_DATA_UNKNOWN); > cgc.sense = &sense; Looks like tbird ate all of the whitespace. You can see if Documentation/email-clients.txt has info in it that would help you, then resend the patch. --- ~Randy Linux Plumbers Conference, 17-19 September 2008, Portland, Oregon USA http://linuxplumbersconf.org/ -- 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