Re: [PATCH] netfilter: nft_numgen: fix ptr_ret.cocci warnings

2018-06-01 Thread Pablo Neira Ayuso
On Thu, May 24, 2018 at 12:40:04PM +0200, Laura Garcia wrote:
> On Wed, May 23, 2018 at 12:58 PM, kbuild test robot
>  wrote:
> > From: kbuild test robot 
> >
> > net/netfilter/nft_numgen.c:117:1-3: WARNING: PTR_ERR_OR_ZERO can be used
> >
> >
> >  Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
> >
> > Generated by: scripts/coccinelle/api/ptr_ret.cocci
> >
> > Fixes: d734a2888922 ("netfilter: nft_numgen: add map lookups for numgen 
> > statements")
> > CC: Laura Garcia Liebana 
> > Signed-off-by: kbuild test robot 
> 
> Acked-by: Laura Garcia Liebana 

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] netfilter: nft_numgen: fix ptr_ret.cocci warnings

2018-05-24 Thread Laura Garcia
On Wed, May 23, 2018 at 12:58 PM, kbuild test robot
 wrote:
> From: kbuild test robot 
>
> net/netfilter/nft_numgen.c:117:1-3: WARNING: PTR_ERR_OR_ZERO can be used
>
>
>  Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
>
> Generated by: scripts/coccinelle/api/ptr_ret.cocci
>
> Fixes: d734a2888922 ("netfilter: nft_numgen: add map lookups for numgen 
> statements")
> CC: Laura Garcia Liebana 
> Signed-off-by: kbuild test robot 

Acked-by: Laura Garcia Liebana 
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] netfilter: nft_numgen: fix ptr_ret.cocci warnings

2018-05-23 Thread kbuild test robot
From: kbuild test robot 

net/netfilter/nft_numgen.c:117:1-3: WARNING: PTR_ERR_OR_ZERO can be used


 Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR

Generated by: scripts/coccinelle/api/ptr_ret.cocci

Fixes: d734a2888922 ("netfilter: nft_numgen: add map lookups for numgen 
statements")
CC: Laura Garcia Liebana 
Signed-off-by: kbuild test robot 
---

 nft_numgen.c |5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

--- a/net/netfilter/nft_numgen.c
+++ b/net/netfilter/nft_numgen.c
@@ -114,10 +114,7 @@ static int nft_ng_inc_map_init(const str
  tb[NFTA_NG_SET_NAME],
  tb[NFTA_NG_SET_ID], genmask);
 
-   if (IS_ERR(priv->map))
-   return PTR_ERR(priv->map);
-
-   return 0;
+   return PTR_ERR_OR_ZERO(priv->map);
 }
 
 static int nft_ng_dump(struct sk_buff *skb, enum nft_registers dreg,
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html