Commit 1fc3de94 authored by Yuval Mintz's avatar Yuval Mintz Committed by David S. Miller

bnx2x: Fix static checker warning regarding `txdata_ptr'

Incorrect checking of array instead of array contents in panic_dump
flow - results of commit e2611998 ("bnx2x: Safe bnx2x_panic_dump()").
Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarYuval Mintz <Yuval.Mintz@qlogic.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent b0b46c77
...@@ -985,7 +985,7 @@ void bnx2x_panic_dump(struct bnx2x *bp, bool disable_int) ...@@ -985,7 +985,7 @@ void bnx2x_panic_dump(struct bnx2x *bp, bool disable_int)
/* Tx */ /* Tx */
for_each_cos_in_tx_queue(fp, cos) for_each_cos_in_tx_queue(fp, cos)
{ {
if (!fp->txdata_ptr) if (!fp->txdata_ptr[cos])
break; break;
txdata = *fp->txdata_ptr[cos]; txdata = *fp->txdata_ptr[cos];
...@@ -1140,7 +1140,7 @@ void bnx2x_panic_dump(struct bnx2x *bp, bool disable_int) ...@@ -1140,7 +1140,7 @@ void bnx2x_panic_dump(struct bnx2x *bp, bool disable_int)
for_each_cos_in_tx_queue(fp, cos) { for_each_cos_in_tx_queue(fp, cos) {
struct bnx2x_fp_txdata *txdata = fp->txdata_ptr[cos]; struct bnx2x_fp_txdata *txdata = fp->txdata_ptr[cos];
if (!fp->txdata_ptr) if (!fp->txdata_ptr[cos])
break; break;
if (!txdata->tx_cons_sb) if (!txdata->tx_cons_sb)
......
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