Re: CVS commit: src/sys/net

[Andrew Doran <ad%netbsd.org@localhost>]

On Thu, Apr 03, 2008 at 07:19:33AM +0000, David Young wrote:
> 
> Module Name:  src
> Committed By: dyoung
> Date:         Thu Apr  3 07:19:33 UTC 2008
> 
> Modified Files:
>       src/sys/net: if_gre.c
> 
> Log Message:
> Fix one of two bugs introduced by the descriptor handling changes
> (rev 1.125): correct the check for fd_getsock() failure in
> gre_socreate().
> 
> The second bug is more complicated to fix.  Since rev 1.125,
> gre_reconf() is using the file descriptor table of the current
> process instead of the process 0's (the kernel's).

I think there was already confusion about which descriptor was being used, I
meant to file a PR about that but forgot. Maybe the easiest way to solve
this is to have the descriptor modifying operations always run in proc0's
context. I'll look again.

Andrew