On Jun 4, 2015, at 4:02 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > This code causes a static checker warning: > > drivers/infiniband/hw/usnic/usnic_uiom.c:476 usnic_uiom_alloc_pd() > warn: passing zero to 'PTR_ERR' > > This code isn't buggy, but iommu_domain_alloc() doesn't return an error > pointer so we can simplify the error handling and silence the static > checker warning. > > The static checker warning is to catch place which do: > > if (!ptr) > return ERR_PTR(ptr); > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Looks good to me, thanks. Reviewed-by: Dave Goodell <dgoodell@xxxxxxxxx> -Dave -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html