Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#4533 closed defect (fixed)

Broken MPEG-2 GOP after 1010b36d8672d891e77aa713e07ff749e8b0797f

Reported by: kierank Owned by:
Priority: important Component: avcodec
Version: git-master Keywords: mpeg2video regression videolan
Cc: michael 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 Changed 5 years ago by michael

  • Resolution set to fixed
  • Status changed from new to closed

comment:2 Changed 4 years ago by cehoyos

  • Keywords mpeg2video regression videolan added
  • Priority changed from normal to important
  • Version changed from unspecified to git-master

videolan ticket 14811

comment:3 Changed 4 years ago by cehoyos

  • Cc michael 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 4 years ago by cehoyos (previous) (diff)
Note: See TracTickets for help on using tickets.