Yeah, not sure i see it either. cheers, jamal
On Mon, Apr 24, 2017 at 8:09 PM, Cong Wang <xiyou.wangc...@gmail.com> wrote: > 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.