On Fri, Dec 23, 2011 at 03:10:39PM +0100, Eric Dumazet wrote:
> Le vendredi 23 décembre 2011 à 14:42 +0100, pablo@xxxxxxxxxxxxx a
> écrit :
> > From: Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx>
> >
>
>
> > +struct nf_acct *nfnl_acct_find_get(const char *acct_name)
> > +{
> > + struct nf_acct *cur, *acct = NULL;
> > +
> > + rcu_read_lock();
> > + list_for_each_entry(cur, &nfnl_acct_list, head) {
> > + if (strncmp(cur->name, acct_name, NFACCT_NAME_MAX)!= 0)
> > + continue;
> > +
> > + if (!try_module_get(THIS_MODULE))
> > + return NULL;
> > +
> > + if (!atomic_inc_not_zero(&cur->refcnt))
>
> You leak the module reference here.
Indeed, thanks. I've fixed this.
--
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
