On Thu, 1 Nov 2012 17:36:55 +0800 Xiaotian Feng <[email protected]> wrote:
> there's a name leak introduced by commit 91a27b2, add the missing > putname. > > Signed-off-by: Xiaotian Feng <[email protected]> > Cc: Jeff Layton <[email protected]> > Cc: Al Viro <[email protected]> > --- > mm/swapfile.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/mm/swapfile.c b/mm/swapfile.c > index 71cd288..459fe30 100644 > --- a/mm/swapfile.c > +++ b/mm/swapfile.c > @@ -1608,6 +1608,8 @@ SYSCALL_DEFINE1(swapoff, const char __user *, > specialfile) > out_dput: > filp_close(victim, NULL); > out: > + if (pathname && !IS_ERR(pathname)) > + putname(pathname); > return err; > } > Nice catch. Reviewed-by: Jeff Layton <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
