Fixes .hdr+.cab installshield cabinet files not being extracted by BOS when relative paths are provided. (#374)

* Fixes .hdr+.cab installshield cabinet files not being extracted when relative paths are provided.

* Fix for unit tests empty file string.

* Better fix for unit test failures, due to https://learn.microsoft.com/en-us/dotnet/api/system.io.path.getfullpath?view=net-9.0 listing several more exceptions than the other functions, most of which I would not imagine should be directly handled.

* Removed try-catch fullpath obtaining, added getting fullpath in scanner via filestream name instead.

* Undid previous changes again, re-added path assertion at request, added assert.throws exception for empty paths in the unit tests

Author: HeroponRikiBestest

BinaryObjectScanner.Test/FileType/InstallShieldCABTests.cs

@@ -1,4 +1,5 @@
-using System.IO;
+using System;
+using System.IO;
 using BinaryObjectScanner.FileType;
 using Xunit;
 
@@ -25,8 +26,7 @@ public void ExtractStream_Null_False()
             string outDir = string.Empty;
             var extractable = new InstallShieldCAB();
 
-            bool actual = extractable.Extract(stream, file, outDir, includeDebug: false);
-            Assert.False(actual);
+            Assert.Throws<ArgumentException>(() => extractable.Extract(stream, file, outDir, includeDebug: false));
         }
 
         [Fact]
@@ -37,8 +37,7 @@ public void ExtractStream_Empty_False()
             string outDir = string.Empty;
             var extractable = new InstallShieldCAB();
 
-            bool actual = extractable.Extract(stream, file, outDir, includeDebug: false);
-            Assert.False(actual);
+            Assert.Throws<ArgumentException>(() => extractable.Extract(stream, file, outDir, includeDebug: false));
         }
     }
 }

BinaryObjectScanner/FileType/InstallShieldCAB.cs

@@ -24,6 +24,8 @@ public bool Extract(string file, string outDir, bool includeDebug)
         /// <inheritdoc/>
         public bool Extract(Stream? stream, string file, string outDir, bool includeDebug)
         {
+            // Handles getting full path if relative paths were passed.
+            file = Path.GetFullPath(file);
             // Get the name of the first cabinet file or header
             var directory = Path.GetDirectoryName(file);
             string noExtension = Path.GetFileNameWithoutExtension(file);

BinaryObjectScanner/Scanner.cs

@@ -207,7 +207,7 @@ private ProtectionDictionary GetInternalProtections(string file)
             try
             {
                 using FileStream fs = File.Open(file, FileMode.Open, FileAccess.Read, FileShare.ReadWrite);
-                return GetInternalProtections(file, fs);
+                return GetInternalProtections(fs.Name, fs);
             }
             catch (Exception ex)
             {