Andreas Rheinhardt: > The last argument of av_strlcpy is supposed to contain the size of the > destination buffer, but it was filled with the size of the source > string, effectively negating its very purpose. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@googlemail.com> > --- > libavformat/webm_chunk.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/webm_chunk.c b/libavformat/webm_chunk.c > index ec1ec4bf91..2c99753b5b 100644 > --- a/libavformat/webm_chunk.c > +++ b/libavformat/webm_chunk.c > @@ -96,7 +96,7 @@ static int get_chunk_filename(AVFormatContext *s, int > is_header, char *filename) > av_log(oc, AV_LOG_ERROR, "No header filename provided\n"); > return AVERROR(EINVAL); > } > - av_strlcpy(filename, wc->header_filename, > strlen(wc->header_filename) + 1); > + av_strlcpy(filename, wc->header_filename, MAX_FILENAME_SIZE); > } else { > if (av_get_frame_filename(filename, MAX_FILENAME_SIZE, > s->url, wc->chunk_index - 1) < 0) { >
I'd like to ping the whole patchset. - Andreas _______________________________________________ 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".