2 Nov
2010
2 Nov
'10
7:39 a.m.
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@audioscience.com
OK, applied now. Thanks!
Takashi