nhyatt/putty-source
1
0
Fork 0
mirror of https://git.tartarus.org/simon/putty.git synced 2025-02-04 06:02:24 +00:00
Code Issues Projects Releases Wiki Activity

Don't try sending on sharing channels.

Browse Source 
The final main loop in do_ssh2_authconn will sometimes loop over all
currently open channels calling ssh2_try_send_and_unthrottle. If the
channel is a sharing one, however, that will reference fields of the
channel structure like 'remwindow', which were never initialised in
the first place (thanks, valgrind). Fix by excluding CHAN_SHARING
channels from that loop.

(cherry picked from commit 7366fde1d4)
This commit is contained in:
Simon Tatham 2015-06-07 21:14:09 +01:00
parent ae93b52a9c
commit d75d136c68
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ssh.c
View File

@ -10325,7 +10325,8 @@ static void do_ssh2_authconn(Ssh ssh, unsigned char *in, int inlen,
* Try to send data on all channels if we can. * Try to send data on all channels if we can.
*/ */
for (i = 0; NULL != (c = index234(ssh->channels, i)); i++) for (i = 0; NULL != (c = index234(ssh->channels, i)); i++)
ssh2_try_send_and_unthrottle(ssh, c); if (c->type != CHAN_SHARING)
ssh2_try_send_and_unthrottle(ssh, c);
} }
} }