Jiri Kosina <jkos...@suse.cz> writes:

>       if (personality(current->personality) == PER_LINUX32
> -         && personality == PER_LINUX)
> -             personality = PER_LINUX32;
> +         && personality(personality) == PER_LINUX)
> +             personality &= ~PER_LINUX | PER_LINUX32;

That doesn't work.  ~PER_LINUX is -1, so this is a no-op.

>       ret = sys_personality(personality);
> -     if (ret == PER_LINUX32)
> -             ret = PER_LINUX;
> +     if (personality(ret) == PER_LINUX32)
> +             ret &= ~PER_LINUX32 | PER_LINUX;

That only "works" because PER_LINUX is 0.

Andreas.

-- 
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to