stream_decoder: reset has_seek_table before read_metadata_seektable_()
authorMax Kellermann <max@duempel.org>
Thu, 14 Jul 2016 08:22:43 +0000 (10:22 +0200)
committerErik de Castro Lopo <erikd@mega-nerd.com>
Thu, 14 Jul 2016 08:42:05 +0000 (18:42 +1000)
commita52177b0d1e79c5cc15ab7d3503bcd1bfb3fd54b
tree5818f9fac864dfe63732437c1d4e457623fbde23
parentf7491f9741a0fa2e623d321882b29a49c35596d8
stream_decoder: reset has_seek_table before read_metadata_seektable_()

If a seek table has already been read successfully, then the
has_seek_table flag is true.  Now imagine the file comes with another
seek table, which doesn't make sense, but libFLAC accepts it happily.
If reading this second seek table fails (for example allocation
failure), read_metadata_seektable_() returns false, but the
has_seek_table flag is still true.  If the calling application happens
to ignore this failure, and at some point tries to seek, the process
will crash due to NULL pointer dereference.  This would sure be an
application bug that needs to be fixed, but libFLAC's internal state
is inconsistent, so let's fix this up.

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>
src/libFLAC/stream_decoder.c