diff options
author | Zoltan Kiss <zoltan.kiss@citrix.com> | 2014-07-18 14:08:05 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-07-20 23:56:06 -0400 |
commit | d8cfbfc4660054150ca1b7c501a8edc0771022f9 (patch) | |
tree | b0415e53ab9d486d7b8345bcb06704b239e5c4c0 /drivers/net/xen-netback | |
parent | 1b860da0404a76af8533099ffe0a965490939369 (diff) |
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/net/xen-netback')
-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, | |||
1045 | 1045 | ||
1046 | /* Check status of header. */ | 1046 | /* Check status of header. */ |
1047 | err = (*gopp_copy)->status; | 1047 | err = (*gopp_copy)->status; |
1048 | (*gopp_copy)++; | ||
1049 | if (unlikely(err)) { | 1048 | if (unlikely(err)) { |
1050 | if (net_ratelimit()) | 1049 | if (net_ratelimit()) |
1051 | netdev_dbg(queue->vif->dev, | 1050 | netdev_dbg(queue->vif->dev, |
@@ -1058,6 +1057,7 @@ static int xenvif_tx_check_gop(struct xenvif_queue *queue, | |||
1058 | xenvif_idx_release(queue, pending_idx, | 1057 | xenvif_idx_release(queue, pending_idx, |
1059 | XEN_NETIF_RSP_ERROR); | 1058 | XEN_NETIF_RSP_ERROR); |
1060 | } | 1059 | } |
1060 | (*gopp_copy)++; | ||
1061 | 1061 | ||
1062 | check_frags: | 1062 | check_frags: |
1063 | for (i = 0; i < nr_frags; i++, gop_map++) { | 1063 | for (i = 0; i < nr_frags; i++, gop_map++) { |