On Fri, May 03, 2019 at 06:11:00AM +0000, Andreas Rheinhardt wrote: > Michael Niedermayer: > > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > > --- > > 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 2c99753b5b..561ec152e7 100644 > > --- a/libavformat/webm_chunk.c > > +++ b/libavformat/webm_chunk.c > > @@ -84,7 +84,7 @@ static int chunk_mux_init(AVFormatContext *s) > > return 0; > > } > > > > -static int get_chunk_filename(AVFormatContext *s, int is_header, char > > *filename) > > +static int get_chunk_filename(AVFormatContext *s, int is_header, char > > filename[MAX_FILENAME_SIZE]) > > { > > WebMChunkContext *wc = s->priv_data; > > AVFormatContext *oc = wc->avf; > > > 1. This is not a fix, merely a cosmetic clarification. After all, this > change does not allow the compiler to infer that every pointer > corresponding to the filename argument will point to an array of at > least MAX_FILENAME_SIZE elements. (C99 added a static keyword for this.)
static analyzers can use such hints to detect violations but the real intent here was that the human developer would see from just looking at the argument that it has a implied size. And that way to avoid a mistake [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Its not that you shouldnt use gotos but rather that you should write readable code and code with gotos often but not always is less readable
signature.asc
Description: PGP signature
_______________________________________________ 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".