On 07/22/2012 10:22 PM, Derek Buitenhuis wrote: > From: Clément Bœsch <[email protected]> > > If st is NULL, it means no 'fmt ' tag is found, but 'data' tag (which > needs a previous 'fmt ' tag to be parsed correctly and st initialized) > check will make sure st is never dereferenced in that case. > --- > libavformat/wav.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/wav.c b/libavformat/wav.c > index c01121f..1bfe6d5 100644 > --- a/libavformat/wav.c > +++ b/libavformat/wav.c > @@ -387,7 +387,7 @@ static int wav_read_header(AVFormatContext *s) > int rf64; > uint32_t tag, list_type; > AVIOContext *pb = s->pb; > - AVStream *st; > + AVStream *st = NULL; > WAVContext *wav = s->priv_data; > int ret, got_fmt = 0; > int64_t next_tag_ofs, data_ofs = -1;
Ok if you include the warning message in the commit log. -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
