[liblouis-liblouisxml] Issue 11 in liblouis: lou_backTranslate() may leave some "outpos" elements un-initialized

  • From: liblouis@xxxxxxxxxxxxxx
  • To: liblouis-liblouisxml@xxxxxxxxxxxxx
  • Date: Fri, 07 May 2010 16:48:20 +0000

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 11 by timothy.ty.lee: lou_backTranslate() may leave some "outpos" elements un-initialized
http://code.google.com/p/liblouis/issues/detail?id=11

To reproduce the problem:
1. Before calling lou_backTranslate(), fill outpos with -1's
2. Back-translate ",numb} #c = ,,y6" using en-us-g2.ctb

When you examine the values returned in outpos, it can be seen that some of
the -1 were not overwritten by the function.  What should happen instead is
that outpos[0..inlen] should be set by lou_backTranslate().

This problem is present in the trunk, and affects all operating systems.

The attached patch aganist trunk overcomes the problem by filling outpos[]
with -1's to mark array as uninitialized when lou_backTranslate() starts.
Before returning to caller, the function examines outpos[] and fills in
unused elements in the array.

Attachments:
        backtranslate_outpos.patch  856 bytes

For a description of the software and to download it go to
http://www.jjb-software.com

Other related posts: