lkml.org 
[lkml]   [2009]   [Jan]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] IB: check for memory allocation failure [v2]
Eugene Teo wrote:
> On Thu, Jan 8, 2009 at 10:05 PM, Niels de Vos
> <niels.devos@wincor-nixdorf.com> wrote:
>> Hi Eugene,
>>
>> please see below.
>
> Thanks for reviewing. Here's an updated patch:
>
> Fix error-path NULL deref in c2_register_device().
>
> Signed-off-by: Eugene Teo <eugeneteo@kernel.org>

(note: your patch was attached, not inlined...)

There is also an additional kfree() at the end of the patch. It wasn't
included in the first version you sent. Looks correct now imho.

Reviewed-by: Niels de Vos <niels.devos@wincor-nixdorf.com>


> ---
> drivers/infiniband/hw/amso1100/c2_provider.c | 17 ++++++++++++-----
> 1 file changed, 12 insertions(+), 5 deletions(-)


[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2009-01-08 16:39    [W:0.039 / U:0.496 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site