]> git.hungrycats.org Git - linux/commit
[PATCH] do_fork() error path memory leak
authorAndrew Morton <akpm@osdl.org>
Mon, 12 Apr 2004 08:03:03 +0000 (01:03 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Mon, 12 Apr 2004 08:03:03 +0000 (01:03 -0700)
commit2386894043fc3be2c0d2d25edb8c741f18c592ec
treee86a2a9c5580679ece847bce71cb029836365cce
parent94ce318546808714dbbdc8a00a1332df9ab79fad
[PATCH] do_fork() error path memory leak

From: <john.l.byrne@hp.com>

In do_fork(), if an error occurs after the mm_struct for the child has been
allocated, it is never freed.  The exit_mm() meant to free it increments
the mm_count and this count is never decremented.  (For a running process
that is exitting, schedule() takes care this; however, the child process
being cleaned up is not running.) In the CLONE_VM case, the parent's
mm_struct will get an extra mm_count and so it will never be freed.

This patch should fix both the CLONE_VM and the not CLONE_VM case; the test
of p->active_mm prevents a panic in the case that a kernel-thread is being
cloned.
kernel/fork.c