openvswitch: fix odd_ptr_err.cocci warnings
authorWu Fengguang <fengguang.wu@intel.com>
Wed, 24 Dec 2014 06:41:39 +0000 (14:41 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 24 Dec 2014 20:18:09 +0000 (15:18 -0500)
net/openvswitch/vport-gre.c:188:5-11: inconsistent IS_ERR and PTR_ERR, PTR_ERR on line 189

 PTR_ERR should access the value just tested by IS_ERR

Semantic patch information:
 There can be false positives in the patch case, where it is the call
 IS_ERR that is wrong.

Generated by: scripts/coccinelle/tests/odd_ptr_err.cocci

CC: Pravin B Shelar <pshelar@nicira.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Acked-by: Pravin B Shelar <pshelar@nicira.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/openvswitch/vport-gre.c

index 28f54e9a6b80618f1f28312d58eb58e222ea4426..d4168c442db5af8683c72bfb24ce95c789dadcf8 100644 (file)
@@ -186,7 +186,7 @@ static int gre_tnl_send(struct vport *vport, struct sk_buff *skb)
        /* Push Tunnel header. */
        skb = __build_header(skb, tunnel_hlen);
        if (IS_ERR(skb)) {
-               err = PTR_ERR(rt);
+               err = PTR_ERR(skb);
                skb = NULL;
                goto err_free_rt;
        }