On Tue, May 14, 2013 at 09:06:40PM +0800, Chen Gang wrote:> err_destroy:> destroy_workqueue(wq);> + if (flags & WQ_UNBOUND) {> +err_free_wq:> + free_workqueue_attrs(wq->unbound_attrs);> + kfree(wq);> + }Doesn't the above make the code free wq twice on after err_destroy?-- tejun