Subject: Re: crcmp(), once again
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: Roland Illig <rillig@NetBSD.org>
List: tech-kern
Date: 11/17/2005 23:40:54
YAMAMOTO Takashi wrote:
>>   - It is unclear why it is necessary at all.
>>     http://mail-index.netbsd.org/tech-kern/2005/11/04/0005.html
> 
> 
> why unclear, while you know where it's used?

Because I don't know why the whole set of credentials needs to be 
compared. Until now, no one has answered this question.

>>   - It is the only reason for sorting group lists
>>     when they enter the kernel via setgroups(),
>>     which itself is considered unwanted.
>>     http://mail-index.netbsd.org/tech-kern/2005/11/03/0021.html
> 
> 
> i don't think the result of setgroups is fed into crcmp.
> the change didn't fix anything and should be backed out, IMO.

See the ChangeLog for src/sys/kern/kern_prot.c.

>>   - It has a big FIXME comment.
>>     src/sys/kern/kern_prot.c
> 
> i don't see why it needs to be fixed.
> credentials with the different order of groups are different.

Please explain why the order of groups is important. I don't see it.

Roland