Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d8b295f2 authored by Russell King's avatar Russell King Committed by Linus Torvalds
Browse files

[PATCH] Fix missing parens in set_personality() 



If you call set_personality() with an expression such as:

	set_personality(foo ? PERS_FOO1 : PERS_FOO2);

then this evaluates to:

	((current->personality == foo ? PERS_FOO1 : PERS_FOO2) ? ...

which is obviously not the intended result.  Add the missing parents
to ensure this gets evaluated as expected:

	((current->personality == (foo ? PERS_FOO1 : PERS_FOO2)) ? ...

Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent e40c6759

include/linux/personality.h

+1 −1
Original line number Diff line number Diff line
@@ -114,7 +114,7 @@ struct exec_domain { 
 * Change personality of the currently running process.

 */

#define set_personality(pers) \

	((current->personality == pers) ? 0 : __set_personality(pers))

	((current->personality == (pers)) ? 0 : __set_personality(pers))



#endif /* __KERNEL__ */