]> code.delx.au - refind/blobdiff - refind/main.c
Further tweaks to new NTFS Windows boot partition filtering.
[refind] / refind / main.c
index 9e653b49109ad9e5f30311e02475c4f64184773c..7fb2b73394c31ecca1a194ff0176e5d53d366b0a 100644 (file)
@@ -34,7 +34,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 /*
- * Modifications copyright (c) 2012-2014 Roderick W. Smith
+ * Modifications copyright (c) 2012-2015 Roderick W. Smith
  *
  * Modifications distributed under the terms of the GNU General Public
  * License (GPL) version 3 (GPLv3), a copy of which must be distributed
 #include "global.h"
 #include "config.h"
 #include "screen.h"
+#include "legacy.h"
 #include "lib.h"
 #include "icns.h"
 #include "menu.h"
 #include "mok.h"
 #include "gpt.h"
 #include "security_policy.h"
+#include "driver_support.h"
 #include "../include/Handle.h"
 #include "../include/refit_call_wrapper.h"
-#include "driver_support.h"
-#include "../include/syslinux_mbr.h"
+#include "../EfiLib/BdsHelper.h"
+#include "../EfiLib/legacy.h"
 
 #ifdef __MAKEWITH_GNUEFI
 #ifndef EFI_SECURITY_VIOLATION
@@ -62,9 +64,6 @@
 #endif
 #endif
 
-#include "../EfiLib/BdsHelper.h"
-#include "../EfiLib/legacy.h"
-
 #ifndef EFI_OS_INDICATIONS_BOOT_TO_FW_UI
 #define EFI_OS_INDICATIONS_BOOT_TO_FW_UI 0x0000000000000001ULL
 #endif
@@ -76,7 +75,8 @@
 //
 // constants
 
-#define MACOSX_LOADER_PATH      L"System\\Library\\CoreServices\\boot.efi"
+#define MACOSX_LOADER_DIR      L"System\\Library\\CoreServices"
+#define MACOSX_LOADER_PATH      ( MACOSX_LOADER_DIR L"\\boot.efi" )
 #if defined (EFIX64)
 #define SHELL_NAMES             L"\\EFI\\tools\\shell.efi,\\EFI\\tools\\shellx64.efi,\\shell.efi,\\shellx64.efi"
 #define GPTSYNC_NAMES           L"\\EFI\\tools\\gptsync.efi,\\EFI\\tools\\gptsync_x64.efi"
 // a ".efi" extension to be found when scanning for boot loaders.
 #define LINUX_MATCH_PATTERNS    L"vmlinuz*,bzImage*"
 
-// Default hint text for program-launch submenus
-#define SUBSCREEN_HINT1            L"Use arrow keys to move cursor; Enter to boot;"
-#define SUBSCREEN_HINT2            L"Insert or F2 to edit options; Esc to return to main menu"
-#define SUBSCREEN_HINT2_NO_EDITOR  L"Esc to return to main menu"
-
-// Load types
-#define TYPE_EFI    1
-#define TYPE_LEGACY 2
-
 static REFIT_MENU_ENTRY MenuEntryAbout    = { L"About rEFInd", TAG_ABOUT, 1, 0, 'A', NULL, NULL, NULL };
 static REFIT_MENU_ENTRY MenuEntryReset    = { L"Reboot Computer", TAG_REBOOT, 1, 0, 'R', NULL, NULL, NULL };
 static REFIT_MENU_ENTRY MenuEntryShutdown = { L"Shut Down Computer", TAG_SHUTDOWN, 1, 0, 'U', NULL, NULL, NULL };
-static REFIT_MENU_ENTRY MenuEntryReturn   = { L"Return to Main Menu", TAG_RETURN, 1, 0, 0, NULL, NULL, NULL };
+REFIT_MENU_ENTRY MenuEntryReturn   = { L"Return to Main Menu", TAG_RETURN, 1, 0, 0, NULL, NULL, NULL };
 static REFIT_MENU_ENTRY MenuEntryExit     = { L"Exit rEFInd", TAG_EXIT, 1, 0, 0, NULL, NULL, NULL };
 static REFIT_MENU_ENTRY MenuEntryFirmware = { L"Reboot to Computer Setup Utility", TAG_FIRMWARE, 1, 0, 0, NULL, NULL, NULL };
 
-static REFIT_MENU_SCREEN MainMenu       = { L"Main Menu", NULL, 0, NULL, 0, NULL, 0, L"Automatic boot",
-                                            L"Use arrow keys to move cursor; Enter to boot;",
-                                            L"Insert or F2 for more options; Esc to refresh" };
+REFIT_MENU_SCREEN MainMenu       = { L"Main Menu", NULL, 0, NULL, 0, NULL, 0, L"Automatic boot",
+                                     L"Use arrow keys to move cursor; Enter to boot;",
+                                     L"Insert or F2 for more options; Esc to refresh" };
 static REFIT_MENU_SCREEN AboutMenu      = { L"About", NULL, 0, NULL, 0, NULL, 0, NULL, L"Press Enter to return to main menu", L"" };
 
 REFIT_CONFIG GlobalConfig = { FALSE, TRUE, FALSE, FALSE, 0, 0, 0, DONT_CHANGE_TEXT_MODE, 20, 0, 0, GRAPHICS_FOR_OSX, LEGACY_TYPE_MAC,
@@ -175,10 +166,10 @@ static VOID AboutrEFInd(VOID)
 
     if (AboutMenu.EntryCount == 0) {
         AboutMenu.TitleImage = BuiltinIcon(BUILTIN_ICON_FUNC_ABOUT);
-        AddMenuInfoLine(&AboutMenu, L"rEFInd Version 0.8.3.4");
+        AddMenuInfoLine(&AboutMenu, L"rEFInd Version 0.8.5.7");
         AddMenuInfoLine(&AboutMenu, L"");
         AddMenuInfoLine(&AboutMenu, L"Copyright (c) 2006-2010 Christoph Pfisterer");
-        AddMenuInfoLine(&AboutMenu, L"Copyright (c) 2012-2014 Roderick W. Smith");
+        AddMenuInfoLine(&AboutMenu, L"Copyright (c) 2012-2015 Roderick W. Smith");
         AddMenuInfoLine(&AboutMenu, L"Portions Copyright (c) Intel Corporation and others");
         AddMenuInfoLine(&AboutMenu, L"Distributed under the terms of the GNU GPLv3 license");
         AddMenuInfoLine(&AboutMenu, L"");
@@ -269,12 +260,12 @@ static BOOLEAN IsValidLoader(EFI_FILE *RootDir, CHAR16 *FileName) {
 } // BOOLEAN IsValidLoader()
 
 // Launch an EFI binary.
-static EFI_STATUS StartEFIImageList(IN EFI_DEVICE_PATH **DevicePaths,
-                                    IN CHAR16 *LoadOptions, IN UINTN LoaderType,
-                                    IN CHAR16 *ImageTitle, IN CHAR8 OSType,
-                                    OUT UINTN *ErrorInStep,
-                                    IN BOOLEAN Verbose,
-                                    IN BOOLEAN IsDriver)
+EFI_STATUS StartEFIImageList(IN EFI_DEVICE_PATH **DevicePaths,
+                             IN CHAR16 *LoadOptions, IN UINTN LoaderType,
+                             IN CHAR16 *ImageTitle, IN CHAR8 OSType,
+                             OUT UINTN *ErrorInStep,
+                             IN BOOLEAN Verbose,
+                             IN BOOLEAN IsDriver)
 {
     EFI_STATUS              Status, ReturnStatus;
     EFI_HANDLE              ChildImageHandle;
@@ -379,7 +370,7 @@ bailout_unload:
 bailout:
     MyFreePool(FullLoadOptions);
     return ReturnStatus;
-} /* static EFI_STATUS StartEFIImageList() */
+} /* EFI_STATUS StartEFIImageList() */
 
 static EFI_STATUS StartEFIImage(IN EFI_DEVICE_PATH *DevicePath,
                                 IN CHAR16 *LoadOptions, IN UINTN LoaderType,
@@ -422,7 +413,7 @@ static EFI_STATUS RebootIntoFirmware(VOID) {
 
 // Record the value of the loader's name/description in rEFInd's "PreviousBoot" EFI variable,
 // if it's different from what's already stored there.
-static VOID StoreLoaderName(IN CHAR16 *Name) {
+VOID StoreLoaderName(IN CHAR16 *Name) {
    EFI_STATUS   Status;
    CHAR16       *OldName = NULL;
    UINTN        Length;
@@ -1010,12 +1001,12 @@ VOID SetLoaderDefaults(LOADER_ENTRY *Entry, CHAR16 *LoaderPath, REFIT_VOLUME *Vo
               StriCmp(NameClues, L"bootmgr.efi") == 0 ||
               StriCmp(NameClues, L"bootmgfw.efi") == 0 ||
               StriCmp(NameClues, L"bkpbootmgfw.efi") == 0) {
-      MergeStrings(&OSIconName, L"win", L',');
+      MergeStrings(&OSIconName, L"win8", L',');
       Entry->OSType = 'W';
       ShortcutLetter = 'W';
       Entry->UseGraphicsMode = GlobalConfig.GraphicsFor & GRAPHICS_FOR_WINDOWS;
    } else if (StriCmp(NameClues, L"xom.efi") == 0) {
-      MergeStrings(&OSIconName, L"xom,win", L',');
+      MergeStrings(&OSIconName, L"xom,win,win8", L',');
       Entry->UseGraphicsMode = TRUE;
       Entry->OSType = 'X';
       ShortcutLetter = 'W';
@@ -1035,42 +1026,10 @@ VOID SetLoaderDefaults(LOADER_ENTRY *Entry, CHAR16 *LoaderPath, REFIT_VOLUME *Vo
    MyFreePool(PathOnly);
 } // VOID SetLoaderDefaults()
 
-// // Add a network (PXE) EFI boot loader to the list, using automatic settings
-// // for icons, options, etc.
-// LOADER_ENTRY * AddNetbootLoaderEntry(IN CHAR16 *LoaderPath, IN CHAR16 *LoaderTitle, IN REFIT_VOLUME *Volume) {
-//    LOADER_ENTRY      *Entry;
-// 
-//    Print(L"Adding iPXE entry for '%s'\n", LoaderTitle);
-//    PauseForKey();
-//    CleanUpPathNameSlashes(LoaderPath);
-//    Entry = InitializeLoaderEntry(NULL);
-//    if (Entry != NULL) {
-//       Entry->Title = StrDuplicate((LoaderTitle != NULL) ? LoaderTitle : LoaderPath);
-//       Entry->me.Title = AllocateZeroPool(sizeof(CHAR16) * 256);
-//       // Extra space at end of Entry->me.Title enables searching on Volume->VolName even if another volume
-//       // name is identical except for something added to the end (e.g., VolB1 vs. VolB12).
-//       SPrint(Entry->me.Title, 255, L"NetBoot %s", (LoaderTitle != NULL) ? LoaderTitle : LoaderPath);
-//       Entry->me.Row = 0;
-//       if ((LoaderPath != NULL) && (LoaderPath[0] != L'\\')) {
-//          Entry->LoaderPath = StrDuplicate(L"\\");
-//       } else {
-//          Entry->LoaderPath = NULL;
-//       }
-//       MergeStrings(&(Entry->LoaderPath), LoaderPath, 0);
-//       Entry->VolName = Volume->VolName;
-//       Entry->DevicePath = FileDevicePath(Volume->DeviceHandle, Entry->LoaderPath);
-//       SetLoaderDefaults(Entry, LoaderPath, Volume);
-//       GenerateSubScreen(Entry, Volume);
-//       AddMenuEntry(&MainMenu, (REFIT_MENU_ENTRY *)Entry);
-//    }
-// 
-//    return(Entry);
-// } // LOADER_ENTRY * AddLoaderEntry()
-
 // Add a specified EFI boot loader to the list, using automatic settings
 // for icons, options, etc.
 LOADER_ENTRY * AddLoaderEntry(IN CHAR16 *LoaderPath, IN CHAR16 *LoaderTitle, IN REFIT_VOLUME *Volume) {
-   LOADER_ENTRY      *Entry;
+   LOADER_ENTRY  *Entry;
 
    CleanUpPathNameSlashes(LoaderPath);
    Entry = InitializeLoaderEntry(NULL);
@@ -1406,17 +1365,15 @@ CHAR16* RuniPXEDiscover(EFI_HANDLE Volume)
    EFI_STATUS       Status;
    EFI_DEVICE_PATH  *FilePath;
    EFI_HANDLE       iPXEHandle;
-   CHAR16           *boot_info;
-   UINTN            boot_info_size = 256 * sizeof(CHAR16);
+   CHAR16           *boot_info = NULL;
+   UINTN            boot_info_size = 0;
 
    FilePath = FileDevicePath (Volume, IPXE_DISCOVER_NAME);
    Status = refit_call6_wrapper(BS->LoadImage, FALSE, SelfImageHandle, FilePath,
                                 NULL, 0, &iPXEHandle);
-   if (Status != 0) {
+   if (Status != 0)
       return NULL;
-   } // if
 
-   boot_info = AllocatePool(256 * sizeof(CHAR16));
    Status = refit_call3_wrapper(BS->StartImage, iPXEHandle, &boot_info_size, &boot_info);
 
    return boot_info;
@@ -1463,9 +1420,9 @@ static VOID ScanEfiFiles(REFIT_VOLUME *Volume) {
          MergeStrings(&MatchPatterns, LINUX_MATCH_PATTERNS, L',');
 
       // check for Mac OS X boot loader
-      if (ShouldScan(Volume, L"System\\Library\\CoreServices")) {
+      if (ShouldScan(Volume, MACOSX_LOADER_DIR)) {
          StrCpy(FileName, MACOSX_LOADER_PATH);
-         if (FileExists(Volume->RootDir, FileName) && !FilenameIn(Volume, Directory, L"boot.efi", GlobalConfig.DontScanFiles)) {
+         if (FileExists(Volume->RootDir, FileName) && !FilenameIn(Volume, MACOSX_LOADER_DIR, L"boot.efi", GlobalConfig.DontScanFiles)) {
             AddLoaderEntry(FileName, L"Mac OS X", Volume);
             if (DuplicatesFallback(Volume, FileName))
                ScanFallbackLoader = FALSE;
@@ -1473,7 +1430,7 @@ static VOID ScanEfiFiles(REFIT_VOLUME *Volume) {
 
          // check for XOM
          StrCpy(FileName, L"System\\Library\\CoreServices\\xom.efi");
-         if (FileExists(Volume->RootDir, FileName) && !FilenameIn(Volume, Directory, L"boot.efi", GlobalConfig.DontScanFiles)) {
+         if (FileExists(Volume->RootDir, FileName) && !FilenameIn(Volume, MACOSX_LOADER_DIR, L"xom.efi", GlobalConfig.DontScanFiles)) {
             AddLoaderEntry(FileName, L"Windows XP (XoM)", Volume);
             if (DuplicatesFallback(Volume, FileName))
                ScanFallbackLoader = FALSE;
@@ -1483,7 +1440,7 @@ static VOID ScanEfiFiles(REFIT_VOLUME *Volume) {
       // check for Microsoft boot loader/menu
       if (ShouldScan(Volume, L"EFI\\Microsoft\\Boot")) {
          StrCpy(FileName, L"EFI\\Microsoft\\Boot\\bkpbootmgfw.efi");
-         if (FileExists(Volume->RootDir, FileName) &&  !FilenameIn(Volume, Directory, L"bkpbootmgfw.efi",
+         if (FileExists(Volume->RootDir, FileName) &&  !FilenameIn(Volume, L"EFI\\Microsoft\\Boot", L"bkpbootmgfw.efi",
              GlobalConfig.DontScanFiles)) {
             AddLoaderEntry(FileName, L"Microsoft EFI boot (Boot Repair backup)", Volume);
             FoundBRBackup = TRUE;
@@ -1491,7 +1448,7 @@ static VOID ScanEfiFiles(REFIT_VOLUME *Volume) {
                ScanFallbackLoader = FALSE;
          }
          StrCpy(FileName, L"EFI\\Microsoft\\Boot\\bootmgfw.efi");
-         if (FileExists(Volume->RootDir, FileName) &&  !FilenameIn(Volume, Directory, L"bootmgfw.efi", GlobalConfig.DontScanFiles)) {
+         if (FileExists(Volume->RootDir, FileName) &&  !FilenameIn(Volume, L"EFI\\Microsoft\\Boot", L"bootmgfw.efi", GlobalConfig.DontScanFiles)) {
             if (FoundBRBackup)
                AddLoaderEntry(FileName, L"Supposed Microsoft EFI boot (probably GRUB)", Volume);
             else
@@ -1578,285 +1535,8 @@ static VOID ScanOptical(VOID) {
    } // for
 } // static VOID ScanOptical()
 
-//
-// legacy boot functions
-//
-
-static EFI_STATUS ActivateMbrPartition(IN EFI_BLOCK_IO *BlockIO, IN UINTN PartitionIndex)
-{
-    EFI_STATUS          Status;
-    UINT8               SectorBuffer[512];
-    MBR_PARTITION_INFO  *MbrTable, *EMbrTable;
-    UINT32              ExtBase, ExtCurrent, NextExtCurrent;
-    UINTN               LogicalPartitionIndex = 4;
-    UINTN               i;
-    BOOLEAN             HaveBootCode;
-
-    // read MBR
-    Status = refit_call5_wrapper(BlockIO->ReadBlocks, BlockIO, BlockIO->Media->MediaId, 0, 512, SectorBuffer);
-    if (EFI_ERROR(Status))
-        return Status;
-    if (*((UINT16 *)(SectorBuffer + 510)) != 0xaa55)
-        return EFI_NOT_FOUND;  // safety measure #1
-
-    // add boot code if necessary
-    HaveBootCode = FALSE;
-    for (i = 0; i < MBR_BOOTCODE_SIZE; i++) {
-        if (SectorBuffer[i] != 0) {
-            HaveBootCode = TRUE;
-            break;
-        }
-    }
-    if (!HaveBootCode) {
-        // no boot code found in the MBR, add the syslinux MBR code
-        SetMem(SectorBuffer, MBR_BOOTCODE_SIZE, 0);
-        CopyMem(SectorBuffer, syslinux_mbr, SYSLINUX_MBR_SIZE);
-    }
-
-    // set the partition active
-    MbrTable = (MBR_PARTITION_INFO *)(SectorBuffer + 446);
-    ExtBase = 0;
-    for (i = 0; i < 4; i++) {
-        if (MbrTable[i].Flags != 0x00 && MbrTable[i].Flags != 0x80)
-            return EFI_NOT_FOUND;   // safety measure #2
-        if (i == PartitionIndex)
-            MbrTable[i].Flags = 0x80;
-        else if (PartitionIndex >= 4 && IS_EXTENDED_PART_TYPE(MbrTable[i].Type)) {
-            MbrTable[i].Flags = 0x80;
-            ExtBase = MbrTable[i].StartLBA;
-        } else
-            MbrTable[i].Flags = 0x00;
-    }
-
-    // write MBR
-    Status = refit_call5_wrapper(BlockIO->WriteBlocks, BlockIO, BlockIO->Media->MediaId, 0, 512, SectorBuffer);
-    if (EFI_ERROR(Status))
-        return Status;
-
-    if (PartitionIndex >= 4) {
-        // we have to activate a logical partition, so walk the EMBR chain
-
-        // NOTE: ExtBase was set above while looking at the MBR table
-        for (ExtCurrent = ExtBase; ExtCurrent; ExtCurrent = NextExtCurrent) {
-            // read current EMBR
-            Status = refit_call5_wrapper(BlockIO->ReadBlocks, BlockIO, BlockIO->Media->MediaId, ExtCurrent, 512, SectorBuffer);
-            if (EFI_ERROR(Status))
-                return Status;
-            if (*((UINT16 *)(SectorBuffer + 510)) != 0xaa55)
-                return EFI_NOT_FOUND;  // safety measure #3
-
-            // scan EMBR, set appropriate partition active
-            EMbrTable = (MBR_PARTITION_INFO *)(SectorBuffer + 446);
-            NextExtCurrent = 0;
-            for (i = 0; i < 4; i++) {
-                if (EMbrTable[i].Flags != 0x00 && EMbrTable[i].Flags != 0x80)
-                    return EFI_NOT_FOUND;   // safety measure #4
-                if (EMbrTable[i].StartLBA == 0 || EMbrTable[i].Size == 0)
-                    break;
-                if (IS_EXTENDED_PART_TYPE(EMbrTable[i].Type)) {
-                    // link to next EMBR
-                    NextExtCurrent = ExtBase + EMbrTable[i].StartLBA;
-                    EMbrTable[i].Flags = (PartitionIndex >= LogicalPartitionIndex) ? 0x80 : 0x00;
-                    break;
-                } else {
-                    // logical partition
-                    EMbrTable[i].Flags = (PartitionIndex == LogicalPartitionIndex) ? 0x80 : 0x00;
-                    LogicalPartitionIndex++;
-                }
-            }
-
-            // write current EMBR
-            Status = refit_call5_wrapper(BlockIO->WriteBlocks, BlockIO, BlockIO->Media->MediaId, ExtCurrent, 512, SectorBuffer);
-            if (EFI_ERROR(Status))
-                return Status;
-
-            if (PartitionIndex < LogicalPartitionIndex)
-                break;  // stop the loop, no need to touch further EMBRs
-        }
-
-    }
-
-    return EFI_SUCCESS;
-} /* static EFI_STATUS ActivateMbrPartition() */
-
-// early 2006 Core Duo / Core Solo models
-static UINT8 LegacyLoaderDevicePath1Data[] = {
-    0x01, 0x03, 0x18, 0x00, 0x0B, 0x00, 0x00, 0x00,
-    0x00, 0x00, 0xE0, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0xFF, 0xFF, 0xF9, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0x04, 0x06, 0x14, 0x00, 0xEB, 0x85, 0x05, 0x2B,
-    0xB8, 0xD8, 0xA9, 0x49, 0x8B, 0x8C, 0xE2, 0x1B,
-    0x01, 0xAE, 0xF2, 0xB7, 0x7F, 0xFF, 0x04, 0x00,
-};
-// mid-2006 Mac Pro (and probably other Core 2 models)
-static UINT8 LegacyLoaderDevicePath2Data[] = {
-    0x01, 0x03, 0x18, 0x00, 0x0B, 0x00, 0x00, 0x00,
-    0x00, 0x00, 0xE0, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0xFF, 0xFF, 0xF7, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0x04, 0x06, 0x14, 0x00, 0xEB, 0x85, 0x05, 0x2B,
-    0xB8, 0xD8, 0xA9, 0x49, 0x8B, 0x8C, 0xE2, 0x1B,
-    0x01, 0xAE, 0xF2, 0xB7, 0x7F, 0xFF, 0x04, 0x00,
-};
-// mid-2007 MBP ("Santa Rosa" based models)
-static UINT8 LegacyLoaderDevicePath3Data[] = {
-    0x01, 0x03, 0x18, 0x00, 0x0B, 0x00, 0x00, 0x00,
-    0x00, 0x00, 0xE0, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0xFF, 0xFF, 0xF8, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0x04, 0x06, 0x14, 0x00, 0xEB, 0x85, 0x05, 0x2B,
-    0xB8, 0xD8, 0xA9, 0x49, 0x8B, 0x8C, 0xE2, 0x1B,
-    0x01, 0xAE, 0xF2, 0xB7, 0x7F, 0xFF, 0x04, 0x00,
-};
-// early-2008 MBA
-static UINT8 LegacyLoaderDevicePath4Data[] = {
-    0x01, 0x03, 0x18, 0x00, 0x0B, 0x00, 0x00, 0x00,
-    0x00, 0x00, 0xC0, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0xFF, 0xFF, 0xF8, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0x04, 0x06, 0x14, 0x00, 0xEB, 0x85, 0x05, 0x2B,
-    0xB8, 0xD8, 0xA9, 0x49, 0x8B, 0x8C, 0xE2, 0x1B,
-    0x01, 0xAE, 0xF2, 0xB7, 0x7F, 0xFF, 0x04, 0x00,
-};
-// late-2008 MB/MBP (NVidia chipset)
-static UINT8 LegacyLoaderDevicePath5Data[] = {
-    0x01, 0x03, 0x18, 0x00, 0x0B, 0x00, 0x00, 0x00,
-    0x00, 0x40, 0xCB, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0xFF, 0xBF, 0xFF, 0xFF, 0x00, 0x00, 0x00, 0x00,
-    0x04, 0x06, 0x14, 0x00, 0xEB, 0x85, 0x05, 0x2B,
-    0xB8, 0xD8, 0xA9, 0x49, 0x8B, 0x8C, 0xE2, 0x1B,
-    0x01, 0xAE, 0xF2, 0xB7, 0x7F, 0xFF, 0x04, 0x00,
-};
-
-static EFI_DEVICE_PATH *LegacyLoaderList[] = {
-    (EFI_DEVICE_PATH *)LegacyLoaderDevicePath1Data,
-    (EFI_DEVICE_PATH *)LegacyLoaderDevicePath2Data,
-    (EFI_DEVICE_PATH *)LegacyLoaderDevicePath3Data,
-    (EFI_DEVICE_PATH *)LegacyLoaderDevicePath4Data,
-    (EFI_DEVICE_PATH *)LegacyLoaderDevicePath5Data,
-    NULL
-};
-
-#define MAX_DISCOVERED_PATHS (16)
-
-static VOID StartLegacy(IN LEGACY_ENTRY *Entry, IN CHAR16 *SelectionName)
-{
-    EFI_STATUS          Status;
-    EG_IMAGE            *BootLogoImage;
-    UINTN               ErrorInStep = 0;
-    EFI_DEVICE_PATH     *DiscoveredPathList[MAX_DISCOVERED_PATHS];
-
-    BeginExternalScreen(TRUE, L"Booting Legacy OS (Mac mode)");
-
-    BootLogoImage = LoadOSIcon(Entry->Volume->OSIconName, L"legacy", TRUE);
-    if (BootLogoImage != NULL)
-        BltImageAlpha(BootLogoImage,
-                      (UGAWidth  - BootLogoImage->Width ) >> 1,
-                      (UGAHeight - BootLogoImage->Height) >> 1,
-                      &StdBackgroundPixel);
-
-    if (Entry->Volume->IsMbrPartition) {
-        ActivateMbrPartition(Entry->Volume->WholeDiskBlockIO, Entry->Volume->MbrPartitionIndex);
-    }
-
-    ExtractLegacyLoaderPaths(DiscoveredPathList, MAX_DISCOVERED_PATHS, LegacyLoaderList);
-
-    StoreLoaderName(SelectionName);
-    Status = StartEFIImageList(DiscoveredPathList, Entry->LoadOptions, TYPE_LEGACY, L"legacy loader", 0, &ErrorInStep, TRUE, FALSE);
-    if (Status == EFI_NOT_FOUND) {
-        if (ErrorInStep == 1) {
-            Print(L"\nPlease make sure that you have the latest firmware update installed.\n");
-        } else if (ErrorInStep == 3) {
-            Print(L"\nThe firmware refused to boot from the selected volume. Note that external\n"
-                  L"hard drives are not well-supported by Apple's firmware for legacy OS booting.\n");
-        }
-    }
-    FinishExternalScreen();
-} /* static VOID StartLegacy() */
-
-// Start a device on a non-Mac using the EFI_LEGACY_BIOS_PROTOCOL
-static VOID StartLegacyUEFI(LEGACY_ENTRY *Entry, CHAR16 *SelectionName)
-{
-    BeginExternalScreen(TRUE, L"Booting Legacy OS (UEFI mode)");
-    StoreLoaderName(SelectionName);
-
-    BdsLibConnectDevicePath (Entry->BdsOption->DevicePath);
-    BdsLibDoLegacyBoot(Entry->BdsOption);
-
-    // If we get here, it means that there was a failure....
-    Print(L"Failure booting legacy (BIOS) OS.");
-    PauseForKey();
-    FinishExternalScreen();
-} // static VOID StartLegacyUEFI()
-
-static LEGACY_ENTRY * AddLegacyEntry(IN CHAR16 *LoaderTitle, IN REFIT_VOLUME *Volume)
-{
-    LEGACY_ENTRY            *Entry, *SubEntry;
-    REFIT_MENU_SCREEN       *SubScreen;
-    CHAR16                  *VolDesc, *LegacyTitle;
-    CHAR16                  ShortcutLetter = 0;
-
-    if (LoaderTitle == NULL) {
-        if (Volume->OSName != NULL) {
-            LoaderTitle = Volume->OSName;
-            if (LoaderTitle[0] == 'W' || LoaderTitle[0] == 'L')
-                ShortcutLetter = LoaderTitle[0];
-        } else
-            LoaderTitle = L"Legacy OS";
-    }
-    if (Volume->VolName != NULL)
-        VolDesc = Volume->VolName;
-    else
-        VolDesc = (Volume->DiskKind == DISK_KIND_OPTICAL) ? L"CD" : L"HD";
-
-    LegacyTitle = AllocateZeroPool(256 * sizeof(CHAR16));
-    if (LegacyTitle != NULL)
-       SPrint(LegacyTitle, 255, L"Boot %s from %s", LoaderTitle, VolDesc);
-    if (IsInSubstring(LegacyTitle, GlobalConfig.DontScanVolumes)) {
-       MyFreePool(LegacyTitle);
-       return NULL;
-    } // if
-
-    // prepare the menu entry
-    Entry = AllocateZeroPool(sizeof(LEGACY_ENTRY));
-    Entry->me.Title = LegacyTitle;
-    Entry->me.Tag          = TAG_LEGACY;
-    Entry->me.Row          = 0;
-    Entry->me.ShortcutLetter = ShortcutLetter;
-    Entry->me.Image        = LoadOSIcon(Volume->OSIconName, L"legacy", FALSE);
-    Entry->me.BadgeImage   = Volume->VolBadgeImage;
-    Entry->Volume          = Volume;
-    Entry->LoadOptions     = (Volume->DiskKind == DISK_KIND_OPTICAL) ? L"CD" :
-                              ((Volume->DiskKind == DISK_KIND_EXTERNAL) ? L"USB" : L"HD");
-    Entry->Enabled         = TRUE;
-
-    // create the submenu
-    SubScreen = AllocateZeroPool(sizeof(REFIT_MENU_SCREEN));
-    SubScreen->Title = AllocateZeroPool(256 * sizeof(CHAR16));
-    SPrint(SubScreen->Title, 255, L"Boot Options for %s on %s", LoaderTitle, VolDesc);
-    SubScreen->TitleImage = Entry->me.Image;
-    SubScreen->Hint1 = StrDuplicate(SUBSCREEN_HINT1);
-    if (GlobalConfig.HideUIFlags & HIDEUI_FLAG_EDITOR) {
-       SubScreen->Hint2 = StrDuplicate(SUBSCREEN_HINT2_NO_EDITOR);
-    } else {
-       SubScreen->Hint2 = StrDuplicate(SUBSCREEN_HINT2);
-    } // if/else
-
-    // default entry
-    SubEntry = AllocateZeroPool(sizeof(LEGACY_ENTRY));
-    SubEntry->me.Title = AllocateZeroPool(256 * sizeof(CHAR16));
-    SPrint(SubEntry->me.Title, 255, L"Boot %s", LoaderTitle);
-    SubEntry->me.Tag          = TAG_LEGACY;
-    SubEntry->Volume          = Entry->Volume;
-    SubEntry->LoadOptions     = Entry->LoadOptions;
-    AddMenuEntry(SubScreen, (REFIT_MENU_ENTRY *)SubEntry);
-
-    AddMenuEntry(SubScreen, &MenuEntryReturn);
-    Entry->me.SubScreen = SubScreen;
-    AddMenuEntry(&MainMenu, (REFIT_MENU_ENTRY *)Entry);
-    return Entry;
-} /* static LEGACY_ENTRY * AddLegacyEntry() */
-
-
 // default volume badge icon based on disk kind
-static EG_IMAGE * GetDiskBadge(IN UINTN DiskType) {
+EG_IMAGE * GetDiskBadge(IN UINTN DiskType) {
    EG_IMAGE * Badge = NULL;
 
    switch (DiskType) {
@@ -1871,229 +1551,7 @@ static EG_IMAGE * GetDiskBadge(IN UINTN DiskType) {
          break;
    } // switch()
    return Badge;
-} // static EG_IMAGE * GetDiskBadge()
-
-/**
-    Create a rEFInd boot option from a Legacy BIOS protocol option.
-*/
-static LEGACY_ENTRY * AddLegacyEntryUEFI(BDS_COMMON_OPTION *BdsOption, IN UINT16 DiskType)
-{
-    LEGACY_ENTRY            *Entry, *SubEntry;
-    REFIT_MENU_SCREEN       *SubScreen;
-    CHAR16                  ShortcutLetter = 0;
-    CHAR16 *LegacyDescription = StrDuplicate(BdsOption->Description);
-
-    if (IsInSubstring(LegacyDescription, GlobalConfig.DontScanVolumes))
-       return NULL;
-
-    // Remove stray spaces, since many EFIs produce descriptions with lots of
-    // extra spaces, especially at the end; this throws off centering of the
-    // description on the screen....
-    LimitStringLength(LegacyDescription, 100);
-
-    // prepare the menu entry
-    Entry = AllocateZeroPool(sizeof(LEGACY_ENTRY));
-    Entry->me.Title = AllocateZeroPool(256 * sizeof(CHAR16));
-    SPrint(Entry->me.Title, 255, L"Boot legacy target %s", LegacyDescription);
-    Entry->me.Tag          = TAG_LEGACY_UEFI;
-    Entry->me.Row          = 0;
-    Entry->me.ShortcutLetter = ShortcutLetter;
-    Entry->me.Image        = LoadOSIcon(L"legacy", L"legacy", TRUE);
-    Entry->LoadOptions     = (DiskType == BBS_CDROM) ? L"CD" :
-                             ((DiskType == BBS_USB) ? L"USB" : L"HD");
-    Entry->me.BadgeImage   = GetDiskBadge(DiskType);
-    Entry->BdsOption       = BdsOption;
-    Entry->Enabled         = TRUE;
-
-    // create the submenu
-    SubScreen = AllocateZeroPool(sizeof(REFIT_MENU_SCREEN));
-    SubScreen->Title = AllocateZeroPool(256 * sizeof(CHAR16));
-    SPrint(SubScreen->Title, 255, L"No boot options for legacy target");
-    SubScreen->TitleImage = Entry->me.Image;
-    SubScreen->Hint1 = StrDuplicate(SUBSCREEN_HINT1);
-    if (GlobalConfig.HideUIFlags & HIDEUI_FLAG_EDITOR) {
-       SubScreen->Hint2 = StrDuplicate(SUBSCREEN_HINT2_NO_EDITOR);
-    } else {
-       SubScreen->Hint2 = StrDuplicate(SUBSCREEN_HINT2);
-    } // if/else
-
-    // default entry
-    SubEntry = AllocateZeroPool(sizeof(LEGACY_ENTRY));
-    SubEntry->me.Title = AllocateZeroPool(256 * sizeof(CHAR16));
-    SPrint(SubEntry->me.Title, 255, L"Boot %s", LegacyDescription);
-    SubEntry->me.Tag          = TAG_LEGACY_UEFI;
-    Entry->BdsOption          = BdsOption; 
-    AddMenuEntry(SubScreen, (REFIT_MENU_ENTRY *)SubEntry);
-
-    AddMenuEntry(SubScreen, &MenuEntryReturn);
-    Entry->me.SubScreen = SubScreen;
-    AddMenuEntry(&MainMenu, (REFIT_MENU_ENTRY *)Entry);
-    return Entry;
-} /* static LEGACY_ENTRY * AddLegacyEntryUEFI() */
-
-/**
-    Scan for legacy BIOS targets on machines that implement EFI_LEGACY_BIOS_PROTOCOL.
-    In testing, protocol has not been implemented on Macs but has been
-    implemented on several Dell PCs and an ASUS motherboard.
-    Restricts output to disks of the specified DiskType.
-*/
-static VOID ScanLegacyUEFI(IN UINTN DiskType)
-{
-    EFI_STATUS                Status;
-    EFI_LEGACY_BIOS_PROTOCOL  *LegacyBios;
-    UINT16                    *BootOrder = NULL;
-    UINTN                     Index = 0;
-    CHAR16                    BootOption[10];
-    UINTN                     BootOrderSize = 0;
-    CHAR16                    Buffer[20];
-    BDS_COMMON_OPTION         *BdsOption;
-    LIST_ENTRY                TempList;
-    BBS_BBS_DEVICE_PATH       *BbsDevicePath = NULL;
-    BOOLEAN                   SearchingForUsb = FALSE;
-
-    InitializeListHead (&TempList);
-    ZeroMem (Buffer, sizeof (Buffer));
-
-    // If LegacyBios protocol is not implemented on this platform, then
-    //we do not support this type of legacy boot on this machine.
-    Status = refit_call3_wrapper(gBS->LocateProtocol, &gEfiLegacyBootProtocolGuid, NULL, (VOID **) &LegacyBios);
-    if (EFI_ERROR (Status))
-       return;
-
-    // EFI calls USB drives BBS_HARDDRIVE, but we want to distinguish them,
-    // so we set DiskType inappropriately elsewhere in the program and
-    // "translate" it here.
-    if (DiskType == BBS_USB) {
-       DiskType = BBS_HARDDISK;
-       SearchingForUsb = TRUE;
-    } // if
-
-    // Grab the boot order
-    BootOrder = BdsLibGetVariableAndSize(L"BootOrder", &gEfiGlobalVariableGuid, &BootOrderSize);
-    if (BootOrder == NULL) {
-        BootOrderSize = 0;
-    }
-
-    Index = 0;
-    while (Index < BootOrderSize / sizeof (UINT16))
-    {
-        // Grab each boot option variable from the boot order, and convert
-        // the variable into a BDS boot option
-        UnicodeSPrint (BootOption, sizeof (BootOption), L"Boot%04x", BootOrder[Index]);
-        BdsOption = BdsLibVariableToOption (&TempList, BootOption);
-
-        if (BdsOption != NULL) {
-           BbsDevicePath = (BBS_BBS_DEVICE_PATH *)BdsOption->DevicePath;
-           // Only add the entry if it is of a requested type (e.g. USB, HD)
-           // Two checks necessary because some systems return EFI boot loaders
-           // with a DeviceType value that would inappropriately include them
-           // as legacy loaders....
-           if ((BbsDevicePath->DeviceType == DiskType) && (BdsOption->DevicePath->Type == DEVICE_TYPE_BIOS)) {
-              // USB flash drives appear as hard disks with certain media flags set.
-              // Look for this, and if present, pass it on with the (technically
-              // incorrect, but internally useful) BBS_TYPE_USB flag set.
-              if (DiskType == BBS_HARDDISK) {
-                 if (SearchingForUsb && (BbsDevicePath->StatusFlag & (BBS_MEDIA_PRESENT | BBS_MEDIA_MAYBE_PRESENT))) {
-                    AddLegacyEntryUEFI(BdsOption, BBS_USB);
-                 } else if (!SearchingForUsb && !(BbsDevicePath->StatusFlag & (BBS_MEDIA_PRESENT | BBS_MEDIA_MAYBE_PRESENT))) {
-                    AddLegacyEntryUEFI(BdsOption, DiskType);
-                 }
-              } else {
-                 AddLegacyEntryUEFI(BdsOption, DiskType);
-              } // if/else
-           } // if
-        } // if (BdsOption != NULL)
-        Index++;
-    } // while
-} /* static VOID ScanLegacyUEFI() */
-
-static VOID ScanLegacyVolume(REFIT_VOLUME *Volume, UINTN VolumeIndex) {
-   UINTN VolumeIndex2;
-   BOOLEAN ShowVolume, HideIfOthersFound;
-
-   ShowVolume = FALSE;
-   HideIfOthersFound = FALSE;
-   if (Volume->IsAppleLegacy) {
-      ShowVolume = TRUE;
-      HideIfOthersFound = TRUE;
-   } else if (Volume->HasBootCode) {
-      ShowVolume = TRUE;
-      if (Volume->BlockIO == Volume->WholeDiskBlockIO &&
-         Volume->BlockIOOffset == 0 &&
-         Volume->OSName == NULL)
-         // this is a whole disk (MBR) entry; hide if we have entries for partitions
-         HideIfOthersFound = TRUE;
-   }
-   if (HideIfOthersFound) {
-      // check for other bootable entries on the same disk
-      for (VolumeIndex2 = 0; VolumeIndex2 < VolumesCount; VolumeIndex2++) {
-         if (VolumeIndex2 != VolumeIndex && Volumes[VolumeIndex2]->HasBootCode &&
-             Volumes[VolumeIndex2]->WholeDiskBlockIO == Volume->WholeDiskBlockIO)
-            ShowVolume = FALSE;
-      }
-   }
-
-   if (ShowVolume)
-      AddLegacyEntry(NULL, Volume);
-} // static VOID ScanLegacyVolume()
-
-// Scan attached optical discs for legacy (BIOS) boot code
-// and add anything found to the list....
-static VOID ScanLegacyDisc(VOID)
-{
-   UINTN                   VolumeIndex;
-   REFIT_VOLUME            *Volume;
-
-   if (GlobalConfig.LegacyType == LEGACY_TYPE_MAC) {
-      for (VolumeIndex = 0; VolumeIndex < VolumesCount; VolumeIndex++) {
-         Volume = Volumes[VolumeIndex];
-         if (Volume->DiskKind == DISK_KIND_OPTICAL)
-            ScanLegacyVolume(Volume, VolumeIndex);
-      } // for
-   } else if (GlobalConfig.LegacyType == LEGACY_TYPE_UEFI) {
-      ScanLegacyUEFI(BBS_CDROM);
-   }
-} /* static VOID ScanLegacyDisc() */
-
-// Scan internal hard disks for legacy (BIOS) boot code
-// and add anything found to the list....
-static VOID ScanLegacyInternal(VOID)
-{
-    UINTN                   VolumeIndex;
-    REFIT_VOLUME            *Volume;
-
-    if (GlobalConfig.LegacyType == LEGACY_TYPE_MAC) {
-       for (VolumeIndex = 0; VolumeIndex < VolumesCount; VolumeIndex++) {
-           Volume = Volumes[VolumeIndex];
-           if (Volume->DiskKind == DISK_KIND_INTERNAL)
-               ScanLegacyVolume(Volume, VolumeIndex);
-       } // for
-    } else if (GlobalConfig.LegacyType == LEGACY_TYPE_UEFI) {
-       // TODO: This actually picks up USB flash drives, too; try to find
-       // a way to differentiate the two....
-       ScanLegacyUEFI(BBS_HARDDISK);
-    }
-} /* static VOID ScanLegacyInternal() */
-
-// Scan external disks for legacy (BIOS) boot code
-// and add anything found to the list....
-static VOID ScanLegacyExternal(VOID)
-{
-   UINTN                   VolumeIndex;
-   REFIT_VOLUME            *Volume;
-
-   if (GlobalConfig.LegacyType == LEGACY_TYPE_MAC) {
-      for (VolumeIndex = 0; VolumeIndex < VolumesCount; VolumeIndex++) {
-         Volume = Volumes[VolumeIndex];
-         if (Volume->DiskKind == DISK_KIND_EXTERNAL)
-            ScanLegacyVolume(Volume, VolumeIndex);
-      } // for
-   } else if (GlobalConfig.LegacyType == LEGACY_TYPE_UEFI) {
-      // TODO: This actually doesn't do anything useful; leaving in hopes of
-      // fixing it later....
-      ScanLegacyUEFI(BBS_USB);
-   }
-} /* static VOID ScanLegacyExternal() */
+} // EG_IMAGE * GetDiskBadge()
 
 //
 // pre-boot tool functions
@@ -2272,50 +1730,6 @@ static VOID LoadDrivers(VOID)
        ConnectAllDriversToAllControllers();
 } /* static VOID LoadDrivers() */
 
-// Determine what (if any) type of legacy (BIOS) boot support is available
-static VOID FindLegacyBootType(VOID) {
-   EFI_STATUS                Status;
-   EFI_LEGACY_BIOS_PROTOCOL  *LegacyBios;
-
-   GlobalConfig.LegacyType = LEGACY_TYPE_NONE;
-
-   // UEFI-style legacy BIOS support is available only with some EFI implementations....
-   Status = refit_call3_wrapper(gBS->LocateProtocol, &gEfiLegacyBootProtocolGuid, NULL, (VOID **) &LegacyBios);
-   if (!EFI_ERROR (Status))
-      GlobalConfig.LegacyType = LEGACY_TYPE_UEFI;
-
-   // Macs have their own system. If the firmware vendor code contains the
-   // string "Apple", assume it's available. Note that this overrides the
-   // UEFI type, and might yield false positives if the vendor string
-   // contains "Apple" as part of something bigger, so this isn't 100%
-   // perfect.
-   if (StriSubCmp(L"Apple", ST->FirmwareVendor))
-      GlobalConfig.LegacyType = LEGACY_TYPE_MAC;
-} // static VOID FindLegacyBootType
-
-// Warn the user if legacy OS scans are enabled but the firmware can't support them....
-static VOID WarnIfLegacyProblems(VOID) {
-   BOOLEAN  found = FALSE;
-   UINTN    i = 0;
-
-   if (GlobalConfig.LegacyType == LEGACY_TYPE_NONE) {
-      do {
-         if (GlobalConfig.ScanFor[i] == 'h' || GlobalConfig.ScanFor[i] == 'b' || GlobalConfig.ScanFor[i] == 'c' ||
-             GlobalConfig.ScanFor[i] == 'H' || GlobalConfig.ScanFor[i] == 'B' || GlobalConfig.ScanFor[i] == 'C')
-            found = TRUE;
-         i++;
-      } while ((i < NUM_SCAN_OPTIONS) && (!found));
-
-      if (found) {
-         Print(L"NOTE: refind.conf's 'scanfor' line specifies scanning for one or more legacy\n");
-         Print(L"(BIOS) boot options; however, this is not possible because your computer lacks\n");
-         Print(L"the necessary Compatibility Support Module (CSM) support or that support is\n");
-         Print(L"disabled in your firmware.\n");
-         PauseForKey();
-      } // if (found)
-   } // if no legacy support
-} // static VOID WarnIfLegacyProblems()
-
 // Locates boot loaders. NOTE: This assumes that GlobalConfig.LegacyType is set correctly.
 static VOID ScanForBootloaders(VOID) {
    UINTN    i;
@@ -2385,7 +1799,8 @@ static VOID FindTool(CHAR16 *Locations, CHAR16 *Names, CHAR16 *Description, UINT
          PathName = StrDuplicate(DirName);
          MergeStrings(&PathName, FileName, (StriCmp(PathName, L"\\") == 0) ? 0 : L'\\');
          for (VolumeIndex = 0; VolumeIndex < VolumesCount; VolumeIndex++) {
-            if ((Volumes[VolumeIndex]->RootDir != NULL) && (FileExists(Volumes[VolumeIndex]->RootDir, PathName))) {
+            if ((Volumes[VolumeIndex]->RootDir != NULL) && (FileExists(Volumes[VolumeIndex]->RootDir, PathName)) &&
+                IsValidLoader(Volumes[VolumeIndex]->RootDir, PathName)) {
                SPrint(FullDescription, 255, L"%s at %s on %s", Description, PathName, Volumes[VolumeIndex]->VolName);
                AddToolEntry(Volumes[VolumeIndex]->DeviceHandle, PathName, FullDescription, BuiltinIcon(Icon), 'S', FALSE);
             } // if
@@ -2451,7 +1866,7 @@ static VOID ScanForTools(VOID) {
          case TAG_SHELL:
             j = 0;
             while ((FileName = FindCommaDelimited(SHELL_NAMES, j++)) != NULL) {
-               if (FileExists(SelfRootDir, FileName)) {
+               if (FileExists(SelfRootDir, FileName) && IsValidLoader(SelfRootDir, FileName)) {
                   AddToolEntry(SelfLoadedImage->DeviceHandle, FileName, L"EFI Shell", BuiltinIcon(BUILTIN_ICON_TOOL_SHELL),
                                'S', FALSE);
                }
@@ -2462,7 +1877,7 @@ static VOID ScanForTools(VOID) {
          case TAG_GPTSYNC:
             j = 0;
             while ((FileName = FindCommaDelimited(GPTSYNC_NAMES, j++)) != NULL) {
-               if (FileExists(SelfRootDir, FileName)) {
+               if (FileExists(SelfRootDir, FileName) && IsValidLoader(SelfRootDir, FileName)) {
                   AddToolEntry(SelfLoadedImage->DeviceHandle, FileName, L"Hybrid MBR tool", BuiltinIcon(BUILTIN_ICON_TOOL_PART),
                                'P', FALSE);
                } // if
@@ -2474,7 +1889,7 @@ static VOID ScanForTools(VOID) {
          case TAG_GDISK:
             j = 0;
             while ((FileName = FindCommaDelimited(GDISK_NAMES, j++)) != NULL) {
-               if (FileExists(SelfRootDir, FileName)) {
+               if (FileExists(SelfRootDir, FileName) && IsValidLoader(SelfRootDir, FileName)) {
                   AddToolEntry(SelfLoadedImage->DeviceHandle, FileName, L"disk partitioning tool",
                                BuiltinIcon(BUILTIN_ICON_TOOL_PART), 'G', FALSE);
                } // if
@@ -2486,7 +1901,7 @@ static VOID ScanForTools(VOID) {
          case TAG_NETBOOT:
             j = 0;
             while ((FileName = FindCommaDelimited(NETBOOT_NAMES, j++)) != NULL) {
-               if (FileExists(SelfRootDir, FileName)) {
+               if (FileExists(SelfRootDir, FileName) && IsValidLoader(SelfRootDir, FileName)) {
                   AddToolEntry(SelfLoadedImage->DeviceHandle, FileName, L"Netboot",
                                BuiltinIcon(BUILTIN_ICON_TOOL_NETBOOT), 'N', FALSE);
                } // if
@@ -2498,7 +1913,8 @@ static VOID ScanForTools(VOID) {
          case TAG_APPLE_RECOVERY:
             FileName = StrDuplicate(L"\\com.apple.recovery.boot\\boot.efi");
             for (VolumeIndex = 0; VolumeIndex < VolumesCount; VolumeIndex++) {
-               if ((Volumes[VolumeIndex]->RootDir != NULL) && (FileExists(Volumes[VolumeIndex]->RootDir, FileName))) {
+               if ((Volumes[VolumeIndex]->RootDir != NULL) && (FileExists(Volumes[VolumeIndex]->RootDir, FileName)) &&
+                   IsValidLoader(Volumes[VolumeIndex]->RootDir, FileName)) {
                   SPrint(Description, 255, L"Apple Recovery on %s", Volumes[VolumeIndex]->VolName);
                   AddToolEntry(Volumes[VolumeIndex]->DeviceHandle, FileName, Description,
                                BuiltinIcon(BUILTIN_ICON_TOOL_APPLE_RESCUE), 'R', TRUE);
@@ -2514,6 +1930,7 @@ static VOID ScanForTools(VOID) {
                SplitVolumeAndFilename(&FileName, &VolName);
                for (VolumeIndex = 0; VolumeIndex < VolumesCount; VolumeIndex++) {
                   if ((Volumes[VolumeIndex]->RootDir != NULL) && (FileExists(Volumes[VolumeIndex]->RootDir, FileName)) &&
+                      IsValidLoader(Volumes[VolumeIndex]->RootDir, FileName) &&
                       ((VolName == NULL) || (StriCmp(VolName, Volumes[VolumeIndex]->VolName) == 0))) {
                      SPrint(Description, 255, L"Microsoft Recovery on %s", Volumes[VolumeIndex]->VolName);
                      AddToolEntry(Volumes[VolumeIndex]->DeviceHandle, FileName, Description,