On Wed, Mar 20, 2013 at 11:03:53PM +0800, Lai Jiangshan wrote:
> > +enomem:
> > +       free_workqueue_attrs(tmp_attrs);
> > +       if (pwq_tbl) {
> > +               for_each_node(node)
> > +                       kfree(pwq_tbl[node]);
> 
> It will free the dfl_pwq multi times.

Oops, you're right.  We need to do


        for_eahc_node(node)
                if (pwq_tbl[node] != dfl_pwq)
                        kfree(pwq_tbl[node]);
        kfree(dfl_pwq);

Thanks!

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to