Behave if child atom is empty (#170)

Credit: https://github.com/ta264 Lidarr#5
mono · Jul 22, 2019 · 21d46df · LeadAssimilator · Apr 8, 2024 · LeadAssimilator
1 parent f4c076e
commit 21d46df
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/TaglibSharp/Mpeg4/Box.cs b/src/TaglibSharp/Mpeg4/Box.cs
@@ -456,6 +456,9 @@ protected IEnumerable<Box> LoadChildren (TagLib.File file)
 			header.Box = this;
 			while (position < end) {
 				Box child = BoxFactory.CreateBox (file, position, header, Handler, children.Count);
+				if (child.Size == 0)
+					break;
+
 				children.Add (child);
 				position += child.Size;
 			}

diff --git a/src/TaglibSharp/Mpeg4/Boxes/UnknownBox.cs b/src/TaglibSharp/Mpeg4/Boxes/UnknownBox.cs
@@ -64,13 +64,12 @@ public class UnknownBox : Box
 		/// <exception cref="ArgumentNullException">
 		///    <paramref name="file" /> is <see langword="null" />.
 		/// </exception>
-		public UnknownBox (BoxHeader header, TagLib.File file, IsoHandlerBox handler)
-			: base (header, handler)
+		public UnknownBox (BoxHeader header, TagLib.File file, IsoHandlerBox handler) : base (header, handler)
 		{
 			if (file == null)
 				throw new ArgumentNullException (nameof (file));
 
-			data = LoadData (file);
+			data = file.ReadBlock (DataSize > 0 ? DataSize : 0);
 		}
 
 		#endregion