From 7cea5357e3b92264314fde86cf6d57e2f3176792 Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@duempel.org>
Date: Tue, 17 Feb 2009 22:56:07 +0100
Subject: faad: check the result of adts_find_frame()

Instead of checking if the buffer is empty after adts_find_frame(),
check adts_find_frame()'s return value.  This is more robust.
---
 src/decoder/faad_plugin.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/decoder/faad_plugin.c b/src/decoder/faad_plugin.c
index 2db6b99b3..ee05db4d5 100644
--- a/src/decoder/faad_plugin.c
+++ b/src/decoder/faad_plugin.c
@@ -392,10 +392,8 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
 	file_time = 0.0;
 
 	do {
-		adts_find_frame(&buffer);
-		faad_buffer_fill(&buffer);
-
-		if (buffer.length == 0)
+		size_t frame_size = adts_find_frame(&buffer);
+		if (frame_size == 0)
 			break;
 
 		decoded = faad_decoder_decode(decoder, &buffer, &frame_info);
-- 
cgit v1.2.3