Messages in this thread | | | From | Paul Mackerras <> | Date | Sat, 7 Jun 2003 21:42:41 +1000 | Subject | Re: [Patch] 2.5.70-bk11 zlib merge #4 pure magic |
| |
Jörn Engel writes:
> On Sat, 7 June 2003 19:40:25 +1000, Paul Mackerras wrote: > > > > Your change won't affect PPP, since pppd already refuses to use > > windowBits == 8 (as a workaround for this bug). > > Seems like I have misread the ppp code then. In that case, please > remove the ppp part from the previous patch or use this one instead, > Linus.
I meant that pppd (i.e. the user-level part of PPP) would refuse to negotiate windowBits == 8 with the peer, so from that point of view it doesn't matter if the kernel driver accepts it or not, since it will never be asked to accept it (by pppd). It is better on the whole if the kernel driver doesn't accept it since that is one less exploitable thing in the kernel (although you would have to be root to exploit it). But if the zlib code also refuses to use windowBits == 8, it then doesn't matter whether the ppp_deflate code accepts it, from either point of view.
On the whole I would say that the change to ppp-comp.h should go in, for now at least.
Paul. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |