datapath: Do not shadow 'err' variable name in tnl_send().

The sparse checker reported that 'err' was used for two different variables
in tnl_send().  The two variables have different types, so this patch
renames the inner one.

Jesse confirmed that the original code was correct as written.  This patch
does not change its behavior.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>

diff --git a/datapath/tunnel.c b/datapath/tunnel.c
index eac3fa3..bf0ab56 100644 (file)
--- a/datapath/tunnel.c
+++ b/datapath/tunnel.c
@@ -1294,12 +1294,12 @@ int tnl_send(struct vport *vport, struct sk_buff *skb)
                                vport_receive(cache_vport, skb);
                                sent_len += orig_len;
                        } else {
-                               int err;
+                               int xmit_err;
 
                                skb->dev = rt_dst(rt).dev;
-                               err = dev_queue_xmit(skb);
+                               xmit_err = dev_queue_xmit(skb);
 
-                               if (likely(net_xmit_eval(err) == 0))
+                               if (likely(net_xmit_eval(xmit_err) == 0))
                                        sent_len += orig_len;
                        }
                } else