diff options
| author |  Martin Panter <vadmium à gmail·com> | 2012-04-23 12:34:48 +0000 |
|---|---|---|
| committer |  waker <wakeroid@gmail.com> | 2012-05-03 21:11:09 +0200 |
| commit | 58e74616a1fd2a66b9ffcd053174e049d5019b77 (patch) | |
| tree | e44690e35befbfa6a07e3a3d985ceca262dfb010 /ChangeLog | |
| parent | 0b2aa0745cd2187035de1ff28577c5286f9c2d28 (diff) | |
Check for reading past end before calling mp4ff_read_sample()
Previously errors like the following would occasionally be reported at the end
of an AAC file:
mp4ff_read_sample: malloc failure (tried to alloc -2147483648 bytes). possible mp4ff bug or memleak! please report a bug to deadbeef developers (i'm serious).
This was because the value of “sample” passed to mp4ff_audio_frame_size()
caused that function to read just off the end of an array.
Bug reported at https://sourceforge.net/support/tracker.php?aid=3321066
Fix inspired by https://bugs.launchpad.net/bugs/40043 “Crash (SIGSEGV) in
memcpy using libfaad2”
Diffstat (limited to 'ChangeLog')
0 files changed, 0 insertions, 0 deletions