You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-04 22:03:09 +02:00
avformat/hlsenc: limit segments count to search duplicated segments filenames
This linear search has a complexity of O(n). When ffmpeg attempts to parse a playlist containing approximately 100,000 segments, it effectively causes a hang for several minutes. This patch limits the allowed size for duplicate searches to a reasonable value. Now it takes between 0.5 and a few seconds (tested on different devices) instead of several minutes. Signed-off-by: Artem Smorodin <artem.smorodin@dacast.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
committed by
Michael Niedermayer
parent
4b6f6af384
commit
0a108a37e1
@ -1172,7 +1172,7 @@ static int hls_append_segment(struct AVFormatContext *s, HLSContext *hls,
|
|||||||
if (hls->use_localtime_mkdir) {
|
if (hls->use_localtime_mkdir) {
|
||||||
filename = vs->avf->url;
|
filename = vs->avf->url;
|
||||||
}
|
}
|
||||||
if ((find_segment_by_filename(vs->segments, filename) || find_segment_by_filename(vs->old_segments, filename))
|
if (vs->nb_entries <= 5000 && (find_segment_by_filename(vs->segments, filename) || find_segment_by_filename(vs->old_segments, filename))
|
||||||
&& !byterange_mode) {
|
&& !byterange_mode) {
|
||||||
av_log(hls, AV_LOG_WARNING, "Duplicated segment filename detected: %s\n", filename);
|
av_log(hls, AV_LOG_WARNING, "Duplicated segment filename detected: %s\n", filename);
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user