On Sat, 2005-28-05 at 03:28 +0200, Thomas Graf wrote:
> * 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 did check netfilter, net/sched/ tcp_diag, xfrm and they do things
> > > 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.
Hrm - you should probably have said that in the patch text ;->
I did test the one i sent - basic things and by code inspection saw
little risk as well. Perhaps we should push mine then you can add on top
> > 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
> 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.
I dont see any issues with my original patch or for that matter yours
except for now that you are saying you havent compiled it even ;->.
All dumps MUST set NLM_F_MULTI - thats the main change really.
All non-dump paths may set NLM_F_MULTI, and from that angle, if theres
any bugs in the old code they are maintained ;->
So how about we push my original patch instead?