[FFmpeg-devel] [RFC] [PATCH] hevc_ps: reorder SPS reading to allow some errors
Christophe Gisquet
christophe.gisquet at gmail.com
Thu Aug 21 19:14:03 CEST 2014
Hi,
2014-08-21 3:06 GMT+02:00 Michael Niedermayer <michaelni at gmx.at>:
> heres a narrower solution for this file
> not sure its better
> i can apply whichever you prefer, probably we wont find out which
> is the best solution before pushing something and waiting for more
> bug reports
Thanks for narrowing it done, and now I have dug it further.
When you look at what get parsed before that,
vui->def_disp_win.left_offset gets an insane value unrelated to the
video dimensions. I tried instead to do as if
vui->default_display_window_flag and related syntax does not exist.
Then, the timing info is decoded, and the values are actually normal:
vui_num_units_in_tick = 1000
vui_time_scale = 23976
(rest less relevant and meaningfull)
So I think the actual "bug" in that stream is that it does not contain
any default_display_window information. Why is that? Probably because
the encoder writes an older VUI syntax, and several bits get
misinterpreted. By luck they don't bother us. It would be nice to know
what syntax is used, but there could be so many variations...
The attached therefore barely improves on your patch.
--
Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-hevc_ps-check-overflow-and-test-alternate-syntax.patch
Type: text/x-patch
Size: 2702 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140821/524b3dad/attachment.bin>
More information about the ffmpeg-devel
mailing list