bugThe FreeType Project - Bugs: bug #50191, uninitialised memory used in...

 
 

bug #50191: uninitialised memory used in af_cjk_metrics_check_digits()

Submitted by:  Tyson Smith <twsmith>
Submitted on:  Tue 31 Jan 2017 12:18:53 AM UTC  
 
Severity: 3 - NormalItem Group: Incorrect behaviour
Status: FixedPrivacy: Public
Assigned to: Werner LEMBERG <wl>Open/Closed: Closed
Planned Release: 2.8

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Tue 31 Jan 2017 06:54:56 PM UTC, comment #2:

Verified. I can no longer reproduce this issue. Thanks!

Tyson Smith <twsmith>
Tue 31 Jan 2017 07:10:22 AM UTC, comment #1:

Thanks for the report. Should be fixed now in git, please test.

Werner LEMBERG <wl>
Project AdministratorIn charge of this item.
Tue 31 Jan 2017 12:18:53 AM UTC, original submission:

Found using ftfuzzer.

Conditional jump or move depends on uninitialised value(s)
at 0x49EC2C: af_cjk_metrics_check_digits (afcjk.c:599)
by 0x4933DD: af_cjk_metrics_init (afcjk.c:633)
by 0x495AF5: af_face_globals_get_metrics (afglobal.c:473)
by 0x495AF5: af_loader_load_glyph (afloader.c:276)
by 0x495AF5: af_autofitter_load_glyph (afmodule.c:550)
by 0x403DD7: FT_Load_Glyph (ftobjs.c:746)
by 0x401E74: LLVMFuzzerTestOneInput (ftfuzzer.cc:264)
by 0x402280: main (ftfuzzer.cc:313)
Uninitialised value was created by a stack allocation
at 0x49EAF0: af_cjk_metrics_check_digits (afcjk.c:564)

Conditional jump or move depends on uninitialised value(s)
at 0x48FA94: af_latin_metrics_check_digits (aflatin.c:977)
by 0x48FA94: af_latin_metrics_init (aflatin.c:1011)
by 0x495AF5: af_face_globals_get_metrics (afglobal.c:473)
by 0x495AF5: af_loader_load_glyph (afloader.c:276)
by 0x495AF5: af_autofitter_load_glyph (afmodule.c:550)
by 0x403DD7: FT_Load_Glyph (ftobjs.c:746)
by 0x401E74: LLVMFuzzerTestOneInput (ftfuzzer.cc:264)
by 0x402280: main (ftfuzzer.cc:313)
Uninitialised value was created by a stack allocation
at 0x48D0FA: af_latin_metrics_init (aflatin.c:1001)

Conditional jump or move depends on uninitialised value(s)
at 0x49EC2C: af_cjk_metrics_check_digits (afcjk.c:599)
by 0x494D64: af_indic_metrics_init (afindic.c:54)
by 0x495AF5: af_face_globals_get_metrics (afglobal.c:473)
by 0x495AF5: af_loader_load_glyph (afloader.c:276)
by 0x495AF5: af_autofitter_load_glyph (afmodule.c:550)
by 0x403DD7: FT_Load_Glyph (ftobjs.c:746)
by 0x401E74: LLVMFuzzerTestOneInput (ftfuzzer.cc:264)
by 0x402280: main (ftfuzzer.cc:313)
Uninitialised value was created by a stack allocation
at 0x49EAF0: af_cjk_metrics_check_digits (afcjk.c:564)

Tyson Smith <twsmith>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #39622:  test_case.ttf added by twsmith (408KiB - application/octet-stream)

 

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 twsmith (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 20 Feb 2017 09:05:36 AM UTCwlPlanned Release2.7.1=>2.8
    Tue 31 Jan 2017 07:10:22 AM UTCwlStatusNone=>Fixed
      PrivacyPrivate=>Public
      Assigned toNone=>wl
      Open/ClosedOpen=>Closed
      Planned ReleaseNone=>2.7.1
    Tue 31 Jan 2017 12:18:54 AM UTCtwsmithAttached File-=>Added test_case.ttf, #39622

    Back to the top


    Powered by Savane 3.1-cleanup1