Commit 8409cca7 authored by Serge E. Hallyn's avatar Serge E. Hallyn Committed by Linus Torvalds

userns: allow ptrace from non-init user namespaces

ptrace is allowed to tasks in the same user namespace according to the
usual rules (i.e.  the same rules as for two tasks in the init user
namespace).  ptrace is also allowed to a user namespace to which the
current task the has CAP_SYS_PTRACE capability.

Changelog:
	Dec 31: Address feedback by Eric:
		. Correct ptrace uid check
		. Rename may_ptrace_ns to ptrace_capable
		. Also fix the cap_ptrace checks.
	Jan  1: Use const cred struct
	Jan 11: use task_ns_capable() in place of ptrace_capable().
	Feb 23: same_or_ancestore_user_ns() was not an appropriate
		check to constrain cap_issubset.  Rather, cap_issubset()
		only is meaningful when both capsets are in the same
		user_ns.
Signed-off-by: default avatarSerge E. Hallyn <serge.hallyn@canonical.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Acked-by: default avatarDaniel Lezcano <daniel.lezcano@free.fr>
Acked-by: default avatarDavid Howells <dhowells@redhat.com>
Cc: James Morris <jmorris@namei.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 39fd3393
...@@ -553,6 +553,8 @@ static inline kernel_cap_t cap_raise_nfsd_set(const kernel_cap_t a, ...@@ -553,6 +553,8 @@ static inline kernel_cap_t cap_raise_nfsd_set(const kernel_cap_t a,
*/ */
#define has_capability(t, cap) (security_real_capable((t), &init_user_ns, (cap)) == 0) #define has_capability(t, cap) (security_real_capable((t), &init_user_ns, (cap)) == 0)
#define has_ns_capability(t, ns, cap) (security_real_capable((t), (ns), (cap)) == 0)
/** /**
* has_capability_noaudit - Determine if a task has a superior capability available (unaudited) * has_capability_noaudit - Determine if a task has a superior capability available (unaudited)
* @t: The task in question * @t: The task in question
......
...@@ -134,21 +134,24 @@ int __ptrace_may_access(struct task_struct *task, unsigned int mode) ...@@ -134,21 +134,24 @@ int __ptrace_may_access(struct task_struct *task, unsigned int mode)
return 0; return 0;
rcu_read_lock(); rcu_read_lock();
tcred = __task_cred(task); tcred = __task_cred(task);
if ((cred->uid != tcred->euid || if (cred->user->user_ns == tcred->user->user_ns &&
cred->uid != tcred->suid || (cred->uid == tcred->euid &&
cred->uid != tcred->uid || cred->uid == tcred->suid &&
cred->gid != tcred->egid || cred->uid == tcred->uid &&
cred->gid != tcred->sgid || cred->gid == tcred->egid &&
cred->gid != tcred->gid) && cred->gid == tcred->sgid &&
!capable(CAP_SYS_PTRACE)) { cred->gid == tcred->gid))
rcu_read_unlock(); goto ok;
return -EPERM; if (ns_capable(tcred->user->user_ns, CAP_SYS_PTRACE))
} goto ok;
rcu_read_unlock();
return -EPERM;
ok:
rcu_read_unlock(); rcu_read_unlock();
smp_rmb(); smp_rmb();
if (task->mm) if (task->mm)
dumpable = get_dumpable(task->mm); dumpable = get_dumpable(task->mm);
if (!dumpable && !capable(CAP_SYS_PTRACE)) if (!dumpable && !task_ns_capable(task, CAP_SYS_PTRACE))
return -EPERM; return -EPERM;
return security_ptrace_access_check(task, mode); return security_ptrace_access_check(task, mode);
...@@ -198,7 +201,7 @@ static int ptrace_attach(struct task_struct *task) ...@@ -198,7 +201,7 @@ static int ptrace_attach(struct task_struct *task)
goto unlock_tasklist; goto unlock_tasklist;
task->ptrace = PT_PTRACED; task->ptrace = PT_PTRACED;
if (capable(CAP_SYS_PTRACE)) if (task_ns_capable(task, CAP_SYS_PTRACE))
task->ptrace |= PT_PTRACE_CAP; task->ptrace |= PT_PTRACE_CAP;
__ptrace_link(task, current); __ptrace_link(task, current);
......
...@@ -127,18 +127,30 @@ int cap_settime(const struct timespec *ts, const struct timezone *tz) ...@@ -127,18 +127,30 @@ int cap_settime(const struct timespec *ts, const struct timezone *tz)
* @child: The process to be accessed * @child: The process to be accessed
* @mode: The mode of attachment. * @mode: The mode of attachment.
* *
* If we are in the same or an ancestor user_ns and have all the target
* task's capabilities, then ptrace access is allowed.
* If we have the ptrace capability to the target user_ns, then ptrace
* access is allowed.
* Else denied.
*
* Determine whether a process may access another, returning 0 if permission * Determine whether a process may access another, returning 0 if permission
* granted, -ve if denied. * granted, -ve if denied.
*/ */
int cap_ptrace_access_check(struct task_struct *child, unsigned int mode) int cap_ptrace_access_check(struct task_struct *child, unsigned int mode)
{ {
int ret = 0; int ret = 0;
const struct cred *cred, *child_cred;
rcu_read_lock(); rcu_read_lock();
if (!cap_issubset(__task_cred(child)->cap_permitted, cred = current_cred();
current_cred()->cap_permitted) && child_cred = __task_cred(child);
!capable(CAP_SYS_PTRACE)) if (cred->user->user_ns == child_cred->user->user_ns &&
ret = -EPERM; cap_issubset(child_cred->cap_permitted, cred->cap_permitted))
goto out;
if (ns_capable(child_cred->user->user_ns, CAP_SYS_PTRACE))
goto out;
ret = -EPERM;
out:
rcu_read_unlock(); rcu_read_unlock();
return ret; return ret;
} }
...@@ -147,18 +159,30 @@ int cap_ptrace_access_check(struct task_struct *child, unsigned int mode) ...@@ -147,18 +159,30 @@ int cap_ptrace_access_check(struct task_struct *child, unsigned int mode)
* cap_ptrace_traceme - Determine whether another process may trace the current * cap_ptrace_traceme - Determine whether another process may trace the current
* @parent: The task proposed to be the tracer * @parent: The task proposed to be the tracer
* *
* If parent is in the same or an ancestor user_ns and has all current's
* capabilities, then ptrace access is allowed.
* If parent has the ptrace capability to current's user_ns, then ptrace
* access is allowed.
* Else denied.
*
* Determine whether the nominated task is permitted to trace the current * Determine whether the nominated task is permitted to trace the current
* process, returning 0 if permission is granted, -ve if denied. * process, returning 0 if permission is granted, -ve if denied.
*/ */
int cap_ptrace_traceme(struct task_struct *parent) int cap_ptrace_traceme(struct task_struct *parent)
{ {
int ret = 0; int ret = 0;
const struct cred *cred, *child_cred;
rcu_read_lock(); rcu_read_lock();
if (!cap_issubset(current_cred()->cap_permitted, cred = __task_cred(parent);
__task_cred(parent)->cap_permitted) && child_cred = current_cred();
!has_capability(parent, CAP_SYS_PTRACE)) if (cred->user->user_ns == child_cred->user->user_ns &&
ret = -EPERM; cap_issubset(child_cred->cap_permitted, cred->cap_permitted))
goto out;
if (has_ns_capability(parent, child_cred->user->user_ns, CAP_SYS_PTRACE))
goto out;
ret = -EPERM;
out:
rcu_read_unlock(); rcu_read_unlock();
return ret; return ret;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment