> We accidentally return success instead of -ENOMEM here. > > Fixes: fe56b9e6a8d9 ('qed: Add module with basic common support') > Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com> >
Thanks. Although you might also argue that it's good enough for this function to always return -ENOMEM [given it's a resource allocation functions] instead of bothering with setting 'rc' with correct error codes. Regardless, Acked-by: Yuval Mintz <yuval.mi...@qlogic.com -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html