coredump: use SUID_DUMPABLE_ENABLED rather than hardcoded 1
Cosmetic. Change setup_new_exec() and task_dumpable() to use SUID_DUMPABLE_ENABLED for /bin/grep. [akpm@linux-foundation.org: checkpatch fixes] Signed-off-by: Oleg Nesterov <oleg@redhat.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
committed by
Linus Torvalds
parent
12a2b4b224
commit
0f4cfb2e4e
@@ -1097,7 +1097,7 @@ void setup_new_exec(struct linux_binprm * bprm)
|
|||||||
current->sas_ss_sp = current->sas_ss_size = 0;
|
current->sas_ss_sp = current->sas_ss_size = 0;
|
||||||
|
|
||||||
if (uid_eq(current_euid(), current_uid()) && gid_eq(current_egid(), current_gid()))
|
if (uid_eq(current_euid(), current_uid()) && gid_eq(current_egid(), current_gid()))
|
||||||
set_dumpable(current->mm, 1);
|
set_dumpable(current->mm, SUID_DUMPABLE_ENABLED);
|
||||||
else
|
else
|
||||||
set_dumpable(current->mm, suid_dumpable);
|
set_dumpable(current->mm, suid_dumpable);
|
||||||
|
|
||||||
|
|||||||
+1
-1
@@ -103,7 +103,7 @@ static inline int task_dumpable(struct task_struct *task)
|
|||||||
if (mm)
|
if (mm)
|
||||||
dumpable = get_dumpable(mm);
|
dumpable = get_dumpable(mm);
|
||||||
task_unlock(task);
|
task_unlock(task);
|
||||||
if(dumpable == 1)
|
if (dumpable == SUID_DUMPABLE_ENABLED)
|
||||||
return 1;
|
return 1;
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user