[PATCH] Fix module refcount leak in __set_personality()
authorSergey Vlasov <vsu@altlinux.ru>
Fri, 24 Mar 2006 11:18:38 +0000 (03:18 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Fri, 24 Mar 2006 15:33:30 +0000 (07:33 -0800)
If the change of personality does not lead to change of exec domain,
__set_personality() returned without releasing the module reference
acquired by lookup_exec_domain().

Signed-off-by: Sergey Vlasov <vsu@altlinux.ru>
Cc: Christoph Hellwig <hch@lst.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
kernel/exec_domain.c

index 867d6dbeb5743d1e49e1906b990031b4bdd7ee56..c01cead2cfd61978cb342853651fe9b158365976 100644 (file)
@@ -140,6 +140,7 @@ __set_personality(u_long personality)
        ep = lookup_exec_domain(personality);
        if (ep == current_thread_info()->exec_domain) {
                current->personality = personality;
+               module_put(ep->module);
                return 0;
        }