Messages in this thread Patch in this message | | | Date | Tue, 14 May 2013 21:06:40 +0800 | From | Chen Gang <> | Subject | [PATCH] kernel/workqueue.c: kfree issue, need check flag 'WQ_UNBOUND' when processing failure. |
| |
When failure occurs in __alloc_workqueue_key(), if 'flags' has 'WQ_UNBOUND', the destroy_workqueue() will not release 'wq'.
In this situation, we need release it, or will cause memory leak.
And also simplify the related code.
Signed-off-by: Chen Gang <gang.chen@asianux.com> --- kernel/workqueue.c | 9 +++++---- 1 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 1ae6028..bf644d4 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -4167,12 +4167,13 @@ struct workqueue_struct *__alloc_workqueue_key(const char *fmt, return wq; -err_free_wq: - free_workqueue_attrs(wq->unbound_attrs); - kfree(wq); - return NULL; err_destroy: destroy_workqueue(wq); + if (flags & WQ_UNBOUND) { +err_free_wq: + free_workqueue_attrs(wq->unbound_attrs); + kfree(wq); + } return NULL; } EXPORT_SYMBOL_GPL(__alloc_workqueue_key); -- 1.7.7.6
| |