DynamicWindow Patch
    Markus Friedl 
    markus at openbsd.org
       
    Thu Jul  8 17:55:05 EST 2004
    
    
  
On Wed, Jul 07, 2004 at 11:44:53PM -0400, Chris Rapier wrote:
> Michael Stevens wrote:
> >>>-     if (len > 0x100000)
> >>>+     if (!buffer->unlimited && len > 0x100000)
> >>>             fatal("buffer_append_space: len %u not supported", len);
> >>
> >>that's dangerous, is it really necessary? we could
> >>crank the 0x100000.
> >
> >
> >I'm not sure I see where the danger is coming from besides memory
> >consumption when the  user locally sets a large window. The problem I
> >see with that is now instead of unlimited, you can only go up to that
> >static size. The unlimited flag is only set for the buffers in/out
> >buffer on the channel.
> 
> Something to keep in mind is that uping any static limit on the buffer 
> size only pushes the problem off on to people down the line. The fact of 
still, how does this matter if the tcp send and receive space are big enough?
    
    
More information about the openssh-unix-dev
mailing list