bugThe FreeType Project - Bugs: bug #49711, Missing FT_MEM_ZERO in...

 
 

bug #49711: Missing FT_MEM_ZERO in smooth/ftgrays.c causes conditional jump to depend on uninitialized value

Submitted by:  Ulrik Sandborg-Petersen <ulrikp>
Submitted on:  Sat 26 Nov 2016 10:00:18 PM UTC  
 
Severity: 3 - NormalItem Group: Incorrect behaviour
Status: FixedPrivacy: Public
Assigned to: Werner LEMBERG <wl>Open/Closed: Closed
Planned Release: 2.7.1

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Tue 29 Nov 2016 05:46:12 PM UTC, comment #3:

Ah, ok. Reverted.

Werner LEMBERG <wl>
Project AdministratorIn charge of this item.
Tue 29 Nov 2016 03:25:16 PM UTC, comment #2:

This was fixed already! The HEAD did not have this problem in spite of what seemed to the reporter. Please revert.

Alexei Podtelezhnikov <podtelez>
Project Member
Tue 29 Nov 2016 11:10:51 AM UTC, comment #1:

Fixed, thanks!

Werner LEMBERG <wl>
Project AdministratorIn charge of this item.
Sat 26 Nov 2016 10:00:18 PM UTC, original submission:

In a test program which I've written, valgrind(1) reports that a conditional jump depends on an unitialized value in smooth/ftgrays.c on line 587.

It turns out it is ras.ex and/or ras.ey that is not initialized.

This has been tested with version 2.7, and it seems to persist in the latest git sources.

My quick fix is to add this on line 1868:

FT_MEM_ZERO(worker, sizeof(gray_TWorker));

I am not sure this is the right way to do it, but it does silence valgrind.

Thanks for a great library!

Best wishes,

Dr. Ulrik Sandborg-Petersen
Scripture Systems ApS, Denmark

Ulrik Sandborg-Petersen <ulrikp>

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by podtelez (Posted a comment)
  • -unavailable- added by wl (Posted a comment)
  • -unavailable- added by ulrikp (Submitted the item)
  • -unavailable- added by ulrikp
  •  

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

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 29 Nov 2016 11:10:51 AM UTCwlStatusNone=>Fixed
      Assigned toNone=>wl
      Open/ClosedOpen=>Closed
      Planned ReleaseNone=>2.7.1
    Sat 26 Nov 2016 10:00:18 PM UTCulrikpCarbon-Copy-=>Added ulrikp

    Back to the top


    Powered by Savane 3.1-cleanup1