diff options
| author | Zoltan Kiss <zoltan.kiss@citrix.com> | 2014-07-18 19:08:05 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-07-28 08:08:26 -0700 |
| commit | c780438d32b674c825f355fdecc0a012e3e7652f (patch) | |
| tree | b9e490659bdaffb540d66de28f5f69bd3220d7fd | |
| parent | 4500fc0ec693850ce8cd6ae505cbffcc73f91786 (diff) | |
xen-netback: Fix pointer incrementation to avoid incorrect logging
[ Upstream commit d8cfbfc4660054150ca1b7c501a8edc0771022f9 ]
Due to this pointer is increased prematurely, the error log contains rubbish.
Signed-off-by: Zoltan Kiss <zoltan.kiss@citrix.com>
Reported-by: Armin Zentai <armin.zentai@ezit.hu>
Cc: netdev@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: xen-devel@lists.xenproject.org
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/net/xen-netback/netback.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/xen-netback/netback.c b/drivers/net/xen-netback/netback.c index 2c5a6f9f61d8..034645a2978f 100644 --- a/drivers/net/xen-netback/netback.c +++ b/drivers/net/xen-netback/netback.c @@ -1022,7 +1022,6 @@ static int xenvif_tx_check_gop(struct xenvif *vif, /* Check status of header. */ err = (*gopp_copy)->status; - (*gopp_copy)++; if (unlikely(err)) { if (net_ratelimit()) netdev_dbg(vif->dev, @@ -1035,6 +1034,7 @@ static int xenvif_tx_check_gop(struct xenvif *vif, xenvif_idx_release(vif, pending_idx, XEN_NETIF_RSP_ERROR); } + (*gopp_copy)++; check_frags: for (i = 0; i < nr_frags; i++, gop_map++) { |