openssh unix dev - May 2003 - [Bug 556] TCP_NODELAY not set completely for port forwarding

http://bugzilla.mindrot.org/show_bug.cgi?id=556

           Summary: TCP_NODELAY not set completely for port forwarding
           Product: Portable OpenSSH
           Version: -current
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: sshd
        AssignedTo: openssh-unix-dev at mindrot.org
        ReportedBy: briang at oasisadvancedengineering.com


When port forwarding is set up, TCP_NODELAY is set on the ports, in order to
prevent buffering.  This flag is not set in the actual SSH connection.  This
causes data that flows from the server to the client to be buffered, causing a
bursing effect.

The solution is to add set_nodelay(newsock) after the accept call in the main
function.  This solves the problem, but requires TCP_NODELAY to be set on all
connections, regardless of port forwards.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

http://bugzilla.mindrot.org/show_bug.cgi?id=556

briang at oasisadvancedengineering.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED





------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

http://bugzilla.mindrot.org/show_bug.cgi?id=556





------- Additional Comments From briang at oasisadvancedengineering.com 
2003-05-14 22:37 -------
Created an attachment (id=294)
 --> (http://bugzilla.mindrot.org/attachment.cgi?id=294&action=view)
Patch to add the TCP_NODELAY flag after the connection acceptance




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

http://bugzilla.mindrot.org/show_bug.cgi?id=556

briang at oasisadvancedengineering.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From briang at oasisadvancedengineering.com 
2003-05-14 22:38 -------
See patch.  Very simple fix



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

http://bugzilla.mindrot.org/show_bug.cgi?id=556

djm at mindrot.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |



------- Additional Comments From djm at mindrot.org  2003-05-14 22:49 -------
the bug isn't fixed until the patch is accepted and applied in out tree.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.