Cutlery/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2025-07-09 03:04:24 -04:00
Code Issues Packages Projects Releases Wiki Activity

Order ids to merge to avoid stack overflow

Browse Source 
As said in https://github.com/jellyfin/jellyfin/issues/3176, merging 1 with 2 and then 2 with 1 cause an stack overflow. sorting ids first fix the problem
This commit is contained in:
Mister Rajoy 2020-05-31 20:23:23 +02:00
parent 730395886d
commit 1b90798b90
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
MediaBrowser.Api/VideosService.cs
View File

@ -125,10 +125,12 @@ namespace MediaBrowser.Api
public void Post(MergeVersions request) public void Post(MergeVersions request)
{ {
var items = request.Ids.Split(',') var items = request.Ids.Split(',')
.Select(i => _libraryManager.GetItemById(i)) .Select(i => _libraryManager.GetItemById(i))
.OfType<Video>() .OfType<Video>()
.ToList(); .ToList();
items = items.OrderBy(i => i.Id).ToList();
if (items.Count < 2) if (items.Count < 2)
{ {