bugUnifont - Bugs: bug #45126, unifontpic.c:129: array index use...

 
 

bug #45126: unifontpic.c:129: array index use before limits check ?

Submitted by:  David Binderman <dcb314>
Submitted on:  Mon 18 May 2015 10:27:06 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Paul Hardy <unifoundry>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Mon 29 Jun 2015 05:28:24 AM UTC, comment #2:

The Unifont 8.0.01 package now contains this change.

Paul Hardy <unifoundry>
Project AdministratorIn charge of this item.
Fri 22 May 2015 04:13:32 PM UTC, comment #1:

Thank you for taking the time to look at the code. I'll make this change in the next release. In general, the programs in this package don't do rigorous checking of input. Some of the man pages warn about that. But this is an easy change to make.

Paul Hardy <unifoundry>
Project AdministratorIn charge of this item.
Mon 18 May 2015 10:27:06 AM UTC, original submission:

[unifontpic.c:129]: (style) Array index 'i' is used before limits check.

for (i = 0; (instring[i] != ':') && (i < 9); i++); /* find the colon separator */

Suggest limit check the array index before use.

David Binderman <dcb314>

 

(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 unifoundry (Posted a comment)
  • -unavailable- added by dcb314 (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 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 20 Jul 2015 12:54:21 AM UTCunifoundryOpen/ClosedOpen=>Closed
    Mon 29 Jun 2015 05:28:23 AM UTCunifoundryStatusIn Progress=>Fixed
    Fri 22 May 2015 04:13:32 PM UTCunifoundryStatusNone=>In Progress
      Assigned toNone=>unifoundry

    Back to the top


    Powered by Savane 3.1-cleanup1