2014-07-11 20:25:49

by Himangi Saraogi

[permalink] [raw]
Subject: [PATCH] net: ipv6: Use BUG_ON

The semantic patch that makes the transformation is as follows:

// <smpl>
@@ expression e; @@
-if (e) BUG();
+BUG_ON(e);
// </smpl>

Signed-off-by: Himangi Saraogi <[email protected]>
Acked-by: Julia Lawall <[email protected]>
---
To send to: "David S. Miller" <[email protected]>,Alexey Kuznetsov <[email protected]>,James Morris <[email protected]>,Hideaki YOSHIFUJI <[email protected]>,Patrick McHardy <[email protected]>,[email protected],[email protected]
net/ipv6/raw.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/net/ipv6/raw.c b/net/ipv6/raw.c
index b2dc60b..dee80fb 100644
--- a/net/ipv6/raw.c
+++ b/net/ipv6/raw.c
@@ -588,8 +588,7 @@ static int rawv6_push_pending_frames(struct sock *sk, struct flowi6 *fl6,
}

offset += skb_transport_offset(skb);
- if (skb_copy_bits(skb, offset, &csum, 2))
- BUG();
+ BUG_ON(skb_copy_bits(skb, offset, &csum, 2));

/* in case cksum was not initialized */
if (unlikely(csum))
@@ -601,8 +600,7 @@ static int rawv6_push_pending_frames(struct sock *sk, struct flowi6 *fl6,
if (csum == 0 && fl6->flowi6_proto == IPPROTO_UDP)
csum = CSUM_MANGLED_0;

- if (skb_store_bits(skb, offset, &csum, 2))
- BUG();
+ BUG_ON(skb_store_bits(skb, offset, &csum, 2));

send:
err = ip6_push_pending_frames(sk);
--
1.9.1


2014-07-11 22:07:06

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] net: ipv6: Use BUG_ON

From: Himangi Saraogi <[email protected]>
Date: Sat, 12 Jul 2014 01:55:38 +0530

> The semantic patch that makes the transformation is as follows:
>
> // <smpl>
> @@ expression e; @@
> -if (e) BUG();
> +BUG_ON(e);
> // </smpl>
>
> Signed-off-by: Himangi Saraogi <[email protected]>
> Acked-by: Julia Lawall <[email protected]>

Applied.