bugicoutils - Bugs: bug #52313, out of bounds read access in...

 
 

bug #52313: out of bounds read access in simple_vec / icotool -l

Submitter:  Hanno Boeck <hanno>
Submitted:  Tue 31 Oct 2017 09:38:14 AM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 31 Oct 2017 04:17:08 PM UTC, comment #1: 

Fixed in 15ef8e8c9994981d25d623602ae1f329b363c20f/release 0.32.1.

F R <res>
Group administrator
Tue 31 Oct 2017 09:38:14 AM UTC, original submission:  

The attached file will cause an out of bounds read access when passed to icotool -l. This can be detected with address sanitizer. In a normal build it will cause a segfault.

ASAN stack trace:
==32295==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x615000000280 at pc 0x000000518c5c bp 0x7ffc48d18b90 sp 0x7ffc48d18b88
READ of size 1 at 0x615000000280 thread T0
    #0 0x518c5b in simple_vec /f/icoutils-0.32.0/icotool/extract.c:440:11
    #1 0x518c5b in extract_icons /f/icoutils-0.32.0/icotool/extract.c:332
    #2 0x51a867 in main /f/icoutils-0.32.0/icotool/main.c:359:22
    #3 0x7fdf2add25dc in __libc_start_main (/lib64/libc.so.6+0x205dc)
    #4 0x41aa79 in _start (/r/icoutils/icotool+0x41aa79)

0x615000000280 is located 0 bytes to the right of 512-byte region [0x615000000080,0x615000000280)
allocated by thread T0 here:
    #0 0x4d9668 in malloc (/r/icoutils/icotool+0x4d9668)
    #1 0x5300fe in xmalloc /f/icoutils-0.32.0/lib/xmalloc.c:41:13
    #2 0x51a867 in main /f/icoutils-0.32.0/icotool/main.c:359:22
    #3 0x7fdf2add25dc in __libc_start_main (/lib64/libc.so.6+0x205dc)

Hanno Boeck <hanno>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #42305:  oob.ico added by hanno (1KiB - image/vnd.microsoft.icon)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by res (Posted a comment)
  • -email is unavailable- added by hanno (Submitted the item)
  •  

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

    Date Changed by Updated Field Previous Value => Replaced by
    2017-10-31 res StatusNone Fixed
        Open/ClosedOpen Closed
    2017-10-31 hanno Attached File- Added oob.ico, #42305

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code