diff options
author | Guangyu Sun <sunguangyucn@gmail.com> | 2022-09-29 21:11:34 +0300 |
---|---|---|
committer | Anton Khirnov <anton@khirnov.net> | 2022-10-14 13:24:21 +0300 |
commit | e7dd643419cc395cecca0c84135346b85f2803ed (patch) | |
tree | e735914a9e59108c5e68185a7456a273a3034446 | |
parent | 79bd6a21a04c7c457bfeb53fddc157bbd80cdaac (diff) |
lavf/async: Fix ring_write return value
This fixes a regression from commit 36117968ad.
wrapped_url_read() used to be able to return positive number from
ffurl_read(). It relies on the result to check if EOF is reached in
async_buffer_task().
But FIFO callbacks must return 0 on success. This should be handled
in ring_write() instead.
Test case:
ffmpeg -f lavfi -i testsrc -t 1 test.mp4
ffmpeg -i async:test.mp4
Signed-off-by: Guangyu Sun <gsun@roblox.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit fc6f7e2a3b2fff6c4df957684d939586a3de448f)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
-rw-r--r-- | libavformat/async.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/libavformat/async.c b/libavformat/async.c index 547417aa1e..3c6f89cab9 100644 --- a/libavformat/async.c +++ b/libavformat/async.c @@ -144,8 +144,14 @@ static int wrapped_url_read(void *src, void *dst, size_t *size) static int ring_write(RingBuffer *ring, URLContext *h, size_t size) { + int ret; + av_assert2(size <= ring_space(ring)); - return av_fifo_write_from_cb(ring->fifo, wrapped_url_read, h, &size); + ret = av_fifo_write_from_cb(ring->fifo, wrapped_url_read, h, &size); + if (ret < 0) + return ret; + + return size; } static int ring_size_of_read_back(RingBuffer *ring) |