On Mon, Apr 24, 2017 at 4:41 PM, Stephen Hemminger <step...@networkplumber.org> wrote: > This looks like a false positive > ________________________________________________________________________________________________________ > *** CID 1405487: Resource leaks (RESOURCE_LEAK) > /net/sched/act_api.c: 635 in tcf_action_init_1() > 629 * if it exists and is only bound to in a_o->init() then > 630 * ACT_P_CREATED is not returned (a zero is). > 631 */ > 632 if (err != ACT_P_CREATED) > 633 module_put(a_o->owner); > 634 >>>> CID 1405487: Resource leaks (RESOURCE_LEAK) >>>> Variable "cookie" going out of scope leaks the storage it points to. > 635 return a;
I don't see how we could leak it either.