[FFmpeg-cvslog] avformat/av1dec: Set position of AVPackets given to BSF

Andreas Rheinhardt git at videolan.org
Mon Aug 23 08:29:00 EEST 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Sun Aug 22 04:03:15 2021 +0200| [8a805095c16553f5c50fbcb11d11d0f34376e038] | committer: Andreas Rheinhardt

avformat/av1dec: Set position of AVPackets given to BSF

Reviewed-by: James Almer <jamrial at gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=8a805095c16553f5c50fbcb11d11d0f34376e038
---

 libavformat/av1dec.c | 44 +++++++++++++++-----------------------------
 1 file changed, 15 insertions(+), 29 deletions(-)

diff --git a/libavformat/av1dec.c b/libavformat/av1dec.c
index f6c575069e..e021615c1f 100644
--- a/libavformat/av1dec.c
+++ b/libavformat/av1dec.c
@@ -22,7 +22,6 @@
 #include "config.h"
 
 #include "libavutil/common.h"
-#include "libavutil/fifo.h"
 #include "libavutil/opt.h"
 #include "libavcodec/av1_parse.h"
 #include "libavcodec/bsf.h"
@@ -299,7 +298,6 @@ typedef struct ObuContext {
     const AVClass *class;
     AVBSFContext *bsf;
     AVRational framerate;
-    AVFifoBuffer *fifo;
 } ObuContext;
 
 //For low overhead obu, we can't foresee the obu size before we parsed the header.
@@ -372,9 +370,6 @@ static int obu_probe(const AVProbeData *p)
 static int obu_read_header(AVFormatContext *s)
 {
     ObuContext *c = s->priv_data;
-    c->fifo = av_fifo_alloc(MAX_OBU_HEADER_SIZE);
-    if (!c->fifo)
-        return AVERROR(ENOMEM);
     return read_header(s, &c->framerate, &c->bsf, c);
 }
 
@@ -383,37 +378,26 @@ static int obu_get_packet(AVFormatContext *s, AVPacket *pkt)
     ObuContext *c = s->priv_data;
     uint8_t header[MAX_OBU_HEADER_SIZE + AV_INPUT_BUFFER_PADDING_SIZE];
     int64_t obu_size;
-    int size = av_fifo_space(c->fifo);
+    int size;
     int ret, len, type;
 
-    av_fifo_generic_write(c->fifo, s->pb, size,
-                          (int (*)(void*, void*, int))avio_read);
-    size = av_fifo_size(c->fifo);
-    if (!size)
-        return 0;
-
-    av_fifo_generic_peek(c->fifo, header, size, NULL);
+    if ((ret = ffio_ensure_seekback(s->pb, MAX_OBU_HEADER_SIZE)) < 0)
+        return ret;
+    size = avio_read(s->pb, header, MAX_OBU_HEADER_SIZE);
+    if (size < 0)
+        return size;
 
     len = read_obu_with_size(header, size, &obu_size, &type);
     if (len < 0) {
         av_log(c, AV_LOG_ERROR, "Failed to read obu\n");
         return len;
     }
+    avio_seek(s->pb, -size, SEEK_CUR);
 
-    ret = av_new_packet(pkt, len);
-    if (ret < 0) {
-        av_log(c, AV_LOG_ERROR, "Failed to allocate packet for obu\n");
-        return ret;
-    }
-    size = FFMIN(size, len);
-    av_fifo_generic_read(c->fifo, pkt->data, size, NULL);
-    len -= size;
-    if (len > 0) {
-        ret = avio_read(s->pb, pkt->data + size, len);
-        if (ret != len) {
-            av_log(c, AV_LOG_ERROR, "Failed to read %d frome file\n", len);
-            return ret < 0 ? ret : AVERROR_INVALIDDATA;
-        }
+    ret = av_get_packet(s->pb, pkt, len);
+    if (ret != len) {
+        av_log(c, AV_LOG_ERROR, "Failed to get packet for obu\n");
+        return ret < 0 ? ret : AVERROR_INVALIDDATA;
     }
     return 0;
 }
@@ -425,7 +409,10 @@ static int obu_read_packet(AVFormatContext *s, AVPacket *pkt)
 
     while (1) {
         ret = obu_get_packet(s, pkt);
-        if (ret < 0)
+        /* In case of AVERROR_EOF we need to flush the BSF. Conveniently
+         * obu_get_packet() returns a blank pkt in this case which
+         * can be used to signal that the BSF should be flushed. */
+        if (ret < 0 && ret != AVERROR_EOF)
             return ret;
         ret = av_bsf_send_packet(c->bsf, pkt);
         if (ret < 0) {
@@ -448,7 +435,6 @@ static int obu_read_close(AVFormatContext *s)
 {
     ObuContext *c = s->priv_data;
 
-    av_fifo_freep(&c->fifo);
     av_bsf_free(&c->bsf);
     return 0;
 }



More information about the ffmpeg-cvslog mailing list