Throw if root directory entry is invalid

ironfede · Oct 7, 2024 · e7919e3 · e7919e3
1 parent 4a58717
commit e7919e3
Showing 1 changed file with 3 additions and 35 deletions.
diff --git a/sources/OpenMcdf/CompoundFile.cs b/sources/OpenMcdf/CompoundFile.cs
@@ -1502,44 +1502,12 @@ internal List<Sector> GetSectorChain(int secID, SectorType chainType)
 
         internal RBTree GetChildrenTree(IDirectoryEntry entry)
         {
-            RBTree bst = new RBTree();
-
-            // Load children from their original tree.
-            LoadChildren(bst, entry);
-            //bst = DoLoadChildrenTrusted(directoryEntries[sid]);
-
-            //bst.Print();
-            //bst.Print();
-            //Trace.WriteLine("#### After rethreading");
-
-            return bst;
-        }
-
-        private RBTree DoLoadChildrenTrusted(IDirectoryEntry de)
-        {
-            RBTree bst = null;
-
-            if (de.Child != DirectoryEntry.NOSTREAM)
-            {
-                bst = new RBTree(directoryEntries[de.Child]);
-            }
-
+            RBTree bst = new();
+            List<int> levelSIDs = new List<int>();
+            LoadChildren(bst, entry.Child, levelSIDs);
             return bst;
         }
 
-        private void LoadChildren(RBTree bst, IDirectoryEntry de)
-        {
-            if (de.Child != DirectoryEntry.NOSTREAM)
-            {
-                IDirectoryEntry child = directoryEntries[de.Child];
-                if (child.StgType != StgType.StgInvalid)
-                {
-                    List<int> levelSIDs = new List<int>();
-                    LoadChildren(bst, child, levelSIDs);
-                }
-            }
-        }
-
         private static void NullifyChildNodes(IDirectoryEntry de)
         {
             de.Parent = null;