From 96fe44e2e8e4c77e66cbb0834d340719741e964c Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Tue, 9 Nov 2021 22:21:37 +0100 Subject: [PATCH] avformat/hlsenc: Only write something if there is something to write This is especially important in case avio_write() would be switched to an unsigned type like size_t, then a potential error from avio_read() (with negative return value) would no longer be handled gracefully by avio_write(). Reviewed-by: Steven Liu Signed-off-by: Andreas Rheinhardt --- libavformat/hlsenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 557cf1bee3..cfd0c036d1 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -2383,8 +2383,8 @@ static int64_t append_single_file(AVFormatContext *s, VariantStream *vs) do { read_byte = avio_read(vs->out, buf, BUFSIZE); - avio_write(vs->out_single_file, buf, read_byte); if (read_byte > 0) { + avio_write(vs->out_single_file, buf, read_byte); total_size += read_byte; ret = total_size; }