Re: Draft comments...

[nisse%lysator.liu.se@localhost (Niels Möller)]

"Joseph Galbraith" <galb-list%vandyke.com@localhost> writes:

>   3. Specify that SSH_MSG_REQUEST_SUCCESS has the format:
> 
>      byte SSH_MSG_REQUEST_SUCCESS
>      ... response specific data
> 
>      where the response specific data is usually non-existent.
> 
>      And specify for "tcpip-forward" request, that
>      if and only if the request port is 0, the
>      response look like:
> 
>      byte   SSH_MSG_REQUEST_SUCCESS
>      uint32 port that was bound on the server
> 
>   I would prefer 3; Markus has expressed a strong
>   preference for 1.
> 
>   If anyone else has an opinion on the matter,
>   can you please speak up?

The change you propose is small, clean, and backwards-compatible for
all cases I care about. So it looks good to me.

I think larger changes should wait until we want to bump the protocol
version number.

/Niels