From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Max Laier" Subject: [PATCH] (theoretical) leak in do_mbind() Date: Sun, 14 Jun 2009 19:40:33 +0200 Message-ID: Mime-Version: 1.0 Content-Transfer-Encoding: 7BIT Return-path: Sender: linux-numa-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" To: linux-numa@vger.kernel.org Currently migrate_prep() can't return an error, but if it ever does we should free the newly created policy again. Trivial patch, really. In case my webmail trashes the patch: http://mlaier.homeunix.org/~mlaier/mbind_leaklet.diff Author: Max Laier Date: Sun Jun 14 07:35:57 2009 +0200 Plug a small leak in mbind in case migrate_prep() fails. diff --git a/mm/mempolicy.c b/mm/mempolicy.c index 3eb4a6f..7169059 100644 --- a/mm/mempolicy.c +++ b/mm/mempolicy.c @@ -975,8 +975,10 @@ static long do_mbind(unsigned long start, unsigned long len, if (flags & (MPOL_MF_MOVE | MPOL_MF_MOVE_ALL)) { err = migrate_prep(); - if (err) + if (err) { + mpol_put(new); return err; + } } down_write(&mm->mmap_sem); vma = check_range(mm, start, end, nmask, -- /"\ Best regards, | mlaier@freebsd.org \ / Max Laier | ICQ #67774661 X http://pf4freebsd.love2party.net/ | mlaier@EFnet / \ ASCII Ribbon Campaign | Against HTML Mail and News