diff mbox

[FFmpeg-devel] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

Message ID 20170601111213.31981-1-daniel.kucera@gmail.com
State Superseded
Headers show

Commit Message

Daniel Kucera June 1, 2017, 11:12 a.m. UTC 
Signed-off-by: Daniel Kucera <daniel.kucera@gmail.com>
---
 libavformat/aviobuf.c | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

Comments

Michael Niedermayer June 2, 2017, 1:31 a.m. UTC | #1 
On Thu, Jun 01, 2017 at 01:12:13PM +0200, Daniel Kucera wrote:
> Signed-off-by: Daniel Kucera <daniel.kucera@gmail.com>
> ---
>  libavformat/aviobuf.c | 18 ++++++++++--------
>  1 file changed, 10 insertions(+), 8 deletions(-)

breaks fate (infinte loops)
example fate-swr-resample-s32p-2626-8000

[...]
diff mbox

Patch

diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c
index 1667e9f08b..40cff4dc56 100644
--- a/libavformat/aviobuf.c
+++ b/libavformat/aviobuf.c
@@ -556,13 +556,14 @@  static void fill_buffer(AVIOContext *s)
     if (s->read_packet)
         len = s->read_packet(s->opaque, dst, len);
     else
-        len = 0;
-    if (len <= 0) {
+        len = AVERROR_EOF;
+    if (len == AVERROR_EOF) {
         /* do not modify buffer if EOF reached so that a seek back can
            be done without rereading data */
         s->eof_reached = 1;
-        if (len < 0)
-            s->error = len;
+    } else if (len < 0) {
+        s->eof_reached = 1;
+        s->error= len;
     } else {
         s->pos += len;
         s->buf_ptr = dst;
@@ -630,13 +631,14 @@  int avio_read(AVIOContext *s, unsigned char *buf, int size)
                 // bypass the buffer and read data directly into buf
                 if(s->read_packet)
                     len = s->read_packet(s->opaque, buf, size);
-
-                if (len <= 0) {
+                if (len == AVERROR_EOF) {
                     /* do not modify buffer if EOF reached so that a seek back can
                     be done without rereading data */
                     s->eof_reached = 1;
-                    if(len<0)
-                        s->error= len;
+                    break;
+                } else if (len < 0) {
+                    s->eof_reached = 1;
+                    s->error= len;
                     break;
                 } else {
                     s->pos += len;