Commit ea47ab11 authored by Al Viro's avatar Al Viro
Browse files

putname(): IS_ERR_OR_NULL() is wrong here 



Mixing NULL and ERR_PTR() just in case is a Bad Idea(tm).  For
struct filename the former is wrong - failures are reported
as ERR_PTR(...), not as NULL.

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent b4a4f213

fs/namei.c

+1 −1
Original line number Diff line number Diff line
@@ -255,7 +255,7 @@ getname_kernel(const char * filename) 


void putname(struct filename *name)

{

	if (IS_ERR_OR_NULL(name))

	if (IS_ERR(name))

		return;



	BUG_ON(name->refcnt <= 0);