bugThe FreeType Project - Bugs: bug #28226, incorrect use of setcurrentpoint

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #28226: incorrect use of setcurrentpoint

Submitter:  Kenneth Sharp <kens>
Submitted:  Tue 08 Dec 2009 11:16:30 AM UTC
   
 
Severity:  3 - Normal Item Group:  Hack required
Status:  Fixed Privacy:  Public
Assigned to:  wl Open/Closed:  Closed
Planned Release:  2.3.12

Mon 14 Dec 2009 09:34:32 PM UTC, comment #1: 

Applied (with slight modifications), thanks.

Werner LEMBERG <wl>
Group administrator
Tue 08 Dec 2009 11:16:30 AM UTC, original submission:  

Some fonts, apparently converted from TrueType by by an unknown utility, use the Type 1 operator 'setcurrentpoint' to set the initial point of the glyph path. This is incorrect (setcurrentpoint can only be used with results from OtherSubrs procedures) and unnecessary since the glyph also uses an hsbw instruction.

The code in t1_decoder_parse_charstrings, around line 1359 checks for this incorrect use of setcurrentpoint and flags an error.

However Ghostscript, the Jaws PostScript interpreter and Adobe Acrobat Distiller all silently ignore the error. Suggest modifying the code with a #ifdef to only flag the error under strict checking or similar.


Kenneth Sharp <kens>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #19225:  0001-Ignore-invalid-setcurrentpoint.patch added by rillian (2KiB - application/octet-stream - Proposed patch disabling the check. From ghostscript svn r10467.)
file #19222:  f1.out added by kens (4KiB - application/octet-stream - german.ps is the original file, f1.out is the t1disasm output of the first font)
file #19221:  german.ps added by kens (65KiB - application/postscript - german.ps is the original file, f1.out is the t1disasm output of the first font)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by wl (Posted a comment)
  • -email is unavailable- added by rillian (Updated the item)
  • -email is unavailable- added by kens (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-12-14 wl Planned ReleaseNone 2.3.12
        Open/ClosedOpen Closed
        Assigned toNone wl
        StatusNone Fixed
    2009-12-09 rillian Attached File- Added 0001-Ignore-invalid-setcurrentpoint.patch, #19225
    2009-12-08 kens Attached File- Added german.ps, #19221
        Attached File- Added f1.out, #19222

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code