OhmNomNom / thyme

A fork of mintty, for the modern world
GNU General Public License v3.0
0 stars 0 forks source link

Crash when looking at UTF-8 file #250

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Many thanks for your great work with mintty.
It's a really nice and useful tool.

I just ran into a problem with it crashing/hanging when looking at UTF-8 files. 
Details below.

What steps will reproduce the problem?

1. set LANG=en_US.UTF-8

LANG=en_US.UTF-8
export LANG

2. Get a specific UTF-8 demo file  (can download using link, also attached)

wget http://www.cl.cam.ac.uk/~mgk25/ucs/examples/UTF-8-demo.txt

3. View the file

less UTF-8-demo.txt

4. Scroll down, pressing the "down arrow" key (leave it pressed). It 
crashes/hangs when reaching 60% of the file.

What is the expected output? What do you see instead?

Should be able to see the whole file. Instead, it crashes/hangs the window. 

What versions of mintty, Cygwin/MSYS, and Windows are you using?

mintty 0.9.5
CYGWIN_NT-6.1-WOW64 Vortex 1.7.7(0.230/5/3) 2010-08-31 09:58 i686 Cygwin
Windows 7 Enterprise 64-bit.

Please provide any additional information below.

Original issue reported on code.google.com by pabluc...@gmail.com on 14 Feb 2011 at 4:30

Attachments:

GoogleCodeExporter commented 9 years ago
Hi, stumbled upon this same error.

mintty crashes on 'curl -s http://dv.wikipedia.org/wiki/Tom_and_Jerry'.

I was also able to narrow down the problem a little to:
echo -n It6K3qbeh96o3o3esCDemt6o3pTep96N3qoiLCAiOCI6ICLeid6p3pHeqN6H3qfeiN6o\
3obeqSIsICI5IjogIt6J3qnekd6o3pTep96I3qjeht6oIN6a3qjelN6n3o3eqiIsIA==|
base64 -d # and then manually scrolling the window

Same mintty/cygwin versions here.
Confirmed on both, 32 and 64 bit Windows 7 Professional

Original comment by fbb.foo...@gmail.com on 14 Feb 2011 at 11:07

GoogleCodeExporter commented 9 years ago
Thanks very much for the detailed reports. Going back through revisions showed 
that I introduced this bug when fixing a problem with combining characters in 
r1078 for 0.9.3.

Original comment by andy.koppe on 14 Feb 2011 at 12:35

GoogleCodeExporter commented 9 years ago
Fixed in r1124 on trunk.

Original comment by andy.koppe on 15 Feb 2011 at 1:06

GoogleCodeExporter commented 9 years ago

Original comment by andy.koppe on 20 Feb 2011 at 5:51