format_mp3: Fix a possible crash in mp3_read().
authorRussell Bryant <russell@russellbryant.com>
Sat, 12 May 2012 00:03:42 +0000 (00:03 +0000)
committerRussell Bryant <russell@russellbryant.com>
Sat, 12 May 2012 00:03:42 +0000 (00:03 +0000)
commit4a2678b561c47985bce68cb138fffdf4d88cca8c
tree192a365f71a135eb0beda18d6f048376c0ac0f73
parent2161d6870cd59c3ddd7f6334152989dc17a1732d
format_mp3: Fix a possible crash in mp3_read().

This patch fixes a potential crash in mp3_read() by not assuming that
dbuf has enough data to finish filling up the output buffer.  The patch
also makes sure that the dbuf state gets reset after we know we read
everything out of it already.

In passing, this patch includes some other cleanups of this module,
including stripping trailing whitespace, formatting fixes based on
coding guidelines, and removing a number of unused members from the
private state struct.

(closes issue ASTERISK-19761)
Reported by: Chris Maciejewsk
Tested by: Chris Maciejewsk
........

Merged revisions 366296 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 366297 from http://svn.asterisk.org/svn/asterisk/branches/10

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@366298 65c4cc65-6c06-0410-ace0-fbb531ad65f3
addons/format_mp3.c