#57 heap-buffer-overflow in apetag.c:243:5

[Kim Davies]

heap-buffer-overflow in apetag.c:243:5

Environment:

Distributor ID: Ubuntu
Description: Ubuntu 24.04.1 LTS
Release: 24.04

Compiler:

Ubuntu clang version 14.0.6
Target: x86_64-pc-linux-gnu
Thread model: posix

Compiling:

CFLAGS="-fsanitize=address -fsanitize=undefined -fno-omit-frame-pointer -g" CXXFLAGS="-fsanitize=address -fsanitize=undefined -fno-omit-frame-pointer -g" make

Version:

commit f669c2c826411c132655be66465077e77f25d93d

PoCfile:

Unzip poc.zip.

Behavior:

./mp3gain poc/mp3gain_001.poc

Output:

/home/exp/src/mp3gain/mp3gain/mp3gain/apetag.c:296:39: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /home/exp/src/mp3gain/mp3gain/mp3gain/apetag.c:296:39 in 
=================================================================
==721901==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x602000000095 at pc 0x560b3a05319b bp 0x7ffca5cc41d0 sp 0x7ffca5cc39a0
READ of size 4 at 0x602000000095 thread T0
    #0 0x560b3a05319a in __asan_memcpy (/home/exp/bin/asan_ubsan/mp3gain+0xd219a) (BuildId: 6adba16c3fc2a25561194f170abc8ab9f4e2b498)
    #1 0x560b3a0bd5b2 in ReadMP3APETag /home/exp/src/mp3gain/mp3gain/mp3gain/apetag.c:243:5
    #2 0x560b3a0c0365 in ReadMP3GainAPETag /home/exp/src/mp3gain/mp3gain/mp3gain/apetag.c:381:3
    #3 0x560b3a09e69b in main_impl /home/exp/src/mp3gain/mp3gain/mp3gain/mp3gain.c:1834:4
    #4 0x560b3a099431 in main /home/exp/src/mp3gain/mp3gain/mp3gain/mp3gain.c:2830:9
    #5 0x7f38ef82a1c9 in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #6 0x7f38ef82a28a in __libc_start_main csu/../csu/libc-start.c:360:3
    #7 0x560b39fd04b4 in _start (/home/exp/bin/asan_ubsan/mp3gain+0x4f4b4) (BuildId: 6adba16c3fc2a25561194f170abc8ab9f4e2b498)

0x602000000095 is located 1 bytes to the right of 4-byte region [0x602000000090,0x602000000094)
allocated by thread T0 here:
    #0 0x560b3a053ec2 in __interceptor_malloc (/home/exp/bin/asan_ubsan/mp3gain+0xd2ec2) (BuildId: 6adba16c3fc2a25561194f170abc8ab9f4e2b498)
    #1 0x560b3a0bc68c in ReadMP3APETag /home/exp/src/mp3gain/mp3gain/mp3gain/apetag.c:216:24
    #2 0x560b3a0c0365 in ReadMP3GainAPETag /home/exp/src/mp3gain/mp3gain/mp3gain/apetag.c:381:3
    #3 0x560b3a09e69b in main_impl /home/exp/src/mp3gain/mp3gain/mp3gain/mp3gain.c:1834:4
    #4 0x560b3a099431 in main /home/exp/src/mp3gain/mp3gain/mp3gain/mp3gain.c:2830:9
    #5 0x7f38ef82a1c9 in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #6 0x7f38ef82a28a in __libc_start_main csu/../csu/libc-start.c:360:3
    #7 0x560b39fd04b4 in _start (/home/exp/bin/asan_ubsan/mp3gain+0x4f4b4) (BuildId: 6adba16c3fc2a25561194f170abc8ab9f4e2b498)

SUMMARY: AddressSanitizer: heap-buffer-overflow (/home/exp/bin/asan_ubsan/mp3gain+0xd219a) (BuildId: 6adba16c3fc2a25561194f170abc8ab9f4e2b498) in __asan_memcpy
Shadow bytes around the buggy address:
  0x0c047fff7fc0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0c047fff7fd0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0c047fff7fe0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0c047fff7ff0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0c047fff8000: fa fa 00 fa fa fa fd fd fa fa fd fd fa fa 00 05
=>0x0c047fff8010: fa fa[04]fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0c047fff8020: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0c047fff8030: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0c047fff8040: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0c047fff8050: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0c047fff8060: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07 
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb
==721901==ABORTING