> 2021年11月10日 下午8:59,Andreas Rheinhardt <andreas.rheinha...@outlook.com> 写道: > > 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.rheinha...@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 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". >
Patchset test ok, LGTM Thanks Steven Liu _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".