Re: [patch 2/3] ib_srpt: checking NULL instead of ERR_PTR

Dan Carpenter
2012-02-03T13:01:49Z

On 11/5/11, Nicholas A. Bellinger <nab@risingtidesystems.com> wrote:
> On Fri, 2011-11-04 at 21:27 +0300, Dan Carpenter wrote:
>> transport_init_session() and target_fabric_configfs_init() don't
>> return NULL pointers, they only return ERR_PTRs or valid pointers.
>>
>> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
>
> Thanks for catching this breakage. It appears that are a few other
> locations outside of ib_srpt where return checking for these two
> functions needs to be fixed as well. I'll fix this up shortly.
>

All the others seem to fixed up. It's just these two place which need
to be fixed still. My patch still applies fine with a little fuzz. ;)

regards,
dan carpenter