[alsa-devel] [PATCH] AudioScience HPI driver: Unsafe memory management when allocating control cache

Takashi Iwai tiwai at suse.de
Tue Nov 2 07:39:04 CET 2010


At Tue, 02 Nov 2010 11:19:36 +1300,
Eliot Blennerhassett wrote:
> 
> On 30/10/10 08:35, Jesper Juhl wrote:
> > I noticed that sound/pci/asihpi/hpicmn.c::hpi_alloc_control_cache() does 
> > not check the return value from kmalloc(), which may fail.
> > If kmalloc() fails we'll dereference a null pointer and things will go bad 
> > fast.
> > There are two memory allocations in that function and there's also the 
> > problem that the first may succeed and the second may fail and nothing is 
> > done about that either which will also go wrong down the line.
> 
> Thanks Jesper,
> 
> your changes look fine.
> 
> Acked-by: Eliot Blennerhassett <linux at audioscience.com>

OK, applied now.  Thanks!


Takashi


More information about the Alsa-devel mailing list