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

 
 

bug #28226: incorrect use of setcurrentpoint

Submitted by:  Kenneth Sharp <kens>
Submitted on:  Tue 08 Dec 2009 11:16:30 AM UTC  
 
Severity: 3 - NormalItem Group: Hack required
Status: FixedPrivacy: Public
Assigned to: Werner LEMBERG <wl>Open/Closed: Closed
Planned Release: 2.3.12

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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

Applied (with slight modifications), thanks.

Werner LEMBERG <wl>
Project AdministratorIn charge of this item.
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>
Project Member

 

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

Attach File(s):
   
   
Comment:
   

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
  • -unavailable- added by wl (Posted a comment)
  • -unavailable- added by rillian (Updated the item)
  • -unavailable- added by kens (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 14 Dec 2009 09:34:32 PM UTCwlPlanned ReleaseNone=>2.3.12
      Open/ClosedOpen=>Closed
      Assigned toNone=>wl
      StatusNone=>Fixed
    Wed 09 Dec 2009 08:24:03 AM UTCrillianAttached File-=>Added 0001-Ignore-invalid-setcurrentpoint.patch, #19225
    Tue 08 Dec 2009 11:16:30 AM UTCkensAttached File-=>Added german.ps, #19221
      Attached File-=>Added f1.out, #19222

    Back to the top


    Powered by Savane 3.1-cleanup1