Opened 3 years ago

Closed 2 months ago

#8177 closed defect (worksforme)

four division by zero bugs in aaccoder.c and lpc.h

Reported by: Suhwan Owned by:
Priority: normal Component: undetermined
Version: git-master Keywords: ubsan
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Summary of the bug:
There're four division by zero bugs in aaccoder.c and lpc.h

libavcodec/aaccoder.c:846:60: runtime error: division by zero
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior libavcodec/aaccoder.c:846:60 in 
libavcodec/aaccoder.c:858:60: runtime error: division by zero
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior libavcodec/aaccoder.c:858:60 in 
libavcodec/aaccoder.c:864:62: runtime error: division by zero
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior libavcodec/aaccoder.c:864:62 in 
libavcodec/lpc.h:155:27: runtime error: division by zero
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior libavcodec/lpc.h:155:27 in

How to reproduce:

% ./ffmpeg_g -stream_loop 1 -y -r 83 -i surge-2-16-B-ima4.mov -loglevel 0 -map 0 -vframes 4 -aframes 74 -strict 3 tmp.m4a

ffmpeg version N-94961-g1d86e4b3eb Copyright (c) 2000-2019 the FFmpeg developers
  built with clang version 6.0.0-1ubuntu2 (tags/RELEASE_600/final)
  configuration: --cc=clang --cxx=clang++ --ld=clang --enable-debug --toolchain=clang-usan

Attachments (2)

gdb-aac (98.3 KB ) - added by Suhwan 3 years ago.
surge-2-16-B-ima4.mov (601.2 KB ) - added by Suhwan 3 years ago.
poc

Download all attachments as: .zip

Change History (3)

by Suhwan, 3 years ago

Attachment: gdb-aac added

by Suhwan, 3 years ago

Attachment: surge-2-16-B-ima4.mov added

poc

comment:1 by Michael Niedermayer, 2 months ago

Resolution: worksforme
Status: newclosed

Does not replicate. Please provide a testcase that replicates with git master. Probably this was fixed long ago

Note: See TracTickets for help on using tickets.