Messages in this thread | | | Subject | Re: [patch] fs: Make unload_nls() NULL pointer safe | From | Dave Kleikamp <> | Date | Mon, 17 Aug 2009 07:40:06 -0500 |
| |
On Sun, 2009-08-16 at 21:05 +0000, Thomas Gleixner wrote: > plain text document attachment (unload_nls.patch) > Most call sites of unload_nls() do: > if (nls) > unload_nls(nls); > > Check the pointer inside unload_nls() like we do in kfree() and > simplify the call sites. > > Signed-off-by: Thomas Gleixner <tglx@linutronix.de> > Cc: Steve French <sfrench@us.ibm.com> > Cc: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> > Cc: Roman Zippel <zippel@linux-m68k.org>
Acked-by: Dave Kleikamp <shaggy@linux.vnet.ibm.com>
> Cc: Petr Vandrovec <vandrove@vc.cvut.cz> > Cc: Anton Altaparmakov <aia21@cantab.net> > > --- > fs/befs/linuxvfs.c | 7 +------ > fs/cifs/cifsfs.c | 3 +-- > fs/fat/inode.c | 16 ++++------------ > fs/hfs/mdb.c | 6 ++---- > fs/hfsplus/super.c | 6 ++---- > fs/isofs/inode.c | 8 ++------ > fs/jfs/super.c | 9 +++------ > fs/ncpfs/inode.c | 12 ++---------- > fs/ncpfs/ioctl.c | 6 ++---- > fs/nls/nls_base.c | 3 ++- > fs/ntfs/super.c | 10 ++++------ > fs/smbfs/inode.c | 10 ++-------- > 12 files changed, 27 insertions(+), 69 deletions(-) -- David Kleikamp IBM Linux Technology Center
| |