lkml.org 
[lkml]   [2003]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [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/

\
 
 \ /
  Last update: 2005-03-22 13:36    [W:1.204 / U:1.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site