Initial round of fixes

Author: HeroponRikiBestest

BinaryObjectScanner/FileType/ISO9660.cs

@@ -107,6 +107,7 @@ public static bool NoteworthyApplicationUse(PrimaryVolumeDescriptor pvd)
                 // character. If these are found to be causing issues they can be added.
             }
 
+            // Seems to come from "FERGUS_MCNEILL - ISOCD 1.00 by Pantaray, Inc. USA -"
             offset = 1;
             potentialAppUseString = applicationUse.ReadNullTerminatedAnsiString(ref offset);
             if (potentialAppUseString == "FS")

BinaryObjectScanner/Protection/AlphaROM.cs

@@ -80,20 +80,25 @@ public class AlphaROM : IDiskImageCheck<ISO9660>, IExecutableCheck<PortableExecu
             // to only consist of capital letters and numbers, a basic byte value check can be performed to ensure
             // at least 5 bytes are numbers and 5 bytes are letters. Unfortunately, there doesn't seem to be quite
             // enough of a pattern to have a better check than this, but it works well enough.
-            if (applicationIdentifierString.Length == 18 
+            if (applicationIdentifierString.Length == 18
                 && Array.FindAll(applicationIdentifierStringBytes, b => b < 60).Length >= 5
                 && Array.FindAll(applicationIdentifierStringBytes, b => b > 60).Length >= 5)
+            {
                 return "AlphaROM";
+            }
 
             // Type #2: Usually 20 characters long, but Redump ID 124334 is 18 characters long. Validate that it
             // starts with YYYYMMDD, followed by 6-8 more numbers, followed by letters.
             if (applicationIdentifierString.Length >= 18 && applicationIdentifierString.Length <= 20)
             {
                 if (Int32.TryParse(applicationIdentifierString.Substring(0, 4), out int year) == false
-                    ||  Int32.TryParse(applicationIdentifierString.Substring(4, 2), out int month) == false
-                    ||  Int32.TryParse(applicationIdentifierString.Substring(6, 2), out int day) == false
-                    ||  Int32.TryParse(applicationIdentifierString.Substring(8, 6), out int extraTime) == false)
+                    || Int32.TryParse(applicationIdentifierString.Substring(4, 2), out int month) == false
+                    || Int32.TryParse(applicationIdentifierString.Substring(6, 2), out int day) == false
+                    || Int32.TryParse(applicationIdentifierString.Substring(8, 6), out int extraTime) == false)
+                {
                     return null;
+                }
+                
                 if (year >= 2009 || year < 2000 || month > 12 || day > 31)
                     return null;
 

BinaryObjectScanner/Protection/Tages.cs

@@ -43,12 +43,12 @@ public class TAGES : IDiskImageCheck<ISO9660>, IExecutableCheck<PortableExecutab
             // Early tages has a 4-byte value at the beginning of the AU data and nothing else.
             // Redump ID  8776, 21321, 35932
             offset = 0;
-            var earlyTagesBytes = applicationUse.ReadBytes(ref offset, 4);
+            var earlyTagesBytes = applicationUse.ReadUInt32LittleEndian(ref offset);
             var zeroBytes = applicationUse.ReadBytes(ref offset, 508);
-            
+
             // Check on earlyTagesBytes needed because Redump ID 56899 begins with "FUN" and is then all 0x00.
             // 0x70 value is probably just by chance of where early TAGES checks, but it seems to be consistent.
-            if (Array.TrueForAll(zeroBytes, b => b == 0x00) && !Array.TrueForAll(earlyTagesBytes, b => b == 0x00) && earlyTagesBytes[3] == 0x70)
+            if (Array.TrueForAll(zeroBytes, b => b == 0x00) && (earlyTagesBytes & 0x70000000) == 0x70000000)
                 return "TAGES (Early)";
 
             // The original releases of Moto Racer 3 (31578, 34669) are so early they have seemingly nothing identifiable.