Commit 66b59f9b authored by Varun Prakash's avatar Varun Prakash Committed by Nicholas Bellinger

cxgbit: add missing __kfree_skb()

Call __kfree_skb() after processing skb to avoid
memory leak.
Signed-off-by: default avatarVarun Prakash <varun@chelsio.com>
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
parent ededd039
...@@ -1510,11 +1510,13 @@ cxgbit_pass_open_rpl(struct cxgbit_device *cdev, struct sk_buff *skb) ...@@ -1510,11 +1510,13 @@ cxgbit_pass_open_rpl(struct cxgbit_device *cdev, struct sk_buff *skb)
if (!cnp) { if (!cnp) {
pr_info("%s stid %d lookup failure\n", __func__, stid); pr_info("%s stid %d lookup failure\n", __func__, stid);
return; goto rel_skb;
} }
cxgbit_wake_up(&cnp->com.wr_wait, __func__, rpl->status); cxgbit_wake_up(&cnp->com.wr_wait, __func__, rpl->status);
cxgbit_put_cnp(cnp); cxgbit_put_cnp(cnp);
rel_skb:
__kfree_skb(skb);
} }
static void static void
...@@ -1530,11 +1532,13 @@ cxgbit_close_listsrv_rpl(struct cxgbit_device *cdev, struct sk_buff *skb) ...@@ -1530,11 +1532,13 @@ cxgbit_close_listsrv_rpl(struct cxgbit_device *cdev, struct sk_buff *skb)
if (!cnp) { if (!cnp) {
pr_info("%s stid %d lookup failure\n", __func__, stid); pr_info("%s stid %d lookup failure\n", __func__, stid);
return; goto rel_skb;
} }
cxgbit_wake_up(&cnp->com.wr_wait, __func__, rpl->status); cxgbit_wake_up(&cnp->com.wr_wait, __func__, rpl->status);
cxgbit_put_cnp(cnp); cxgbit_put_cnp(cnp);
rel_skb:
__kfree_skb(skb);
} }
static void static void
...@@ -1819,12 +1823,16 @@ static void cxgbit_set_tcb_rpl(struct cxgbit_device *cdev, struct sk_buff *skb) ...@@ -1819,12 +1823,16 @@ static void cxgbit_set_tcb_rpl(struct cxgbit_device *cdev, struct sk_buff *skb)
struct tid_info *t = lldi->tids; struct tid_info *t = lldi->tids;
csk = lookup_tid(t, tid); csk = lookup_tid(t, tid);
if (unlikely(!csk)) if (unlikely(!csk)) {
pr_err("can't find connection for tid %u.\n", tid); pr_err("can't find connection for tid %u.\n", tid);
else goto rel_skb;
} else {
cxgbit_wake_up(&csk->com.wr_wait, __func__, rpl->status); cxgbit_wake_up(&csk->com.wr_wait, __func__, rpl->status);
}
cxgbit_put_csk(csk); cxgbit_put_csk(csk);
rel_skb:
__kfree_skb(skb);
} }
static void cxgbit_rx_data(struct cxgbit_device *cdev, struct sk_buff *skb) static void cxgbit_rx_data(struct cxgbit_device *cdev, struct sk_buff *skb)
......
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