summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2021-11-09 22:21:37 +0100
committerAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2021-11-18 04:53:07 +0100
commit96fe44e2e8e4c77e66cbb0834d340719741e964c (patch)
treec89e0a3abe7673e719911e293d095e3f586cb0fd
parent7596b6530d5f00ea0510b5c698714c76369811e7 (diff)
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 <lq@chinaffmpeg.org> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
-rw-r--r--libavformat/hlsenc.c2
1 files changed, 1 insertions, 1 deletions
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;
}