ipv6: fix warning in xfrm6_mode_tunnel_input
authorstephen hemminger <stephen@networkplumber.org>
Sat, 16 Feb 2013 07:38:15 +0000 (07:38 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 18 Feb 2013 17:42:47 +0000 (12:42 -0500)
Should not use assignment in conditional:
 warning: suggest parentheses around assignment used as truth value [-Wparentheses]

Problem introduced by:
commit 14bbd6a565e1bcdc240d44687edb93f721cfdf99
Author: Pravin B Shelar <pshelar@nicira.com>
Date:   Thu Feb 14 09:44:49 2013 +0000

    net: Add skb_unclone() helper function.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/xfrm6_mode_tunnel.c

index 93c41a81c4c32632338a87c43e0adf368ca2c7da..9bf6a74a71d261061201e3863fd32713c91720d7 100644 (file)
@@ -69,7 +69,8 @@ static int xfrm6_mode_tunnel_input(struct xfrm_state *x, struct sk_buff *skb)
        if (!pskb_may_pull(skb, sizeof(struct ipv6hdr)))
                goto out;
 
-       if (err = skb_unclone(skb, GFP_ATOMIC))
+       err = skb_unclone(skb, GFP_ATOMIC);
+       if (err)
                goto out;
 
        if (x->props.flags & XFRM_STATE_DECAP_DSCP)