]> asedeno.scripts.mit.edu Git - PuTTY.git/commit
Fix assertion failure in xterm mouse tracking.
authorSimon Tatham <anakin@pobox.com>
Wed, 2 Sep 2015 17:56:20 +0000 (18:56 +0100)
committerSimon Tatham <anakin@pobox.com>
Wed, 2 Sep 2015 17:56:20 +0000 (18:56 +0100)
commit0de1ac95e5fe91e71061120de314b9bd852c2303
treeb8ae522420365baeaca6b95662648792919117e8
parent7524da621b1689b3384020cd6d83c990ef86bfa1
Fix assertion failure in xterm mouse tracking.

The original version of the xterm mouse tracking protocol did not
support character-cell coordinates greater than 223. If term_mouse()
got one, it would fail to construct an escape sequence for the mouse
event, and would then call ldisc_send() with a zero-length string -
which fails an assertion that I added in November (c269dd0135) on the
occasion of moving ldisc_echoedit_update() into its own function. So
the corresponding operation before that change would have done a
gratuitous ldisc_echoedit_update(), which is exactly the sort of thing
the assertion was there to catch :-)

Later extensions to the mouse tracking protocol support larger
coordinates anyway (try ESC[?1006h or ESC[?1015h in addition to the
ESC[?1000h that turns the whole system on in the first place). It's
only clients that don't use one of those extensions which would have
had the problem.

Thanks to Mirko Wolle for the report.
terminal.c