* jamal <1117242749.6251.15.camel@xxxxxxxxxxxxxxxxxxxxx> 2005-05-27 21:12
> > Yet unconverted
> > are: netfilter, net/sched/ tcp_diag, xfrm, kernel/audit.
> >
>
> I didnt find any issues with these as far as PID extrapolation.
I should have written, yet unchecked.
> > I also fixed some uses of NL_CREDS() to fetch the pid for
> > message generation which is wrong. Various dumping procedures
> > did not set the multi flag at all, I fixed all of them.
> >
> > Do not apply these patches yet, they're way too intrusive
>
> Do you mean the two patches? Those looked fine to me;
Yes, those two patches. My part comes directly out of my fingers
and is only compile tested. I didn't even run them yet.
> i.e its mostly the patch i posted this morning with the new Macros, no?
I went through all users of NLMSG_PUT in the modules you touched and
checked if they properly set NLM_F_MULTI and used the new macro when
appropriate.
Anyways, I want to be careful with this, there might be userspace
apps which rely on pid being filled in from siocb->scm->creds or
userspace apps which don't expect the NLM_F_MULTI flag even when
they should so we should check with all parties involved.
|