Andreas Rheinhardt: > It is sane, but UB. It could happen in case of allocation errors > in vc2_encode_init(). > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com> > --- > libavcodec/vc2enc_dwt.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/vc2enc_dwt.c b/libavcodec/vc2enc_dwt.c > index a8d3f1c669..441af040ec 100644 > --- a/libavcodec/vc2enc_dwt.c > +++ b/libavcodec/vc2enc_dwt.c > @@ -276,6 +276,8 @@ av_cold int ff_vc2enc_init_transforms(VC2TransformContext > *s, int p_stride, > > av_cold void ff_vc2enc_free_transforms(VC2TransformContext *s) > { > - av_free(s->buffer - s->padding); > - s->buffer = NULL; > + if (s->buffer) { > + av_free(s->buffer - s->padding); > + s->buffer = NULL; > + } > }
Will apply these patches later tonight unless there are objections. - 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".