diff options
| author |   Zoltan Kiss <zoltan.kiss@citrix.com> | 2014-07-18 19:08:05 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-07-20 20:56:06 -0700 |
| commit | d8cfbfc4660054150ca1b7c501a8edc0771022f9 (patch) | |
| tree | b0415e53ab9d486d7b8345bcb06704b239e5c4c0 /drivers | |
| parent | xen-netback: Fix releasing header slot on error path (diff) | |
| download | linux-dev-d8cfbfc4660054150ca1b7c501a8edc0771022f9.tar.xz linux-dev-d8cfbfc4660054150ca1b7c501a8edc0771022f9.zip | |
xen-netback: Fix pointer incrementation to avoid incorrect logging
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>
Diffstat (limited to 'drivers')
| -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 6fff911dc134..c65b636bcab9 100644 --- a/drivers/net/xen-netback/netback.c +++ b/drivers/net/xen-netback/netback.c @@ -1045,7 +1045,6 @@ static int xenvif_tx_check_gop(struct xenvif_queue *queue, /* Check status of header. */ err = (*gopp_copy)->status; - (*gopp_copy)++; if (unlikely(err)) { if (net_ratelimit()) netdev_dbg(queue->vif->dev, @@ -1058,6 +1057,7 @@ static int xenvif_tx_check_gop(struct xenvif_queue *queue, xenvif_idx_release(queue, pending_idx, XEN_NETIF_RSP_ERROR); } + (*gopp_copy)++; check_frags: for (i = 0; i < nr_frags; i++, gop_map++) { |