es1;4205;0cOn Wed, Nov 30, 2016 at 01:13:57PM -0500, Keith Busch wrote:
> On Tue, Nov 29, 2016 at 02:52:00PM -0700, Scott Bauer wrote:
> > +   dev = get_or_create_opal_dev(bdev, key->opal_act.key.lr, true);
> > +   if (!dev)
> > +           return -ENOMEM;
> 
> The alloc_opal_dev from this call returns ERR_PTR values on error, so
> the check should be:
> 
>       if (IS_ERR_OR_NULL(dev))
>               return PTR_ERR(dev);

Nice catch, I'll go double check the rest of the return values for this
scenario.
--
To unsubscribe from this list: send the line "unsubscribe linux-block" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to