On Thu, Nov 17, 2016 at 02:41:14AM -0300, James Almer wrote: > Signed-off-by: James Almer <[email protected]> > --- > libavformat/mov.c | 15 ++++----------- > 1 file changed, 4 insertions(+), 11 deletions(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index 8d6cc12..5ba9965 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -5647,20 +5647,13 @@ static int mov_read_header(AVFormatContext *s) > break; > case AVMEDIA_TYPE_VIDEO: > if (sc->display_matrix) { > - AVPacketSideData *sd, *tmp; > + int ret; > > - tmp = av_realloc_array(st->side_data, > - st->nb_side_data + 1, sizeof(*tmp)); > - if (!tmp) > + ret = av_stream_add_side_data(st, AV_PKT_DATA_DISPLAYMATRIX, > (uint8_t*)sc->display_matrix, > + sizeof(int32_t) * 9); > + if (ret < 0)
> return AVERROR(ENOMEM); is this intended not to be return ret ? should be ok otherwise [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into democracies and democracies degenerate into despotisms. -- Aristotle
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
