bugmake - Bugs: bug #37878, index out of bounds when handling...

 
 

bug #37878: index out of bounds when handling a file contains '(' and not ends with ')'

Submitter:  None
Submitted:  Fri 07 Dec 2012 02:34:54 AM UTC
   
 
Severity:  3 - Normal Item Group:  Bug
Status:  Duplicate Privacy:  Public
Assigned to:  None Open/Closed:  Closed
Component Version:  3.82 Operating System:  POSIX-Based
Fixed Release:  None Triage Status:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 13 Jan 2013 05:41:24 PM UTC, comment #2: 

This bug has already been fixed; the fix will be available in the next release of GNU make.

Paul D. Smith <psmith>
Group administrator
Fri 07 Dec 2012 03:21:37 AM UTC, comment #1: 


read.c:3023
if (! (flags & PARSEFS_NOAR)
    && tp == tmpbuf && tp[0] != '(' && tp[nlen-1] != ')')
                                                  ^^
                                           should be '=='


junhee cho <zlqhem>
Fri 07 Dec 2012 02:34:54 AM UTC, original submission:  

-----------------------------------------------------------------
read.c:3023

if (! (flags & PARSEFS_NOAR)
    && tp == tmpbuf && tp[0] != '(' && tp[nlen=1] != ')')
                                                  ^^
                                                  should be '=='
-----------------------------------------------------------------

It occurs indexing out of bounds when handling the following prerequisite detected. Above if condition becomes true
even if the file is not a form of archive member reference.


target: this_is_a_target_not_archive_memeber_reference(blah).jpeg
    ..recipe..


read.c: 3034
    e = next_token (e);  // At here, index out of bounds occurs

I've used electric fence (http://perens.com/FreeSoftware/ElectricFence/) for detecting this.









Anonymous

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by psmith (Posted a comment)
  • -email is unavailable- added by zlqhem (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-01-13 psmith StatusNone Duplicate
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code