lkml.org 
[lkml]   [2005]   [Nov]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] kill_proc_info_as_uid: don't use hardcoded constants
On Wed, Nov 23, 2005 at 05:24:05PM +0300, Oleg Nesterov wrote:
> Use symbolic names instead of hardcoded constants.

I'm more than happy with that. Seems like the original code in
check_kill_permission() was changed in a similar way after I made that
_as_uid() function.

I don't think that I am a person with any say in the signal.c code,but
in case of doubt:

Acked-by: Harald Welte <laforge@gnumonks.org>

> Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
>
> --- 2.6.15-rc2/kernel/signal.c~1_KUID 2005-11-22 19:35:52.000000000 +0300
> +++ 2.6.15-rc2/kernel/signal.c 2005-11-23 19:17:35.000000000 +0300
> @@ -1163,8 +1163,7 @@ int kill_proc_info_as_uid(int sig, struc
> ret = -ESRCH;
> goto out_unlock;
> }
> - if ((!info || ((unsigned long)info != 1 &&
> - (unsigned long)info != 2 && SI_FROMUSER(info)))
> + if ((info == SEND_SIG_NOINFO || (!is_si_special(info) && SI_FROMUSER(info)))
> && (euid != p->suid) && (euid != p->uid)
> && (uid != p->suid) && (uid != p->uid)) {
> ret = -EPERM;

--
- Harald Welte <laforge@gnumonks.org> http://gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2005-11-24 21:53    [W:0.030 / U:0.684 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site