Commit 83217790 authored by Roopa Prabhu's avatar Roopa Prabhu Committed by David S. Miller

enic: Check if rq/wq buf not NULL before freeing them

Signed-off-by: default avatarScott Feldman <scofeldm@cisco.com>
Signed-off-by: default avatarRoopa Prabhu <roprabhu@cisco.com>
Signed-off-by: default avatarVasanthy Kolluri <vkolluri@cisco.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ce16cc02
...@@ -77,8 +77,10 @@ void vnic_rq_free(struct vnic_rq *rq) ...@@ -77,8 +77,10 @@ void vnic_rq_free(struct vnic_rq *rq)
vnic_dev_free_desc_ring(vdev, &rq->ring); vnic_dev_free_desc_ring(vdev, &rq->ring);
for (i = 0; i < VNIC_RQ_BUF_BLKS_MAX; i++) { for (i = 0; i < VNIC_RQ_BUF_BLKS_MAX; i++) {
kfree(rq->bufs[i]); if (rq->bufs[i]) {
rq->bufs[i] = NULL; kfree(rq->bufs[i]);
rq->bufs[i] = NULL;
}
} }
rq->ctrl = NULL; rq->ctrl = NULL;
......
...@@ -77,8 +77,10 @@ void vnic_wq_free(struct vnic_wq *wq) ...@@ -77,8 +77,10 @@ void vnic_wq_free(struct vnic_wq *wq)
vnic_dev_free_desc_ring(vdev, &wq->ring); vnic_dev_free_desc_ring(vdev, &wq->ring);
for (i = 0; i < VNIC_WQ_BUF_BLKS_MAX; i++) { for (i = 0; i < VNIC_WQ_BUF_BLKS_MAX; i++) {
kfree(wq->bufs[i]); if (wq->bufs[i]) {
wq->bufs[i] = NULL; kfree(wq->bufs[i]);
wq->bufs[i] = NULL;
}
} }
wq->ctrl = NULL; wq->ctrl = NULL;
......
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