On 29 September 2014 18:14, Sergei Shtylyov
<sergei.shtylyov@xxxxxxxxxxxxxxxxxx> wrote:
>>
>> index a476b99..19e79f0 100644
>> --- a/net/netfilter/nf_tables_api.c
>> +++ b/net/netfilter/nf_tables_api.c
>> @@ -2344,6 +2344,11 @@ static int nf_tables_fill_set(struct sk_buff *skb,
>> const struct nft_ctx *ctx,
>> goto nla_put_failure;
>> }
>>
>> + if (set->policy != NFT_SET_POL_PERFORMANCE) {
>> + if (nla_put_be32(skb, NFTA_SET_POLICY,
>> htonl(set->policy)))
>> + goto nla_put_failure;
>
>
> Why not fold these two *if* stetement into a single one?
>
I don't have any preference.
Should I send a patch to change it?
regards.
--
Arturo Borrero González
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
