Browse Source

Merge pull request #7403 from 1337joe/fix-external-file-prefix-check

Cody Robibero 3 years ago
parent
commit
4791d56f6c

+ 2 - 1
MediaBrowser.Providers/MediaInfo/MediaInfoResolver.cs

@@ -166,7 +166,8 @@ namespace MediaBrowser.Providers.MediaInfo
             foreach (var file in files)
             {
                 var fileNameWithoutExtension = Path.GetFileNameWithoutExtension(file.AsSpan());
-                if (prefix.Equals(fileNameWithoutExtension[..prefix.Length], StringComparison.OrdinalIgnoreCase)
+                if (fileNameWithoutExtension.Length >= prefix.Length
+                    && prefix.Equals(fileNameWithoutExtension[..prefix.Length], StringComparison.OrdinalIgnoreCase)
                     && (fileNameWithoutExtension.Length == prefix.Length || _namingOptions.MediaFlagDelimiters.Contains(fileNameWithoutExtension[prefix.Length])))
                 {
                     var externalPathInfo = _externalPathParser.ParseFile(file, fileNameWithoutExtension[prefix.Length..].ToString());

+ 1 - 0
tests/Jellyfin.Providers.Tests/MediaInfo/MediaInfoResolverTests.cs

@@ -157,6 +157,7 @@ public class MediaInfoResolverTests
     }
 
     [Theory]
+    [InlineData("cover.jpg")]
     [InlineData("My.Video.mp3")]
     [InlineData("My.Video.png")]
     [InlineData("My.Video.txt")]