Re: adopt(pid_t pid) syscall proposal [patch included]

From: Andy Lutomirski
Date: Tue Jun 11 2013 - 12:53:40 EST

On 06/10/2013 06:23 PM, vcaputo@xxxxxxxxxxxxxxxxx wrote:
+ if (!uid_eq(cred->euid, tcred->suid) &&
+ !uid_eq(cred->euid, tcred->uid) &&
+ !uid_eq(cred->uid, tcred->suid) &&
+ !uid_eq(cred->uid, tcred->uid) &&
+ !ns_capable(cred->user_ns, CAP_KILL)) {
+ ret = -EPERM;
+ goto out_unlock;
+ }

That check's far too permissive.

This sounds like it will break anything that uses wait and expects its children to not be stolen out from under it.

Also, you'll have problems with screen -x or the default tmux shareable configuration. It sounds like this is better done in userspace.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at