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.

faad: check the result of adts_find_frame()
7cea5357 · Max Kellermann · 77db32f8 · 7cea5357
Commit 7cea5357 authored Feb 17, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

faad_plugin.c src/decoder/faad_plugin.c +2 -4

No files found.
--- 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);