[FFmpeg-devel] [PATCH 1/2] avformat/hlsenc: Remove nonsense memset
Andreas Rheinhardt
andreas.rheinhardt at outlook.com
Wed Nov 10 14:59:40 EET 2021
The memset here is both unnecessary (avio_read() ignores the previous
content of the destination buffer) as well as nonsense (for a char
buf[BUFSIZE] sizeof(buf) and sizeof(BUFSIZE) are not the same; the
latter is sizeof(int)).
Fixes Coverity issue #1465863.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
---
libavformat/hlsenc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index 98608a834a..557cf1bee3 100644
--- a/libavformat/hlsenc.c
+++ b/libavformat/hlsenc.c
@@ -2382,7 +2382,6 @@ static int64_t append_single_file(AVFormatContext *s, VariantStream *vs)
}
do {
- memset(buf, 0, sizeof(BUFSIZE));
read_byte = avio_read(vs->out, buf, BUFSIZE);
avio_write(vs->out_single_file, buf, read_byte);
if (read_byte > 0) {
--
2.30.2
More information about the ffmpeg-devel
mailing list