[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