Florent Parent writes:
> This is what I did to make it work for me. A better fix would probably be 
> around the struct proc definition.  If fact, you had noted "broken" 
> probably as a memo to fix something here...
> 
>       struct proc *p = curproc ? curproc : &proc0;    /* XXX broken */
> 
> 
> *** ng_ksocket.c.orig   Sat Jan 19 11:05:28 2002
> --- ng_ksocket.c        Sat Jan 19 11:45:23 2002
> ***************
> *** 759,765 ****
>                         sopt.sopt_name = ksopt->name;
>                         sopt.sopt_val = ksopt->value;
>                         sopt.sopt_valsize = valsize;
> !                       sopt.sopt_p = p;
>                         error = sosetopt(so, &sopt);
>                         break;
>                     }
> --- 759,765 ----
>                         sopt.sopt_name = ksopt->name;
>                         sopt.sopt_val = ksopt->value;
>                         sopt.sopt_valsize = valsize;
> !                       sopt.sopt_p = 0;
>                         error = sosetopt(so, &sopt);
>                         break;
>                     }

Wow, didn't know you could do that :-)

My understanding of the use of that parameter is very incomplete.
I just copied what was found in the NFS code at the time (since
it also makes use of a socket in the kernel).

So this 'struct proc' argument can be NULL now? What about when
calling other socket functions like socreate(), et. al.? 
If so, your fix looks like the right onw.

-Archie

__________________________________________________________________________
Archie Cobbs     *     Packet Design     *     http://www.packetdesign.com

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-net" in the body of the message

Reply via email to