From e88524f274093d594978297ff28999daf1d19416 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 19 Aug 2014 20:26:12 +0200 Subject: decoder/faad: check InputStream::KnownSize() Replace the bogus GetSize() check and call GetSize() only when necessary. --- src/decoder/plugins/FaadDecoderPlugin.cxx | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src/decoder/plugins') diff --git a/src/decoder/plugins/FaadDecoderPlugin.cxx b/src/decoder/plugins/FaadDecoderPlugin.cxx index af63553eb..21351f2ba 100644 --- a/src/decoder/plugins/FaadDecoderPlugin.cxx +++ b/src/decoder/plugins/FaadDecoderPlugin.cxx @@ -112,8 +112,7 @@ adts_song_duration(DecoderBuffer *buffer) { const InputStream &is = decoder_buffer_get_stream(buffer); const bool estimate = !is.CheapSeeking(); - const auto file_size = is.GetSize(); - if (estimate && file_size <= 0) + if (estimate && !is.KnownSize()) return -1; unsigned sample_rate = 0; @@ -149,6 +148,7 @@ adts_song_duration(DecoderBuffer *buffer) if (offset <= 0) return -1; + const auto file_size = is.GetSize(); frames = (frames * file_size) / offset; break; } @@ -202,6 +202,10 @@ faad_song_duration(DecoderBuffer *buffer, InputStream &is) return song_length; } else if (data.size >= 5 && memcmp(data.data, "ADIF", 4) == 0) { /* obtain the duration from the ADIF header */ + + if (!is.KnownSize()) + return -1; + size_t skip_size = (data.data[4] & 0x80) ? 9 : 0; if (8 + skip_size > data.size) -- cgit v1.2.3