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?
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to