Issue 123501

Summary: EDITING: Crashes on typing Miao chars
Product: General Reporter: nrs <nrsi.cheuk>
Component: uiAssignee: hdu <hdu>
Status: CLOSED FIXED QA Contact: Clarence GUO <clarence.guo.bj>
Severity: Blocker    
Priority: P2 CC: clarence.guo.bj, fanyuzhen, gwilbor, hdu, issues, steve.yin.aoo
Version: 4.0.1   
Target Milestone: 4.1.0   
Hardware: PC   
OS: Windows 7   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on:    
Issue Blocks: 124312    
Attachments:
Description Flags
Ahmao Unicode keyboard + layout none

Description nrs 2013-10-20 05:24:07 UTC 
Created attachment 81785 [details]
Ahmao Unicode keyboard + layout

After a Miao char (on supplementary plane) is typed into Writer/Calc/Impress, subsequent char input crashes it.  This problem does not occur in other apps like LO Writer/Calc/Impress or MS Word/Excel/Powerpoint.

This problem is attested on the following build & OS:
- AOO 4.0.1 on 64-bit English Windows 7 Ultimate SP1

Steps to reproduce the bug:
1. download & install Miao Unicode font at https://github.com/phjamr/MiaoUnicode/blob/master/MiaoUnicode-Regular.ttf?raw=true
2. unzip attached archive & install Ahmao keyboard (run setup.exe)
3. install AOO & run Writer/Calc/Impress
4. select Miao Unicode font
5. on Windows language bar, select MR
6. refer to the 2 .jpg files in attached archive for keys mapped in each state
7. type any Miao (non-ASCII) char mapped to any key in any state
8. w/o breaking the line, type any other chars (incl. ASCII) --> CRASH!!!

Control experiments with other apps:
9. run LO Writer/Calc/Impress or MS Word/Excel/PowerPoint
10. repeat #4-7
11. w/o breaking the line, type any other chars (incl. ASCII): chars appear as expected*

*NB: LO supports Graphite, so small letters at different tone positions can be rendered correctly (on the top/top-right/right of the big letters).  MSO, however, does not support Graphite, so small & big letters are always linear & the trailing invisible tone letters rendered graphically.

As the Miao script is used by multiple language groups with as many as 6 million in population, pls. kindly fix this ASAP.  Thank you!
Comment 1 Giacomo Boschi 2013-10-31 09:25:24 UTC 
I encountered a similar bug: when trying to use Graphite to type in tengwar characters, the program crashes everytime I try to type a double accent/curl.

This problem is attested on the following build & OS:
- AOO 4.0.1 on 64-bit Italian Windows 7 Ultimate SP1

Steps to reproduce the bug:
1. download & install Tengwar Telcontar font at http://freetengwar.sourceforge.net/tengtelc.html
2. download & install the Tengwar keyboard Layout http://freetengwar.sourceforge.net/keylayouts.html
3. install AOO & run Writer
4. select Tengwar Telcontar font
5. on Windows language bar, select EN and then the "Tengwar unicode (QWERTY)" layout
6. Try to type a syllabe with double accent (for example, press the "K" "E" "E" keys.)
7. w/o breaking the line, type any other chars (incl. ASCII)

AOO will crash systematically at step 7.
Comment 2 Rob Weir 2013-11-27 16:25:50 UTC 
Marking as confirmed due to the well-written and detailed steps reported by users.

Raising to P2.
Comment 3 SVN Robot 2014-02-21 10:32:36 UTC 
"hdu" committed SVN revision 1570513 into trunk:
#i123501# fix invalid iterators in graphite
Comment 4 hdu@apache.org 2014-02-21 10:35:30 UTC 
Kudos for the great bug-reports. The problem is fixed with the commit above. Please check the next nightly build at http://ci.apache.org/projects/openoffice/
Comment 5 nrs 2014-02-25 10:29:30 UTC 
Thanks for fixing the crash.  Now it reveals another bug, which was just now filed as #124312.  Pls. kindly fix that one as well at your earliest convenience.  Thanks!
Comment 6 Clarence GUO 2014-02-26 05:34:47 UTC 
Verified on AOO Rev 1566593 en-US, after step 5, input some unicode strings, still crash.

Test ENV: Win7 professional 64 bit SP1, Chinese version.
Comment 7 nrs 2014-02-26 06:20:47 UTC 
@Clarence: Thanks for testing, but I believe you need Rev. 1570513 or later to see the fix.
Comment 8 hdu@apache.org 2014-02-27 07:41:12 UTC 
(In reply to Clarence GUO from comment #6)
> Verified on AOO Rev 1566593 en-US, after step 5, input some unicode strings,
> still crash.

Can you provide some of these strings that make it crash?
Comment 9 Clarence GUO 2014-02-27 09:26:51 UTC 
@yutgor, OK, then I tested on Rev 1571942 from Buildbots, the defect was fixed
Comment 10 Steve Yin 2014-04-03 08:32:34 UTC 
Verified on branch AOO410. Rev. 1583666