Re: p = kmalloc(sizeof(*p), )

On Sun, Sep 18, 2005 at 06:30:58PM +0100, Al Viro wrote:
> On Sun, Sep 18, 2005 at 07:25:24PM +0300, Denis Vlasenko wrote:
> > Do these qualify?
> > 
> > http://www.uwsg.iu.edu/hypermail/linux/kernel/0105.1/0579.html
> > o Fix wrong kmalloc sizes in ixj/emu10k1 (David Chan) 
> 
> ixj does, emu10k does not (sizeof(p) instead of sizeof(*p) would be
> exact same bug).

Funny, a few days ago, I found such a bug in a coworker's code. I agree
that finding this at 3am would be close to impossible. OK, you converted
me :-)

Regards,
Willy

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

References:
- p = kmalloc(sizeof(*p), )
  - From: Russell King <[email protected]>
- Re: p = kmalloc(sizeof(*p), )
  - From: Alan Cox <[email protected]>
- Re: p = kmalloc(sizeof(*p), )
  - From: Al Viro <[email protected]>
- Re: p = kmalloc(sizeof(*p), )
  - From: Denis Vlasenko <[email protected]>
- Re: p = kmalloc(sizeof(*p), )
  - From: Al Viro <[email protected]>

Prev by Date: Re: [KORG] REMINDER: master.kernel.org extended downtime
Next by Date: Re: I request inclusion of reiser4 in the mainline kernel
Previous by thread: Re: p = kmalloc(sizeof(*p), )
Next by thread: Re: p = kmalloc(sizeof(*p), )
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Netfilter] [Bugtraq] [Photo] [Gimp] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Video 4 Linux] [Linux for the blind]