bugThe FreeType Project - Bugs: bug #28521, Problem with fix for bug #28226

 
 

bug #28521: Problem with fix for bug #28226

Submitted by:  Kenneth Sharp <kens>
Submitted on:  Fri 08 Jan 2010 10:52:14 AM UTC  
 
Severity: 3 - NormalItem Group: None
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.

 

Fri 08 Jan 2010 05:14:05 PM UTC, comment #1:

Applied, thanks.

Werner LEMBERG <wl>
Project AdministratorIn charge of this item.
Fri 08 Jan 2010 10:52:14 AM UTC, original submission:

Apologies for this one, its entirely my fault. Mea culpa etc.

Issue #28226 involved a work-around for a font which illegally used the setcurrentpoint operator, this operator is only supposed to be used with the result of OtherSubrs and the font used it directly. The patch I supplied removed the block of code which checked this usage entirely.

This turns out to be a Bad Thing. If the setcurrenpoint is being used correctly it should reset the flex flag in the decoder. If we don't do this then the flag never gets reset and we omit any further contours from the glyph (at least until we close the path or similar).

Oops.

The attached patch restores the code for resetting the flex flag while still not flagging an error for the invalid usage.

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 #19438:  t1decode.dif added by kens (951B - video/x-dv)

 

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 kens (Submitted the item)
  • -unavailable- added by kens
  •  

    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 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 08 Jan 2010 05:14:05 PM UTCwlStatusNone=>Fixed
      Assigned toNone=>wl
      Open/ClosedOpen=>Closed
      Planned ReleaseNone=>2.3.12
    Fri 08 Jan 2010 10:52:14 AM UTCkensAttached File-=>Added t1decode.dif, #19438
      Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1