Jacob Lifshay: > Fixes: https://trac.ffmpeg.org/ticket/11591 > > Signed-off-by: Jacob Lifshay <programmerj...@gmail.com> > --- > libavcodec/mpeg12enc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c > index 8364368fde..e48437eb8a 100644 > --- a/libavcodec/mpeg12enc.c > +++ b/libavcodec/mpeg12enc.c > @@ -472,7 +472,7 @@ static int mpeg1_encode_picture_header(MPVMainEncContext > *const m) > (side_data->size / 3 & A53_MAX_CC_COUNT) | 0x40); // > flags, cc_count > put_bits(&s->pb, 8, 0xff); // em_data > > - ff_copy_bits(&s->pb, side_data->data, side_data->size); > + ff_copy_bits(&s->pb, side_data->data, side_data->size * 8); > > put_bits(&s->pb, 8, 0xff); // marker_bits > } else {
Thanks for this patch (and for the bug report; and sorry for causing you trouble). Your patch was also my first attempt to fix this, but unfortunately ff_copy_bits() may overread, yet the side data buffer need not be padded, so my patch simply reverts the ff_copy_bits(). - 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".
