diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-02-16 04:56:10 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-02-17 12:18:43 -0500 |
| commit | 785f35775d968e0f45231b754e945fcb3ed6bded (patch) | |
| tree | a3fd8bef4bf3cc14c46ec5e589935fb5ab8f63f2 | |
| parent | 2bd624b4611ffee36422782d16e1c944d1351e98 (diff) | |
dpaa_eth: small leak on error
This should be >= instead of > here. It means that we don't increment
the free count enough so it becomes off by one.
Fixes: 9ad1a3749333 ("dpaa_eth: add support for DPAA Ethernet")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c index c9b7ad65e563..726b5693ae8a 100644 --- a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c +++ b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | |||
| @@ -1668,7 +1668,7 @@ static struct sk_buff *sg_fd_to_skb(const struct dpaa_priv *priv, | |||
| 1668 | 1668 | ||
| 1669 | free_buffers: | 1669 | free_buffers: |
| 1670 | /* compensate sw bpool counter changes */ | 1670 | /* compensate sw bpool counter changes */ |
| 1671 | for (i--; i > 0; i--) { | 1671 | for (i--; i >= 0; i--) { |
| 1672 | dpaa_bp = dpaa_bpid2pool(sgt[i].bpid); | 1672 | dpaa_bp = dpaa_bpid2pool(sgt[i].bpid); |
| 1673 | if (dpaa_bp) { | 1673 | if (dpaa_bp) { |
| 1674 | count_ptr = this_cpu_ptr(dpaa_bp->percpu_count); | 1674 | count_ptr = this_cpu_ptr(dpaa_bp->percpu_count); |
