Opened 3 months ago

Last modified 2 months ago

#9456 new defect

ffprobe coded_width, coded_height, and all dec_ctx->properties fields broken (regression)

Reported by: Gregory Beauregard Owned by:
Priority: important Component: ffprobe
Version: git-master Keywords: codecpar regression
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Summary of the bug: The coded_height and coded_width parameters don't work in ffprobe; they always display the same as the width and height fields.
How to reproduce: Run the following command on any 1080p h264 file. Expected coded_height is 1088, but it will show 1080.

ffprobe -select_streams v -show_entries stream=coded_width,coded_height file.mkv

This behavior is a regression since https://github.com/FFmpeg/FFmpeg/commit/04f49645a519233f3638104e0df5215758652fcb

This behavior was also broken in the past and was fixed in https://trac.ffmpeg.org/ticket/6958 before regressing.

ffprobe should not be outputting wrong info

Change History (6)

comment:1 by Gregory Beauregard, 3 months ago

My personal dog in this is I would like to access this info so I have a way to do calculations needed for bitstream filter (bsf) cropping, but this should be removed rather than displaying the wrong value if the wish is to deprecate it.

A fix or removal would also need backported to 4.4.

comment:2 by Gregory Beauregard, 3 months ago

Removal or re-implementation of all fields that depended on ist->dec_ctx->properties being set (like closed_captions and film-grain and more?) will also need backported to 4.4 if the API has already been removed (precluding the revert of the commit). All ffprobe fields that depended on this being set regressed in the same commit and now show fake wrong results with ffprobe.

If the API is not yet removed the better solution is probably reverting the commit in the 4.4 branch so the deprecation or fix of all these ffprobe fields can be done correctly in master.

Last edited 3 months ago by Gregory Beauregard (previous) (diff)

comment:3 by Gregory Beauregard, 3 months ago

avstream.codec was removed before 4.4 release: https://github.com/FFmpeg/FFmpeg/commit/3749eede

So we will need to backport a deprecation or re-implementation of these ffprobe fields to 4.4 branch

Last edited 3 months ago by Gregory Beauregard (previous) (diff)

comment:4 by Gregory Beauregard, 3 months ago

Summary: ffprobe coded_width and coded_height don't work (regression)ffprobe coded_width, coded_height, and all dec_ctx->properties fields broken (regression)

comment:5 by Gregory Beauregard, 2 months ago

Component: ffprobeavcodec
Keywords: codecpar regression added
Priority: normalimportant

updated properties to correspond to triage on previous instance of this bug https://trac.ffmpeg.org/ticket/6958 since the bug is the same except for also breaking a few additional things

Last edited 2 months ago by Gregory Beauregard (previous) (diff)

comment:6 by Gregory Beauregard, 2 months ago

Component: avcodecffprobe
Note: See TracTickets for help on using tickets.