bnx2x: NULL dereference on error in debug code
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 8 Jan 2013 03:42:14 +0000 (03:42 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 10 Jan 2013 07:53:26 +0000 (23:53 -0800)
"vfop" is NULL here.  I've changed the debugging to not use it.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Ariel Elior <ariele@broadcom.com>
Acked-by: Ariel Elior <ariele@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c

index f2f5c8ecfd791900ad7fd5c2cc8cd09ecc50fa08..08db503e86d897f287bb3518931d3b57f5b3232c 100644 (file)
@@ -463,8 +463,7 @@ static int bnx2x_vfop_qdtor_cmd(struct bnx2x *bp,
                return bnx2x_vfop_transition(bp, vf, bnx2x_vfop_qdtor,
                                             cmd->block);
        }
-       DP(BNX2X_MSG_IOV, "VF[%d] failed to add a vfop. rc %d\n",
-          vf->abs_vfid, vfop->rc);
+       DP(BNX2X_MSG_IOV, "VF[%d] failed to add a vfop.\n", vf->abs_vfid);
        return -ENOMEM;
 }