Bug 541

Hi,

In OpenSSH 3.6.1p1, the function packet_set_interactive() in packet.c appears 
to have changed from 3.5p1.  It appears there may be a typographical error.

Specifically, 3.6.1p1 shows:

        /* Only set socket options if using a socket.  */
        if (!packet_connection_is_on_socket())
        if (interactive)
                set_nodelay(connection_in);

when it is probably supposed to say:

        /* Only set socket options if using a socket.  */
        if (!packet_connection_is_on_socket())
           return;
 
        if (interactive)
                set_nodelay(connection_in);

In other words, it appears somebody accidentily deleted the "return" statement 
that was present (and functional) in 3.5p1.  As such, the set_nodelay() only 
gets executed in 3.6.1p1 if the connection is NOT on a socket, which is likely 
not how it was meant to be written.