On Mon, May 02, 2016 at 01:34:45AM +0300, [email protected] wrote: > From: Jan Sebechlebsky <[email protected]> > > TeeSlave.bsfs is array of pointers to AVBitStreamFilterContext, > so element size should be really size of a pointer, not size > of TeeSlave structure. > > Signed-off-by: Jan Sebechlebsky <[email protected]> > --- > libavformat/tee.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/tee.c b/libavformat/tee.c > index 499ef33..879d5b8 100644 > --- a/libavformat/tee.c > +++ b/libavformat/tee.c > @@ -324,7 +324,7 @@ static int open_slave(AVFormatContext *avf, char *slave, > TeeSlave *tee_slave) > } > tee_slave->header_written = 1; > > - tee_slave->bsfs = av_calloc(avf2->nb_streams, sizeof(TeeSlave)); > + tee_slave->bsfs = av_calloc(avf2->nb_streams, > sizeof(AVBitStreamFilterContext*));
a sizeof based on tee_slave->bsfs itself instead of its type would make this kind of bugs harder to create so maybe that should be used instead but above is ok too [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid, but not doing something just because microsoft did it is even more stupid. If everything ms did were stupid they would be bankrupt already.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
