Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#4533 closed defect (fixed)

Broken MPEG-2 GOP after 1010b36d8672d891e77aa713e07ff749e8b0797f

Reported by: Kieran Kunhya Owned by:
Priority: important Component: avcodec
Version: git-master Keywords: mpeg2video regression videolan
Cc: Michael Niedermayer Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Only reproducible via API so no ffmpeg log etc.

https://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavcodec/mpeg12dec.c;h=efbd53e529fb0ec2c1dde19371a6caa587dd0cb4;hb=HEAD#l1318

ff_set_sar needs to be called AFTER ff_set_dimensions
Otherwise av_image_check_sar uses an invalid width/height

(Thanks to Meuuh for report)

Change History (3)

comment:1 by Michael Niedermayer, 9 years ago

Resolution: fixed
Status: newclosed

comment:2 by Carl Eugen Hoyos, 9 years ago

Keywords: mpeg2video regression videolan added
Priority: normalimportant
Version: unspecifiedgit-master

videolan ticket 14811

comment:3 by Carl Eugen Hoyos, 9 years ago

Cc: Michael Niedermayer added

This should probably be backported but neither here nor on videolan is enough information for me to know which branches are affected.
The fix was backported to 2.6, sorry for the noise!

Last edited 9 years ago by Carl Eugen Hoyos (previous) (diff)
Note: See TracTickets for help on using tickets.