On Thu, Jan 8, 2009 at 10:05 PM, Niels de Vos
<[email protected]> 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 <[email protected]>
---
drivers/infiniband/hw/amso1100/c2_provider.c | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)
Eugene Teo wrote:
> On Thu, Jan 8, 2009 at 10:05 PM, Niels de Vos
> <[email protected]> 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 <[email protected]>
(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 <[email protected]>
> ---
> drivers/infiniband/hw/amso1100/c2_provider.c | 17 ++++++++++++-----
> 1 file changed, 12 insertions(+), 5 deletions(-)