Commit a44acd55 authored by Dan Carpenter's avatar Dan Carpenter Committed by David S. Miller

bnx2x: unlock before returning an error

We introduced a new return here but forgot to drop the lock.
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarDmitry Kravkov <dmitry@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8bca5d1e
...@@ -1836,6 +1836,7 @@ static int bnx2x_vlan_mac_del_all(struct bnx2x *bp, ...@@ -1836,6 +1836,7 @@ static int bnx2x_vlan_mac_del_all(struct bnx2x *bp,
rc = exeq->remove(bp, exeq->owner, exeq_pos); rc = exeq->remove(bp, exeq->owner, exeq_pos);
if (rc) { if (rc) {
BNX2X_ERR("Failed to remove command\n"); BNX2X_ERR("Failed to remove command\n");
spin_unlock_bh(&exeq->lock);
return rc; return rc;
} }
list_del(&exeq_pos->link); list_del(&exeq_pos->link);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment