On Thu, Nov 23, 2023 at 03:10:51PM +0100, Florian Westphal wrote: > 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. Thanks, I will send a patch.
