Re: [PATCH] new setprocuid syscall

BERECZ Szabolcs (szabi@inf.elte.hu)
Tue, 20 Feb 2001 13:14:46 +0100 (CET)


On Mon, 19 Feb 2001, Peter Samuelson wrote:

> [BERECZ Szabolcs]
> > + p = find_task_by_pid(pid);
> > + p->fsuid = p->euid = p->suid = p->uid = uid;
> Race -- you need to make sure the task_struct doesn't disappear out
> from under you.

Yes, but we need a write_lock, not a read_lock.

> Anyway, why not use the interface 'chown uid /proc/pid'? No new
> syscall, no arch-dependent part, no user-space tool, etc.

We need a userspace tool, because we must check if the user, who want to
change the uid, knows the other user's passwd.
Or what if user1 want to change user2's process to user3 uid?

Bye,
Szabolcs

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/