[FFmpeg-devel] [PATCH] Fix crash when trying to get a fragment time for a non-existing fragment

Marth64 marth64 at proxyid.net
Thu Dec 12 00:41:09 EET 2024


Hi Eugene,

This makes sense to me.
get_frag_stream_info() has a chance to return null but is unchecked here.
It appears to be checked in all other scenarios in the file.

get_frag_time() is always expected to return a timestamp and in
undefined state will resort to returning AV_NOPTS_VALUE.
So, we could return AV_NOPTS_VALUE if frag_stream_info is null.

Therefore I think this LGTM and will resolve a potential crash.

Any objections to merging this?


More information about the ffmpeg-devel mailing list