On Sun, 23 Jun 2019 06:46:12 +0200 Andreas Rheinhardt <andreas.rheinha...@gmail.com> wrote:
> The mov flavour of timed text uses the first two bytes of the packet > as a length field. And up until 11bef2fe said length field has been > read correctly in the mov2textsub bsf. But since then the next two > bytes are read as if they were the length field. This is fixed in > this commit. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > --- > libavcodec/movsub_bsf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/movsub_bsf.c b/libavcodec/movsub_bsf.c > index 5878607061..cd48aa7bb8 100644 > --- a/libavcodec/movsub_bsf.c > +++ b/libavcodec/movsub_bsf.c > @@ -75,8 +75,8 @@ static int mov2textsub(AVBSFContext *ctx, AVPacket > *pkt) return AVERROR_INVALIDDATA; > } > > - pkt->data += 2; > pkt->size = FFMIN(pkt->size - 2, AV_RB16(pkt->data)); > + pkt->data += 2; > > return 0; > } LGTM --phil _______________________________________________ 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".
