Message ID | 1415129311-12480-1-git-send-email-fabf@skynet.be |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
On Tue, 2014-11-04 at 20:28 +0100, Fabian Frederick wrote: trivia: > diff --git a/net/ipv4/esp4.c b/net/ipv4/esp4.c [] > @@ -392,8 +392,11 @@ static int esp_input(struct xfrm_state *x, struct sk_buff *skb) > if (elen <= 0) > goto out; > > - if ((err = skb_cow_data(skb, 0, &trailer)) < 0) > + err = skb_cow_data(skb, 0, &trailer); > + > + if (err < 0) > goto out; Generally it's better/more common to use foo = bar(); if (!foo) [error_handler...] without the blank line between the set and the test. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/04/2014 08:28 PM, Fabian Frederick wrote: > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > net/ipv4/esp4.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/net/ipv4/esp4.c b/net/ipv4/esp4.c > index 360b565..9dd66ee 100644 > --- a/net/ipv4/esp4.c > +++ b/net/ipv4/esp4.c > @@ -392,8 +392,11 @@ static int esp_input(struct xfrm_state *x, struct sk_buff *skb) > if (elen <= 0) > goto out; > > - if ((err = skb_cow_data(skb, 0, &trailer)) < 0) > + err = skb_cow_data(skb, 0, &trailer); > + If you already feel the need to change this (?), then please don't add an extra newline here ... > + if (err < 0) > goto out; > + > nfrags = err; > > assoclen = sizeof(*esph); > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Joe Perches <joe@perches.com> Date: Tue, 04 Nov 2014 11:33:51 -0800 > On Tue, 2014-11-04 at 20:28 +0100, Fabian Frederick wrote: > > trivia: > >> diff --git a/net/ipv4/esp4.c b/net/ipv4/esp4.c > [] >> @@ -392,8 +392,11 @@ static int esp_input(struct xfrm_state *x, struct sk_buff *skb) >> if (elen <= 0) >> goto out; >> >> - if ((err = skb_cow_data(skb, 0, &trailer)) < 0) >> + err = skb_cow_data(skb, 0, &trailer); >> + >> + if (err < 0) >> goto out; > > Generally it's better/more common to use > > foo = bar(); > if (!foo) > [error_handler...] > > without the blank line between the set > and the test. +1 -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/net/ipv4/esp4.c b/net/ipv4/esp4.c index 360b565..9dd66ee 100644 --- a/net/ipv4/esp4.c +++ b/net/ipv4/esp4.c @@ -392,8 +392,11 @@ static int esp_input(struct xfrm_state *x, struct sk_buff *skb) if (elen <= 0) goto out; - if ((err = skb_cow_data(skb, 0, &trailer)) < 0) + err = skb_cow_data(skb, 0, &trailer); + + if (err < 0) goto out; + nfrags = err; assoclen = sizeof(*esph);
Signed-off-by: Fabian Frederick <fabf@skynet.be> --- net/ipv4/esp4.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)