Hugh Dickins 7c2c7d9930 fix setuid sometimes wouldn't
check_unsafe_exec() also notes whether the fs_struct is being
shared by more threads than will get killed by the exec, and if so
sets LSM_UNSAFE_SHARE to make bprm_set_creds() careful about euid.
But /proc/<pid>/cwd and /proc/<pid>/root lookups make transient
use of get_fs_struct(), which also raises that sharing count.

This might occasionally cause a setuid program not to change euid,
in the same way as happened with files->count (check_unsafe_exec
also looks at sighand->count, but /proc doesn't raise that one).

We'd prefer exec not to unshare fs_struct: so fix this in procfs,
replacing get_fs_struct() by get_fs_path(), which does path_get
while still holding task_lock, instead of raising fs->count.

Signed-off-by: Hugh Dickins <hugh@veritas.com>
Cc: stable@kernel.org
___

 fs/proc/base.c |   50 +++++++++++++++--------------------------------
 1 file changed, 16 insertions(+), 34 deletions(-)
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2009-03-28 17:30:00 -07:00
..
2009-03-27 14:44:00 -04:00
2009-01-22 13:15:57 +03:00
2009-01-22 13:15:57 +03:00
2009-01-22 13:15:58 +03:00
2009-01-22 13:15:57 +03:00
2009-03-27 14:44:01 -04:00
2009-01-22 13:15:58 +03:00
2009-03-27 14:44:01 -04:00
2008-11-14 10:39:25 +11:00
2009-03-22 11:28:37 -07:00
2009-03-27 14:43:59 -04:00
2008-10-17 02:38:36 +11:00
2009-03-27 14:44:05 -04:00
2009-01-22 13:15:55 +03:00
2009-03-27 14:44:02 -04:00
2009-01-22 13:15:58 +03:00
2009-03-28 17:30:00 -07:00
2009-01-22 13:15:59 +03:00
2009-03-26 02:18:35 +01:00
2009-01-22 13:15:59 +03:00
2009-03-27 14:44:03 -04:00
2008-12-31 18:07:43 -05:00
2009-01-03 11:45:54 -08:00
2009-02-06 17:34:07 -08:00
2009-03-27 14:44:03 -04:00
2009-03-27 14:44:03 -04:00
2009-03-28 17:30:00 -07:00
2009-03-27 14:43:57 -04:00
2009-03-27 14:44:05 -04:00
2009-03-16 08:32:27 -06:00
2009-03-28 17:30:00 -07:00
2009-03-16 08:34:35 -06:00
2009-03-28 17:30:00 -07:00
2009-03-16 08:34:35 -06:00
2009-02-18 15:37:53 -08:00
2009-02-18 15:37:53 -08:00