Re: [PATCH nf-next 1/8] netfilter: flowtable: move nf_flowtable out of container structures

[Florian Westphal <fw@xxxxxxxxx>]

Simon Horman <horms@xxxxxxxxxx> wrote:
> > -	err = nf_flow_table_init(&ct_ft->nf_ft);
> > +	ct_ft->nf_ft = kzalloc(sizeof(*ct_ft->nf_ft), GFP_KERNEL);
> > +	if (!ct_ft->nf_ft)
> > +		goto err_alloc;
> 
> Hi Florian,
> 
> This branch will cause the function to return err, but err is 0 here.
> Perhaps it should be set to a negative error value instead?

Yes, this should fail with -ENOMEM.