Loading...
common/MachOAnalyzer.cpp dyld-940 /dev/null
--- dyld/dyld-940/common/MachOAnalyzer.cpp
+++ /dev/null
@@ -1,6540 +0,0 @@
-/*
- * Copyright (c) 2017 Apple Inc. All rights reserved.
- *
- * @APPLE_LICENSE_HEADER_START@
- *
- * This file contains Original Code and/or Modifications of Original Code
- * as defined in and that are subject to the Apple Public Source License
- * Version 2.0 (the 'License'). You may not use this file except in
- * compliance with the License. Please obtain a copy of the License at
- * http://www.opensource.apple.com/apsl/ and read it before using this
- * file.
- *
- * The Original Code and all software distributed under the License are
- * distributed on an 'AS IS' basis, WITHOUT WARRANTY OF ANY KIND, EITHER
- * EXPRESS OR IMPLIED, AND APPLE HEREBY DISCLAIMS ALL SUCH WARRANTIES,
- * INCLUDING WITHOUT LIMITATION, ANY WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE, QUIET ENJOYMENT OR NON-INFRINGEMENT.
- * Please see the License for the specific language governing rights and
- * limitations under the License.
- *
- * @APPLE_LICENSE_HEADER_END@
- */
-
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <sys/errno.h>
-#include <sys/fcntl.h>
-#include <sys/mman.h>
-#include <mach/mach.h>
-#include <assert.h>
-#include <limits.h>
-#include <stdlib.h>
-#include <string.h>
-#include <unistd.h>
-#include <mach-o/reloc.h>
-#include <mach-o/x86_64/reloc.h>
-#include <mach-o/nlist.h>
-#include <TargetConditionals.h>
-
-#include "MachOAnalyzer.h"
-#include "CodeSigningTypes.h"
-#include "Array.h"
-
-// FIXME: We should get this from cctools
-#define DYLD_CACHE_ADJ_V2_FORMAT 0x7F
-
-namespace dyld3 {
-
-
-bool MachOAnalyzer::isValidMainExecutable(Diagnostics& diag, const char* path, uint64_t sliceLength,
-                                          const GradedArchs& archs, Platform platform) const
-{
-    if ( !this->validMachOForArchAndPlatform(diag, (size_t)sliceLength, path, archs, platform, true) )
-        return false;
-
-    if ( !this->isDynamicExecutable() ) {
-        diag.error("could not use '%s' because it is not an executable, filetype=0x%08X", path, this->filetype);
-        return false;
-    }
-
-    if ( !this->validLinkedit(diag, path) )
-        return false;
-
-    return true;
-}
-
-bool MachOAnalyzer::loadFromBuffer(Diagnostics& diag, const closure::FileSystem& fileSystem,
-                                   const char* path, const GradedArchs& archs, Platform platform,
-                                   closure::LoadedFileInfo& info)
-{
-    // if fat, remap just slice needed
-    bool fatButMissingSlice;
-    const FatFile*       fh = (FatFile*)info.fileContent;
-    uint64_t sliceOffset = info.sliceOffset;
-    uint64_t sliceLen = info.sliceLen;
-    if ( fh->isFatFileWithSlice(diag, info.fileContentLen, archs, info.isOSBinary, sliceOffset, sliceLen, fatButMissingSlice) ) {
-        // unmap anything before slice
-        fileSystem.unloadPartialFile(info, sliceOffset, sliceLen);
-        // Update the info to keep track of the new slice offset.
-        info.sliceOffset = sliceOffset;
-        info.sliceLen = sliceLen;
-    }
-    else if ( diag.hasError() ) {
-        // We must have generated an error in the fat file parsing so use that error
-        fileSystem.unloadFile(info);
-        return false;
-    }
-    else if ( fatButMissingSlice ) {
-        diag.error("missing compatible arch in %s", path);
-        fileSystem.unloadFile(info);
-        return false;
-    }
-
-    const MachOAnalyzer* mh = (MachOAnalyzer*)info.fileContent;
-
-    // validate is mach-o of requested arch and platform
-    if ( !mh->validMachOForArchAndPlatform(diag, (size_t)info.sliceLen, path, archs, platform, info.isOSBinary) ) {
-        fileSystem.unloadFile(info);
-        return false;
-    }
-
-    // if has zero-fill expansion, re-map
-    mh = mh->remapIfZeroFill(diag, fileSystem, info);
-
-    // on error, remove mappings and return nullptr
-    if ( diag.hasError() ) {
-        fileSystem.unloadFile(info);
-        return false;
-    }
-
-    // now that LINKEDIT is at expected offset, finish validation
-    mh->validLinkedit(diag, path);
-
-    // on error, remove mappings and return nullptr
-    if ( diag.hasError() ) {
-        fileSystem.unloadFile(info);
-        return false;
-    }
-
-    return true;
-}
-
-
-closure::LoadedFileInfo MachOAnalyzer::load(Diagnostics& diag, const closure::FileSystem& fileSystem,
-                                            const char* path, const GradedArchs& archs, Platform platform, char realerPath[MAXPATHLEN])
-{
-    // FIXME: This should probably be an assert, but if we happen to have a diagnostic here then something is wrong
-    // above us and we should quickly return instead of doing unnecessary work.
-    if (diag.hasError())
-        return closure::LoadedFileInfo();
-
-    closure::LoadedFileInfo info;
-    if (!fileSystem.loadFile(path, info, realerPath, ^(const char *format, ...) {
-        va_list list;
-        va_start(list, format);
-        diag.error(format, list);
-        va_end(list);
-    })) {
-        return closure::LoadedFileInfo();
-    }
-
-    // If we now have an error, but succeeded, then we must have tried multiple paths, one of which errored, but
-    // then succeeded on a later path.  So clear the error.
-    if (diag.hasError())
-        diag.clearError();
-
-    bool loaded = loadFromBuffer(diag, fileSystem, path, archs, platform, info);
-    if (!loaded)
-        return {};
-    return info;
-}
-
-// for use with already mmap()ed file
-bool MachOAnalyzer::isOSBinary(int fd, uint64_t sliceOffset, uint64_t sliceSize) const
-{
-#ifdef F_GETSIGSINFO
-    if ( fd == -1 )
-        return false;
-
-    uint32_t sigOffset;
-    uint32_t sigSize;
-    if ( !this->hasCodeSignature(sigOffset, sigSize) )
-        return false;
-
-    // register code signature
-    fsignatures_t sigreg;
-    sigreg.fs_file_start = sliceOffset;                // start of mach-o slice in fat file
-    sigreg.fs_blob_start = (void*)(long)sigOffset;     // start of CD in mach-o file
-    sigreg.fs_blob_size  = sigSize;                    // size of CD
-    if ( ::fcntl(fd, F_ADDFILESIGS_RETURN, &sigreg) == -1 )
-        return false;
-
-    // ask if code signature is for something in the OS
-    fgetsigsinfo siginfo = { (off_t)sliceOffset, GETSIGSINFO_PLATFORM_BINARY, 0 };
-    if ( ::fcntl(fd, F_GETSIGSINFO, &siginfo) == -1 )
-        return false;
-
-    return (siginfo.fg_sig_is_platform);
-#else
-    return false;
-#endif
-}
-
-// for use when just the fat_header has been read
-bool MachOAnalyzer::sliceIsOSBinary(int fd, uint64_t sliceOffset, uint64_t sliceSize)
-{
-    if ( fd == -1 )
-        return false;
-
-    // need to mmap() slice so we can find the code signature
-	void* mappedSlice = ::mmap(nullptr, (size_t)sliceSize, PROT_READ, MAP_PRIVATE, fd, sliceOffset);
-	if ( mappedSlice == MAP_FAILED )
-		return false;
-
-    const MachOAnalyzer* ma = (MachOAnalyzer*)mappedSlice;
-    bool result = ma->isOSBinary(fd, sliceOffset, sliceSize);
-    ::munmap(mappedSlice, (size_t)sliceSize);
-
-    return result;
-}
-
-
-
-#if DEBUG
-// only used in debug builds of cache builder to verify segment moves are valid
-void MachOAnalyzer::validateDyldCacheDylib(Diagnostics& diag, const char* path) const
-{
-    validLinkedit(diag, path);
-    validSegments(diag, path, 0xffffffff);
-}
-#endif
-
-uint64_t MachOAnalyzer::mappedSize() const
-{
-    uint64_t vmSpace;
-    bool     hasZeroFill;
-    analyzeSegmentsLayout(vmSpace, hasZeroFill);
-    return vmSpace;
-}
-
-bool MachOAnalyzer::validMachOForArchAndPlatform(Diagnostics& diag, size_t sliceLength, const char* path, const GradedArchs& archs, Platform reqPlatform, bool isOSBinary) const
-{
-    // must start with mach-o magic value
-    if ( (this->magic != MH_MAGIC) && (this->magic != MH_MAGIC_64) ) {
-        diag.error("could not use '%s' because it is not a mach-o file: 0x%08X 0x%08X", path, this->magic, this->cputype);
-        return false;
-    }
-
-    if ( !archs.grade(this->cputype, this->cpusubtype, isOSBinary) ) {
-        diag.error("could not use '%s' because it is not a compatible arch", path);
-        return false;
-    }
-
-    // must be a filetype dyld can load
-    switch ( this->filetype ) {
-        case MH_EXECUTE:
-        case MH_DYLIB:
-        case MH_BUNDLE:
-        case MH_DYLINKER:
-           break;
-#if BUILDING_DYLDINFO || BUILDING_APP_CACHE_UTIL || BUILDING_RUN_STATIC
-        // Allow offline tools to analyze binaries dyld doesn't load
-        case MH_KEXT_BUNDLE:
-        case MH_FILESET:
-            break;
-#endif
-        default:
-            diag.error("could not use '%s' because it is not a dylib, bundle, or executable, filetype=0x%08X", path, this->filetype);
-           return false;
-    }
-
-    // validate load commands structure
-    if ( !this->validLoadCommands(diag, path, sliceLength) ) {
-        return false;
-    }
-
-    // filter out static executables
-    if ( (this->filetype == MH_EXECUTE) && !isDynamicExecutable() ) {
-#if !BUILDING_DYLDINFO && !BUILDING_APP_CACHE_UTIL
-        // dyldinfo should be able to inspect static executables such as the kernel
-        diag.error("could not use '%s' because it is a static executable", path);
-        return false;
-#endif
-    }
-
-    // HACK: If we are asking for no platform, then make sure the binary doesn't have one
-#if BUILDING_DYLDINFO || BUILDING_APP_CACHE_UTIL
-    if ( isFileSet() ) {
-        // A statically linked kernel collection should contain a 0 platform
-        __block bool foundPlatform = false;
-        __block bool foundBadPlatform = false;
-        forEachSupportedPlatform(^(Platform platform, uint32_t minOS, uint32_t sdk) {
-            foundPlatform = true;
-            if ( platform != Platform::unknown ) {
-                foundBadPlatform = true;
-            }
-        });
-        if (!foundPlatform) {
-            diag.error("could not use '%s' because we expected it to have a platform", path);
-            return false;
-        }
-        if (foundBadPlatform) {
-            diag.error("could not use '%s' because is has the wrong platform", path);
-            return false;
-        }
-    } else if ( reqPlatform == Platform::unknown ) {
-        // Unfortunately the static kernel has a platform, but kext's don't, so we can't
-        // verify the platform of the kernel.
-        if ( !isStaticExecutable() ) {
-            __block bool foundPlatform = false;
-            forEachSupportedPlatform(^(Platform platform, uint32_t minOS, uint32_t sdk) {
-                foundPlatform = true;
-            });
-            if (foundPlatform) {
-                diag.error("could not use '%s' because we expected it to have no platform", path);
-                return false;
-            }
-        }
-    } else
-#endif
-    if ( !this->loadableIntoProcess(reqPlatform, path) ) {
-        diag.error("could not use '%s' because it was not built for platform %s", path, MachOFile::platformName(reqPlatform));
-        return false;
-    }
-
-    // validate dylib loads
-    if ( !validEmbeddedPaths(diag, reqPlatform, path) )
-        return false;
-
-    // validate segments
-    if ( !validSegments(diag, path, sliceLength) )
-        return false;
-
-    // validate entry
-    if ( this->filetype == MH_EXECUTE ) {
-        if ( !validMain(diag, path) )
-            return false;
-    }
-
-    // further validations done in validLinkedit()
-
-    return true;
-}
-
-bool MachOAnalyzer::validLinkedit(Diagnostics& diag, const char* path) const
-{
-    // validate LINKEDIT layout
-    if ( !validLinkeditLayout(diag, path) )
-        return false;
-
-    // rdar://75492733 (enforce that binaries built against Fall2021 SDK have a LC_UUID)
-    if ( enforceFormat(Malformed::noUUID) ) {
-        if ( !hasLoadCommand(LC_UUID) ) {
-            diag.error("missing LC_UUID");
-            return false;
-        }
-    }
-
-    if ( hasLoadCommand(LC_DYLD_CHAINED_FIXUPS) ) {
-        if ( !validChainedFixupsInfo(diag, path) )
-            return false;
-    }
-#if SUPPORT_ARCH_arm64e
-    else if ( (this->cputype == CPU_TYPE_ARM64) && (this->maskedCpuSubtype() == CPU_SUBTYPE_ARM64E) ) {
-        if ( !validChainedFixupsInfoOldArm64e(diag, path) )
-            return false;
-    }
-#endif
-    else {
-        // validate rebasing info
-        if ( !validRebaseInfo(diag, path) )
-            return false;
-
-       // validate binding info
-        if ( !validBindInfo(diag, path) )
-            return false;
-    }
-
-    return true;
-}
-
-bool MachOAnalyzer::validLoadCommands(Diagnostics& diag, const char* path, size_t fileLen) const
-{
-    // check load command don't exceed file length
-    if ( this->sizeofcmds + machHeaderSize() > fileLen ) {
-        diag.error("in '%s' load commands exceed length of file", path);
-        return false;
-    }
-
-    // walk all load commands and sanity check them
-    Diagnostics walkDiag;
-    forEachLoadCommand(walkDiag, ^(const load_command* cmd, bool& stop) {});
-    if ( walkDiag.hasError() ) {
-#if BUILDING_CACHE_BUILDER
-        diag.error("in '%s' %s", path, walkDiag.errorMessage().c_str());
-#else
-        diag.error("in '%s' %s", path, walkDiag.errorMessage());
-#endif
-        return false;
-    }
-
-    // check load commands fit in TEXT segment
-    __block bool foundTEXT    = false;
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        if ( strcmp(info.segName, "__TEXT") == 0 ) {
-            foundTEXT = true;
-            if ( this->sizeofcmds + machHeaderSize() > info.fileSize ) {
-                diag.error("in '%s' load commands exceed length of __TEXT segment", path);
-            }
-            if ( info.fileOffset != 0 ) {
-                diag.error("in '%s' __TEXT segment not start of mach-o", path);
-            }
-            stop = true;
-        }
-    });
-    if ( !diag.noError() && !foundTEXT ) {
-        diag.error("in '%s' __TEXT segment not found", path);
-        return false;
-    }
-
-    return true;
-}
-
-const MachOAnalyzer* MachOAnalyzer::remapIfZeroFill(Diagnostics& diag, const closure::FileSystem& fileSystem, closure::LoadedFileInfo& info) const
-{
-    uint64_t vmSpaceRequired;
-    bool     hasZeroFill;
-    analyzeSegmentsLayout(vmSpaceRequired, hasZeroFill);
-
-    if ( hasZeroFill ) {
-        vm_address_t newMappedAddr;
-        if ( ::vm_allocate(mach_task_self(), &newMappedAddr, (size_t)vmSpaceRequired, VM_FLAGS_ANYWHERE) != 0 ) {
-            diag.error("vm_allocate failure");
-            return nullptr;
-        }
-
-        // re-map each segment read-only, with runtime layout
-#if BUILDING_APP_CACHE_UTIL
-        // The auxKC is mapped with __DATA first, so we need to get either the __DATA or __TEXT depending on what is earliest
-        __block uint64_t baseAddress = ~0ULL;
-        forEachSegment(^(const SegmentInfo& segmentInfo, bool& stop) {
-            baseAddress = std::min(baseAddress, segmentInfo.vmAddr);
-        });
-        uint64_t textSegVMAddr = preferredLoadAddress();
-#else
-        uint64_t baseAddress = preferredLoadAddress();
-#endif
-
-        forEachSegment(^(const SegmentInfo& segmentInfo, bool& stop) {
-            if ( (segmentInfo.fileSize != 0) && (segmentInfo.vmSize != 0) ) {
-                kern_return_t r = vm_copy(mach_task_self(), (vm_address_t)((long)info.fileContent+segmentInfo.fileOffset), (vm_size_t)segmentInfo.fileSize, (vm_address_t)(newMappedAddr+segmentInfo.vmAddr-baseAddress));
-                if ( r != KERN_SUCCESS ) {
-                    diag.error("vm_copy() failure");
-                    stop = true;
-                }
-            }
-        });
-        if ( diag.noError() ) {
-            // remove original mapping and return new mapping
-            fileSystem.unloadFile(info);
-
-            // make the new mapping read-only
-            ::vm_protect(mach_task_self(), newMappedAddr, (vm_size_t)vmSpaceRequired, false, VM_PROT_READ);
-
-#if BUILDING_APP_CACHE_UTIL
-            if ( textSegVMAddr != baseAddress ) {
-                info.unload = [](const closure::LoadedFileInfo& info) {
-                    // Unloading binaries where __DATA is first requires working out the real range of the binary
-                    // The fileContent points at the mach_header, not the actaul start of the file content, unfortunately.
-                    const MachOAnalyzer* ma = (const MachOAnalyzer*)info.fileContent;
-                    __block uint64_t baseAddress = ~0ULL;
-                    ma->forEachSegment(^(const SegmentInfo& segInfo, bool& stop) {
-                        baseAddress = std::min(baseAddress, segInfo.vmAddr);
-                    });
-                    uint64_t textSegVMAddr = ma->preferredLoadAddress();
-
-                    uint64_t basePointerOffset = textSegVMAddr - baseAddress;
-                    uint8_t* bufferStart = (uint8_t*)info.fileContent - basePointerOffset;
-                    ::vm_deallocate(mach_task_self(), (vm_address_t)bufferStart, (size_t)info.fileContentLen);
-                };
-
-                // And update the file content to the new location
-                info.fileContent = (const void*)(newMappedAddr + textSegVMAddr - baseAddress);
-                info.fileContentLen = vmSpaceRequired;
-                return (const MachOAnalyzer*)info.fileContent;
-            }
-#endif
-
-            // Set vm_deallocate as the unload method.
-            info.unload = [](const closure::LoadedFileInfo& info) {
-                ::vm_deallocate(mach_task_self(), (vm_address_t)info.fileContent, (size_t)info.fileContentLen);
-            };
-
-            // And update the file content to the new location
-            info.fileContent = (const void*)newMappedAddr;
-            info.fileContentLen = vmSpaceRequired;
-            return (const MachOAnalyzer*)info.fileContent;
-        }
-        else {
-            // new mapping failed, return old mapping with an error in diag
-            ::vm_deallocate(mach_task_self(), newMappedAddr, (size_t)vmSpaceRequired);
-            return nullptr;
-        }
-    }
-
-    return this;
-}
-
-void MachOAnalyzer::analyzeSegmentsLayout(uint64_t& vmSpace, bool& hasZeroFill) const
-{
-    __block bool     writeExpansion = false;
-    __block uint64_t lowestVmAddr   = 0xFFFFFFFFFFFFFFFFULL;
-    __block uint64_t highestVmAddr  = 0;
-    __block uint64_t sumVmSizes     = 0;
-    forEachSegment(^(const SegmentInfo& segmentInfo, bool& stop) {
-        if ( strcmp(segmentInfo.segName, "__PAGEZERO") == 0 )
-            return;
-        if ( segmentInfo.writable() && (segmentInfo.fileSize !=  segmentInfo.vmSize) )
-            writeExpansion = true; // zerofill at end of __DATA
-        if ( segmentInfo.vmSize == 0 ) {
-            // Always zero fill if we have zero-sized segments
-            writeExpansion = true;
-        }
-        if ( segmentInfo.vmAddr < lowestVmAddr )
-            lowestVmAddr = segmentInfo.vmAddr;
-        if ( segmentInfo.vmAddr+segmentInfo.vmSize > highestVmAddr )
-            highestVmAddr = segmentInfo.vmAddr+segmentInfo.vmSize;
-        sumVmSizes += segmentInfo.vmSize;
-    });
-    uint64_t totalVmSpace = (highestVmAddr - lowestVmAddr);
-    // LINKEDIT vmSize is not required to be a multiple of page size.  Round up if that is the case
-    const uint64_t pageSize = uses16KPages() ? 0x4000 : 0x1000;
-    totalVmSpace = (totalVmSpace + (pageSize - 1)) & ~(pageSize - 1);
-    bool hasHole = (totalVmSpace != sumVmSizes); // segments not contiguous
-
-    // The aux KC may have __DATA first, in which case we always want to vm_copy to the right place
-    bool hasOutOfOrderSegments = false;
-#if BUILDING_APP_CACHE_UTIL
-    uint64_t textSegVMAddr = preferredLoadAddress();
-    hasOutOfOrderSegments = textSegVMAddr != lowestVmAddr;
-#endif
-
-    vmSpace     = totalVmSpace;
-    hasZeroFill = writeExpansion || hasHole || hasOutOfOrderSegments;
-}
-
-bool MachOAnalyzer::enforceFormat(Malformed kind) const
-{
-    // TODO: Add a mapping from generic releases to platform versions
-#if BUILDING_DYLDINFO || BUILDING_APP_CACHE_UTIL || BUILDING_RUN_STATIC
-    // HACK: If we are the kernel, we have a different format to enforce
-    if ( isFileSet() ) {
-        bool result = false;
-        switch (kind) {
-        case Malformed::linkeditOrder:
-        case Malformed::linkeditAlignment:
-        case Malformed::dyldInfoAndlocalRelocs:
-            result = true;
-            break;
-        case Malformed::segmentOrder:
-        // The aux KC has __DATA first
-            result = false;
-            break;
-        case Malformed::linkeditPermissions:
-        case Malformed::executableData:
-        case Malformed::writableData:
-        case Malformed::codeSigAlignment:
-        case Malformed::sectionsAddrRangeWithinSegment:
-        case Malformed::loaderPathsAreReal:
-        case Malformed::mainExecInDyldCache:
-            result = true;
-            break;
-        case Malformed::noLinkedDylibs:
-        case Malformed::textPermissions:
-            // The kernel has its own __TEXT_EXEC for executable memory
-            result = false;
-            break;
-        case Malformed::noUUID:
-            result = true;
-            break;
-        }
-        return result;
-    }
-
-    if ( isStaticExecutable() ) {
-        bool result = false;
-        switch (kind) {
-        case Malformed::linkeditOrder:
-        case Malformed::linkeditAlignment:
-        case Malformed::dyldInfoAndlocalRelocs:
-            result = true;
-            break;
-        case Malformed::segmentOrder:
-            result = false;
-            break;
-        case Malformed::linkeditPermissions:
-        case Malformed::executableData:
-        case Malformed::codeSigAlignment:
-        case Malformed::textPermissions:
-        case Malformed::sectionsAddrRangeWithinSegment:
-        case Malformed::loaderPathsAreReal:
-        case Malformed::mainExecInDyldCache:
-            result = true;
-            break;
-        case Malformed::noLinkedDylibs:
-        case Malformed::writableData:
-        case Malformed::noUUID:
-            // The kernel has __DATA_CONST marked as r/o
-            result = false;
-            break;
-        }
-        return result;
-    }
-
-#endif
-
-    __block bool result = false;
-    forEachSupportedPlatform(^(Platform platform, uint32_t minOS, uint32_t sdk) {
-        switch (platform) {
-        case Platform::macOS:
-            switch (kind) {
-            case Malformed::linkeditOrder:
-            case Malformed::linkeditAlignment:
-            case Malformed::dyldInfoAndlocalRelocs:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x000A0E00) // macOS 10.14
-                    result = true;
-                break;
-            case Malformed::segmentOrder:
-            case Malformed::linkeditPermissions:
-            case Malformed::textPermissions:
-            case Malformed::executableData:
-            case Malformed::writableData:
-            case Malformed::codeSigAlignment:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x000A0F00) // macOS 10.15
-                    result = true;
-                break;
-            case Malformed::sectionsAddrRangeWithinSegment:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x000A1000) // macOS 10.16
-                    result = true;
-                break;
-            case Malformed::noLinkedDylibs:
-            case Malformed::loaderPathsAreReal:
-            case Malformed::mainExecInDyldCache:
-            case Malformed::noUUID:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x000C0000) // macOS 12.0
-                    result = true;
-                break;
-            }
-            break;
-        case Platform::iOS:
-        case Platform::tvOS:
-        case Platform::iOSMac:
-            switch (kind) {
-            case Malformed::linkeditOrder:
-            case Malformed::dyldInfoAndlocalRelocs:
-            case Malformed::textPermissions:
-            case Malformed::executableData:
-            case Malformed::writableData:
-                result = true;
-                break;
-            case Malformed::linkeditAlignment:
-            case Malformed::segmentOrder:
-            case Malformed::linkeditPermissions:
-            case Malformed::codeSigAlignment:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x000D0000) // iOS 13
-                    result = true;
-                break;
-            case Malformed::sectionsAddrRangeWithinSegment:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x000E0000) // iOS 14
-                    result = true;
-                break;
-            case Malformed::noLinkedDylibs:
-            case Malformed::loaderPathsAreReal:
-            case Malformed::mainExecInDyldCache:
-            case Malformed::noUUID:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x000F0000) // iOS 15
-                    result = true;
-            }
-            break;
-        case Platform::watchOS:
-            switch (kind) {
-            case Malformed::linkeditOrder:
-            case Malformed::dyldInfoAndlocalRelocs:
-            case Malformed::textPermissions:
-            case Malformed::executableData:
-            case Malformed::writableData:
-                result = true;
-                break;
-            case Malformed::linkeditAlignment:
-            case Malformed::segmentOrder:
-            case Malformed::linkeditPermissions:
-            case Malformed::codeSigAlignment:
-            case Malformed::sectionsAddrRangeWithinSegment:
-            case Malformed::noLinkedDylibs:
-            case Malformed::loaderPathsAreReal:
-            case Malformed::mainExecInDyldCache:
-            case Malformed::noUUID:
-                // enforce these checks on new binaries only
-                if (sdk >= 0x00080000) // watchOS 8
-                    result = true;
-                break;
-            }
-            break;
-        case Platform::driverKit:
-            result = true;
-            break;
-        default:
-            result = true;
-            break;
-        }
-    });
-    // if binary is so old, there is no platform info, don't enforce malformed errors
-    return result;
-}
-
-bool MachOAnalyzer::validEmbeddedPaths(Diagnostics& diag, Platform platform, const char* path) const
-{
-    __block int         index = 1;
-    __block bool        allGood = true;
-    __block int         dependentsCount = 0;
-    __block const char* installName = nullptr;
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        const dylib_command* dylibCmd;
-        const rpath_command* rpathCmd;
-        switch ( cmd->cmd ) {
-            case LC_ID_DYLIB:
-            case LC_LOAD_DYLIB:
-            case LC_LOAD_WEAK_DYLIB:
-            case LC_REEXPORT_DYLIB:
-            case LC_LOAD_UPWARD_DYLIB:
-                dylibCmd = (dylib_command*)cmd;
-                if ( dylibCmd->dylib.name.offset > cmd->cmdsize ) {
-                    diag.error("in '%s' load command #%d name offset (%u) outside its size (%u)", path, index, dylibCmd->dylib.name.offset, cmd->cmdsize);
-                    stop = true;
-                    allGood = false;
-                }
-                else {
-                    bool foundEnd = false;
-                    const char* start = (char*)dylibCmd + dylibCmd->dylib.name.offset;
-                    const char* end   = (char*)dylibCmd + cmd->cmdsize;
-                    for (const char* s=start; s < end; ++s) {
-                        if ( *s == '\0' ) {
-                            foundEnd = true;
-                            break;
-                        }
-                    }
-                    if ( !foundEnd ) {
-                        diag.error("in '%s' load command #%d string extends beyond end of load command", path, index);
-                        stop = true;
-                        allGood = false;
-                    }
-                }
-                if ( cmd->cmd  == LC_ID_DYLIB )
-                    installName = (char*)dylibCmd + dylibCmd->dylib.name.offset;
-                else
-                    ++dependentsCount;
-                break;
-            case LC_RPATH:
-                rpathCmd = (rpath_command*)cmd;
-                if ( rpathCmd->path.offset > cmd->cmdsize ) {
-                    diag.error("in '%s' load command #%d path offset (%u) outside its size (%u)", path, index, rpathCmd->path.offset, cmd->cmdsize);
-                    stop = true;
-                    allGood = false;
-                }
-                else {
-                    bool foundEnd = false;
-                    const char* start = (char*)rpathCmd + rpathCmd->path.offset;
-                    const char* end   = (char*)rpathCmd + cmd->cmdsize;
-                    for (const char* s=start; s < end; ++s) {
-                        if ( *s == '\0' ) {
-                            foundEnd = true;
-                            break;
-                        }
-                    }
-                    if ( !foundEnd ) {
-                        diag.error("in '%s' load command #%d string extends beyond end of load command", path, index);
-                        stop = true;
-                        allGood = false;
-                    }
-                }
-                break;
-        }
-        ++index;
-    });
-    if ( !allGood )
-        return false;
-
-    if ( this->filetype == MH_DYLIB ) {
-        if ( installName == nullptr ) {
-            diag.error("in '%s' MH_DYLIB is missing LC_ID_DYLIB", path);
-            return false;
-        }
-
-        if ( this->enforceFormat(MachOAnalyzer::Malformed::loaderPathsAreReal) ) {
-            // new binary, so check that part after @xpath/ is real (not symlinks)
-            if ( (strncmp(installName, "@loader_path/", 13) == 0) || (strncmp(installName, "@executable_path/", 17) == 0) ) {
-                if ( const char* s = strchr(installName, '/') ) {
-                    while (strncmp(s, "/..", 3) == 0)
-                        s += 3;
-                    const char* trailingInstallPath = s;
-                    const char* trailingRealPath = &path[strlen(path)-strlen(trailingInstallPath)];
-                    if (strcmp(trailingRealPath, trailingInstallPath) != 0 ) {
-                        diag.error("install name '%s' contains symlinks", installName);
-                        return false;
-                    }
-                }
-            }
-        }
-    }
-    else {
-        if ( installName != nullptr ) {
-            diag.error("in '%s' LC_ID_DYLIB found in non-MH_DYLIB", path);
-            return false;
-        }
-    }
-
-    // all new binaries must link with something else
-    if ( (dependentsCount == 0) && enforceFormat(Malformed::noLinkedDylibs) ) {
-        const char* libSystemDir = this->builtForPlatform(Platform::driverKit, true) ? "/System/DriverKit/usr/lib/system/" : "/usr/lib/system/";
-        // except for dylibs in libSystem.dylib which are ok to link with nothing (they are on bottom)
-        bool isNotLibSystem = (installName == nullptr) || (strncmp(installName, libSystemDir, strlen(libSystemDir)) != 0);
-
-        if ( this->isDyldManaged() && isNotLibSystem ) {
-            diag.error("in '%s' missing LC_LOAD_DYLIB (must link with at least libSystem.dylib)", path);
-            return false;
-        }
-    }
-
-    return true;
-}
-
-bool MachOAnalyzer::validSegments(Diagnostics& diag, const char* path, size_t fileLen) const
-{
-    // check segment load command size
-    __block bool badSegmentLoadCommand = false;
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        if ( cmd->cmd == LC_SEGMENT_64 ) {
-            const segment_command_64* seg = (segment_command_64*)cmd;
-            int32_t sectionsSpace = cmd->cmdsize - sizeof(segment_command_64);
-            if ( sectionsSpace < 0 ) {
-               diag.error("in '%s' load command size too small for LC_SEGMENT_64", path);
-               badSegmentLoadCommand = true;
-               stop = true;
-            }
-            else if ( (sectionsSpace % sizeof(section_64)) != 0 ) {
-               diag.error("in '%s' segment load command size 0x%X will not fit whole number of sections", path, cmd->cmdsize);
-               badSegmentLoadCommand = true;
-               stop = true;
-            }
-            else if ( sectionsSpace != (seg->nsects * sizeof(section_64)) ) {
-               diag.error("in '%s' load command size 0x%X does not match nsects %d", path, cmd->cmdsize, seg->nsects);
-               badSegmentLoadCommand = true;
-               stop = true;
-            }
-            else if ( greaterThanAddOrOverflow(seg->fileoff, seg->filesize, fileLen) ) {
-                diag.error("in '%s' segment load command content extends beyond end of file", path);
-                badSegmentLoadCommand = true;
-                stop = true;
-            }
-            else if ( (seg->filesize > seg->vmsize) && ((seg->vmsize != 0) || ((seg->flags & SG_NORELOC) == 0)) ) {
-                // <rdar://problem/19986776> dyld should support non-allocatable __LLVM segment
-                diag.error("in '%s' segment '%s' filesize exceeds vmsize", path, seg->segname);
-                badSegmentLoadCommand = true;
-                stop = true;
-            }
-        }
-        else if ( cmd->cmd == LC_SEGMENT ) {
-            const segment_command* seg = (segment_command*)cmd;
-            int32_t sectionsSpace = cmd->cmdsize - sizeof(segment_command);
-            if ( sectionsSpace < 0 ) {
-               diag.error("in '%s' load command size too small for LC_SEGMENT", path);
-               badSegmentLoadCommand = true;
-               stop = true;
-            }
-            else if ( (sectionsSpace % sizeof(section)) != 0 ) {
-               diag.error("in '%s' segment load command size 0x%X will not fit whole number of sections", path, cmd->cmdsize);
-               badSegmentLoadCommand = true;
-               stop = true;
-            }
-            else if ( sectionsSpace != (seg->nsects * sizeof(section)) ) {
-               diag.error("in '%s' load command size 0x%X does not match nsects %d", path, cmd->cmdsize, seg->nsects);
-               badSegmentLoadCommand = true;
-               stop = true;
-            }
-            else if ( (seg->filesize > seg->vmsize) && ((seg->vmsize != 0) || ((seg->flags & SG_NORELOC) == 0)) ) {
-                // <rdar://problem/19986776> dyld should support non-allocatable __LLVM segment
-                diag.error("in '%s' segment  '%s' filesize exceeds vmsize", path, seg->segname);
-                badSegmentLoadCommand = true;
-                stop = true;
-            }
-        }
-    });
-     if ( badSegmentLoadCommand )
-         return false;
-
-    // check mapping permissions of segments
-    __block bool badPermissions = false;
-    __block bool badSize        = false;
-    __block bool hasTEXT        = false;
-    __block bool hasLINKEDIT    = false;
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        if ( strcmp(info.segName, "__TEXT") == 0 ) {
-            if ( (info.protections != (VM_PROT_READ|VM_PROT_EXECUTE)) && enforceFormat(Malformed::textPermissions) ) {
-                diag.error("in '%s' __TEXT segment permissions is not 'r-x'", path);
-                badPermissions = true;
-                stop = true;
-            }
-            hasTEXT = true;
-        }
-        else if ( strcmp(info.segName, "__LINKEDIT") == 0 ) {
-            if ( (info.protections != VM_PROT_READ) && enforceFormat(Malformed::linkeditPermissions) ) {
-                diag.error("in '%s' __LINKEDIT segment permissions is not 'r--'", path);
-                badPermissions = true;
-                stop = true;
-            }
-            hasLINKEDIT = true;
-        }
-        else if ( (info.protections & 0xFFFFFFF8) != 0 ) {
-            diag.error("in '%s' %s segment permissions has invalid bits set", path, info.segName);
-            badPermissions = true;
-            stop = true;
-        }
-        if ( greaterThanAddOrOverflow(info.fileOffset, info.fileSize, fileLen) ) {
-            diag.error("in '%s' %s segment content extends beyond end of file", path, info.segName);
-            badSize = true;
-            stop = true;
-        }
-        if ( is64() ) {
-            if ( info.vmAddr+info.vmSize < info.vmAddr ) {
-                diag.error("in '%s' %s segment vm range wraps", path, info.segName);
-                badSize = true;
-                stop = true;
-            }
-       }
-       else {
-            if ( (uint32_t)(info.vmAddr+info.vmSize) < (uint32_t)(info.vmAddr) ) {
-                diag.error("in '%s' %s segment vm range wraps", path, info.segName);
-                badSize = true;
-                stop = true;
-            }
-       }
-    });
-    if ( badPermissions || badSize )
-        return false;
-    if ( !hasTEXT ) {
-        diag.error("in '%s' missing __TEXT segment", path);
-        return false;
-    }
-    if ( !hasLINKEDIT ) {
-       diag.error("in '%s' missing __LINKEDIT segment", path);
-       return false;
-    }
-
-    // check for overlapping segments
-    __block bool badSegments = false;
-    forEachSegment(^(const SegmentInfo& info1, bool& stop1) {
-        uint64_t seg1vmEnd   = info1.vmAddr + info1.vmSize;
-        uint64_t seg1FileEnd = info1.fileOffset + info1.fileSize;
-        forEachSegment(^(const SegmentInfo& info2, bool& stop2) {
-            if ( info1.segIndex == info2.segIndex )
-                return;
-            uint64_t seg2vmEnd   = info2.vmAddr + info2.vmSize;
-            uint64_t seg2FileEnd = info2.fileOffset + info2.fileSize;
-            if ( ((info2.vmAddr <= info1.vmAddr) && (seg2vmEnd > info1.vmAddr) && (seg1vmEnd > info1.vmAddr )) || ((info2.vmAddr >= info1.vmAddr ) && (info2.vmAddr < seg1vmEnd) && (seg2vmEnd > info2.vmAddr)) ) {
-                diag.error("in '%s' segment %s vm range overlaps segment %s", path, info1.segName, info2.segName);
-                badSegments = true;
-                stop1 = true;
-                stop2 = true;
-            }
-             if ( ((info2.fileOffset  <= info1.fileOffset) && (seg2FileEnd > info1.fileOffset) && (seg1FileEnd > info1.fileOffset)) || ((info2.fileOffset  >= info1.fileOffset) && (info2.fileOffset  < seg1FileEnd) && (seg2FileEnd > info2.fileOffset )) ) {
-                 if ( !inDyldCache() ) {
-                     // HACK: Split shared caches might put the __TEXT in a SubCache, then the __DATA in a later SubCache.
-                     // The file offsets are in to each SubCache file, which means that they might overlap
-                     // For now we have no choice but to disable this error
-                     diag.error("in '%s' segment %s file content overlaps segment %s", path, info1.segName, info2.segName);
-                     badSegments = true;
-                     stop1 = true;
-                     stop2 = true;
-                 }
-            }
-            if ( (info1.segIndex < info2.segIndex) && !stop1 ) {
-                if ( (info1.vmAddr > info2.vmAddr) || ((info1.fileOffset > info2.fileOffset ) && (info1.fileOffset != 0) && (info2.fileOffset  != 0)) ){
-                    if ( !inDyldCache() && enforceFormat(Malformed::segmentOrder) && !isStaticExecutable() ) {
-                        //<rdar://80084852> whitelist go libraries __DWARF segments
-                        if ( (strcmp(info1.segName, "__DWARF") != 0 && strcmp(info2.segName, "__DWARF") != 0) ) {
-                            // dyld cache __DATA_* segments are moved around
-                            // The static kernel also has segments with vmAddr's before __TEXT
-                            diag.error("in '%s' segment load commands out of order with respect to layout for %s and %s", path, info1.segName, info2.segName);
-                            badSegments = true;
-                            stop1 = true;
-                            stop2 = true;
-                        }
-                    }
-                }
-            }
-        });
-    });
-    if ( badSegments )
-        return false;
-
-    // check sections are within segment
-    __block bool badSections = false;
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        if ( cmd->cmd == LC_SEGMENT_64 ) {
-            const segment_command_64* seg = (segment_command_64*)cmd;
-            const section_64* const sectionsStart = (section_64*)((char*)seg + sizeof(struct segment_command_64));
-            const section_64* const sectionsEnd   = &sectionsStart[seg->nsects];
-            for (const section_64* sect=sectionsStart; (sect < sectionsEnd); ++sect) {
-                if ( (int64_t)(sect->size) < 0 ) {
-                    diag.error("in '%s' section '%s' size too large 0x%llX", path, sect->sectname, sect->size);
-                    badSections = true;
-                }
-                else if ( sect->addr < seg->vmaddr ) {
-                    diag.error("in '%s' section '%s' start address 0x%llX is before containing segment's address 0x%0llX", path, sect->sectname, sect->addr, seg->vmaddr);
-                    badSections = true;
-                }
-                else if ( sect->addr+sect->size > seg->vmaddr+seg->vmsize ) {
-                    bool ignoreError = !enforceFormat(Malformed::sectionsAddrRangeWithinSegment);
-#if BUILDING_APP_CACHE_UTIL
-                    if ( (seg->vmsize == 0) && !strcmp(seg->segname, "__CTF") )
-                        ignoreError = true;
-#endif
-                    if ( !ignoreError ) {
-                        diag.error("in '%s' section '%s' end address 0x%llX is beyond containing segment's end address 0x%0llX", path, sect->sectname, sect->addr+sect->size, seg->vmaddr+seg->vmsize);
-                        badSections = true;
-                    }
-                }
-            }
-        }
-        else if ( cmd->cmd == LC_SEGMENT ) {
-            const segment_command* seg = (segment_command*)cmd;
-            const section* const sectionsStart = (section*)((char*)seg + sizeof(struct segment_command));
-            const section* const sectionsEnd   = &sectionsStart[seg->nsects];
-            for (const section* sect=sectionsStart; !stop && (sect < sectionsEnd); ++sect) {
-               if ( (int64_t)(sect->size) < 0 ) {
-                    diag.error("in '%s' section %s size too large 0x%X", path, sect->sectname, sect->size);
-                    badSections = true;
-                }
-                else if ( sect->addr < seg->vmaddr ) {
-                    diag.error("in '%s' section %s start address 0x%X is before containing segment's address 0x%0X", path,  sect->sectname, sect->addr, seg->vmaddr);
-                    badSections = true;
-                }
-                else if ( sect->addr+sect->size > seg->vmaddr+seg->vmsize ) {
-                    diag.error("in '%s' section %s end address 0x%X is beyond containing segment's end address 0x%0X", path, sect->sectname, sect->addr+sect->size, seg->vmaddr+seg->vmsize);
-                    badSections = true;
-                }
-            }
-        }
-    });
-
-    return !badSections;
-}
-
-
-bool MachOAnalyzer::validMain(Diagnostics& diag, const char* path) const
-{
-    // FIXME: Also check that dylibs with inDyldCache() set are actaully in the cache
-    if ( this->inDyldCache() && MachOAnalyzer::enforceFormat(Malformed::mainExecInDyldCache) ) {
-        diag.error("MH_EXECUTE is in dyld shared cache");
-        return false;
-    }
-    const char* executableTextSegmentName = "__TEXT";
-#if BUILDING_APP_CACHE_UTIL
-    // The kernel has __start in __TEXT_EXEC, or for x86_64 it's __HIB
-    if ( isStaticExecutable() ) {
-        if ( isArch("x86_64") || isArch("x86_64h") )
-            executableTextSegmentName = "__HIB";
-        else
-            executableTextSegmentName = "__TEXT_EXEC";
-    }
-#endif
-
-    __block uint64_t textSegStartAddr = 0;
-    __block uint64_t textSegStartSize = 0;
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        if ( strcmp(info.segName, executableTextSegmentName) == 0 ) {
-            textSegStartAddr = info.vmAddr;
-            textSegStartSize = info.vmSize;
-            stop = true;
-       }
-    });
-
-    __block int mainCount   = 0;
-    __block int threadCount = 0;
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        entry_point_command* mainCmd;
-        uint64_t startAddress;
-        switch (cmd->cmd) {
-            case LC_MAIN:
-                ++mainCount;
-                mainCmd = (entry_point_command*)cmd;
-                if ( mainCmd->entryoff >= textSegStartSize ) {
-                    startAddress = preferredLoadAddress() + mainCmd->entryoff;
-                    __block bool foundSegment = false;
-                    forEachSegment(^(const SegmentInfo& info, bool& stopSegment) {
-                        // Skip segments which don't contain this address
-                        if ( (startAddress < info.vmAddr) || (startAddress >= info.vmAddr+info.vmSize) )
-                            return;
-                        foundSegment = true;
-                        if ( (info.protections & VM_PROT_EXECUTE) == 0 )
-                            diag.error("LC_MAIN points to non-executable segment");
-                        stopSegment = true;
-                    });
-                    if (!foundSegment)
-                        diag.error("LC_MAIN entryoff is out of range");
-                    stop = true;
-                }
-                break;
-            case LC_UNIXTHREAD:
-                ++threadCount;
-                startAddress = entryAddrFromThreadCmd((thread_command*)cmd);
-                if ( startAddress == 0 ) {
-                    diag.error("LC_UNIXTHREAD not valid for arch %s", archName());
-                    stop = true;
-                }
-#if BUILDING_DYLDINFO
-                else if ( isStaticExecutable() ) {
-                    __block bool foundSegment = false;
-                    forEachSegment(^(const SegmentInfo& info, bool& stopSegment) {
-                        // Skip segments which don't contain this address
-                        if ( (startAddress < info.vmAddr) || (startAddress >= info.vmAddr+info.vmSize) )
-                            return;
-                        foundSegment = true;
-                        if ( (info.protections & VM_PROT_EXECUTE) == 0 )
-                            diag.error("LC_UNIXTHREAD points to non-executable segment");
-                        stopSegment = true;
-                    });
-                    if (!foundSegment)
-                        diag.error("LC_UNIXTHREAD entry is out of range");
-                    stop = true;
-                }
-#endif
-                else if ( (startAddress < textSegStartAddr) || (startAddress >= textSegStartAddr+textSegStartSize) ) {
-                    diag.error("LC_UNIXTHREAD entry not in %s segment", executableTextSegmentName);
-                    stop = true;
-                }
-                break;
-        }
-    });
-    if ( diag.hasError() )
-        return false;
-
-    if ( this->builtForPlatform(Platform::driverKit) ) {
-        if ( mainCount + threadCount == 0 )
-            return true;
-        diag.error("LC_MAIN not allowed for driverkit");
-        return false;
-    }
-
-    if ( mainCount+threadCount == 1 )
-        return true;
-
-    if ( mainCount + threadCount == 0 )
-        diag.error("missing LC_MAIN or LC_UNIXTHREAD");
-    else
-        diag.error("only one LC_MAIN or LC_UNIXTHREAD is allowed");
-    return false;
-}
-
-
-namespace {
-    struct LinkEditContentChunk
-    {
-        const char* name;
-        uint32_t    alignment;
-        uint32_t    fileOffsetStart;
-        uint32_t    size;
-
-        // only have a few chunks, so bubble sort is ok.  Don't use libc's qsort because it may call malloc
-        static void sort(LinkEditContentChunk array[], unsigned long count)
-        {
-            for (unsigned i=0; i < count-1; ++i) {
-                bool done = true;
-                for (unsigned j=0; j < count-i-1; ++j) {
-                    if ( array[j].fileOffsetStart > array[j+1].fileOffsetStart ) {
-                        LinkEditContentChunk temp = array[j];
-                        array[j]   = array[j+1];
-                        array[j+1] = temp;
-                        done = false;
-                    }
-                }
-                if ( done )
-                    break;
-            }
-        }
-    };
-} // anonymous namespace
-
-
-
-bool MachOAnalyzer::validLinkeditLayout(Diagnostics& diag, const char* path) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return false;
-    const uint32_t ptrSize = pointerSize();
-
-    // build vector of all blobs in LINKEDIT
-    LinkEditContentChunk blobs[32];
-    LinkEditContentChunk* bp = blobs;
-    if ( leInfo.dyldInfo != nullptr ) {
-        if ( leInfo.dyldInfo->rebase_size != 0 )
-            *bp++ = {"rebase opcodes",          ptrSize, leInfo.dyldInfo->rebase_off, leInfo.dyldInfo->rebase_size};
-        if ( leInfo.dyldInfo->bind_size != 0 )
-            *bp++ = {"bind opcodes",            ptrSize, leInfo.dyldInfo->bind_off, leInfo.dyldInfo->bind_size};
-        if ( leInfo.dyldInfo->weak_bind_size != 0 )
-            *bp++ = {"weak bind opcodes",       ptrSize, leInfo.dyldInfo->weak_bind_off, leInfo.dyldInfo->weak_bind_size};
-        if ( leInfo.dyldInfo->lazy_bind_size != 0 )
-            *bp++ = {"lazy bind opcodes",       ptrSize, leInfo.dyldInfo->lazy_bind_off, leInfo.dyldInfo->lazy_bind_size};
-        if ( leInfo.dyldInfo->export_size!= 0 )
-            *bp++ = {"exports trie",            ptrSize, leInfo.dyldInfo->export_off, leInfo.dyldInfo->export_size};
-    }
-    if ( leInfo.exportsTrie != nullptr ) {
-        if ( leInfo.exportsTrie->datasize != 0 )
-            *bp++ = {"exports trie",            ptrSize, leInfo.exportsTrie->dataoff, leInfo.exportsTrie->datasize};
-    }
-    if ( leInfo.chainedFixups != nullptr ) {
-        if ( leInfo.chainedFixups->datasize != 0 )
-            *bp++ = {"chained fixups",          ptrSize, leInfo.chainedFixups->dataoff, leInfo.chainedFixups->datasize};
-    }
-    
-    if ( leInfo.dynSymTab != nullptr ) {
-        if ( leInfo.dynSymTab->nlocrel != 0 )
-            *bp++ = {"local relocations",       ptrSize, leInfo.dynSymTab->locreloff, static_cast<uint32_t>(leInfo.dynSymTab->nlocrel*sizeof(relocation_info))};
-        if ( leInfo.dynSymTab->nextrel != 0 )
-            *bp++ = {"external relocations",    ptrSize, leInfo.dynSymTab->extreloff, static_cast<uint32_t>(leInfo.dynSymTab->nextrel*sizeof(relocation_info))};
-        if ( leInfo.dynSymTab->nindirectsyms != 0 )
-            *bp++ = {"indirect symbol table",   4,       leInfo.dynSymTab->indirectsymoff, leInfo.dynSymTab->nindirectsyms*4};
-    }
-    if ( leInfo.splitSegInfo != nullptr ) {
-        if ( leInfo.splitSegInfo->datasize != 0 )
-            *bp++ = {"shared cache info",       ptrSize, leInfo.splitSegInfo->dataoff, leInfo.splitSegInfo->datasize};
-    }
-    if ( leInfo.functionStarts != nullptr ) {
-        if ( leInfo.functionStarts->datasize != 0 )
-            *bp++ = {"function starts",         ptrSize, leInfo.functionStarts->dataoff, leInfo.functionStarts->datasize};
-    }
-    if ( leInfo.dataInCode != nullptr ) {
-        if ( leInfo.dataInCode->datasize != 0 )
-            *bp++ = {"data in code",            ptrSize, leInfo.dataInCode->dataoff, leInfo.dataInCode->datasize};
-    }
-    if ( leInfo.symTab != nullptr ) {
-        if ( leInfo.symTab->nsyms != 0 )
-            *bp++ = {"symbol table",            ptrSize, leInfo.symTab->symoff, static_cast<uint32_t>(leInfo.symTab->nsyms*(ptrSize == 8 ? sizeof(nlist_64) : sizeof(struct nlist)))};
-        if ( leInfo.symTab->strsize != 0 )
-            *bp++ = {"symbol table strings",    1,       leInfo.symTab->stroff, leInfo.symTab->strsize};
-    }
-    if ( leInfo.codeSig != nullptr ) {
-        if ( leInfo.codeSig->datasize != 0 )
-            *bp++ = {"code signature",          ptrSize, leInfo.codeSig->dataoff, leInfo.codeSig->datasize};
-    }
-
-    // check for bad combinations
-    if ( (leInfo.dyldInfo != nullptr) && (leInfo.dyldInfo->cmd == LC_DYLD_INFO_ONLY) && (leInfo.dynSymTab != nullptr) ) {
-        if ( (leInfo.dynSymTab->nlocrel != 0) && enforceFormat(Malformed::dyldInfoAndlocalRelocs) ) {
-            diag.error("in '%s' malformed mach-o contains LC_DYLD_INFO_ONLY and local relocations", path);
-            return false;
-        }
-        if ( leInfo.dynSymTab->nextrel != 0 ) {
-            diag.error("in '%s' malformed mach-o contains LC_DYLD_INFO_ONLY and external relocations", path);
-            return false;
-        }
-    }
-
-    bool checkMissingDyldInfo = true;
-#if BUILDING_DYLDINFO || BUILDING_APP_CACHE_UTIL
-    checkMissingDyldInfo = !isFileSet() && !isStaticExecutable() && !isKextBundle();
-#endif
-    if ( (leInfo.dyldInfo == nullptr) && (leInfo.dynSymTab == nullptr) && checkMissingDyldInfo ) {
-        diag.error("in '%s' malformed mach-o misssing LC_DYLD_INFO and LC_DYSYMTAB", path);
-        return false;
-    }
-
-    // FIXME: Remove this hack
-#if BUILDING_APP_CACHE_UTIL
-    if ( isFileSet() )
-        return true;
-#endif
-
-    const unsigned long blobCount = bp - blobs;
-    if ( blobCount == 0 ) {
-        diag.error("in '%s' malformed mach-o missing LINKEDIT", path);
-        return false;
-    }
-
-    uint32_t linkeditFileEnd = leInfo.layout.linkeditFileOffset + leInfo.layout.linkeditFileSize;
-
-
-    // sort blobs by file-offset and error on overlaps
-    LinkEditContentChunk::sort(blobs, blobCount);
-    uint32_t     prevEnd = leInfo.layout.linkeditFileOffset;
-    const char*  prevName = "start of LINKEDIT";
-    for (unsigned long i=0; i < blobCount; ++i) {
-        const LinkEditContentChunk& blob = blobs[i];
-        if ( blob.fileOffsetStart < prevEnd ) {
-            diag.error("in '%s' LINKEDIT overlap of %s and %s", path, prevName, blob.name);
-            return false;
-        }
-        if (greaterThanAddOrOverflow(blob.fileOffsetStart, blob.size, linkeditFileEnd)) {
-            diag.error("in '%s' LINKEDIT content '%s' extends beyond end of segment", path, blob.name);
-            return false;
-        }
-        if ( (blob.fileOffsetStart & (blob.alignment-1)) != 0 ) {
-            // <rdar://problem/51115705> relax code sig alignment for pre iOS13
-            Malformed kind = (strcmp(blob.name, "code signature") == 0) ? Malformed::codeSigAlignment : Malformed::linkeditAlignment;
-            if ( enforceFormat(kind) )
-                diag.error("in '%s' mis-aligned LINKEDIT content '%s'", path, blob.name);
-        }
-        prevEnd  = blob.fileOffsetStart + blob.size;
-        prevName = blob.name;
-    }
-
-    // Check for invalid symbol table sizes
-    if ( leInfo.symTab != nullptr ) {
-        if ( leInfo.symTab->nsyms > 0x10000000 ) {
-            diag.error("in '%s' malformed mach-o image: symbol table too large", path);
-            return false;
-        }
-        if ( leInfo.dynSymTab != nullptr ) {
-            // validate indirect symbol table
-            if ( leInfo.dynSymTab->nindirectsyms != 0 ) {
-                if ( leInfo.dynSymTab->nindirectsyms > 0x10000000 ) {
-                    diag.error("in '%s' malformed mach-o image: indirect symbol table too large", path);
-                    return false;
-                }
-            }
-            if ( (leInfo.dynSymTab->nlocalsym > leInfo.symTab->nsyms) || (leInfo.dynSymTab->ilocalsym > leInfo.symTab->nsyms) ) {
-                diag.error("in '%s' malformed mach-o image: indirect symbol table local symbol count exceeds total symbols", path);
-                return false;
-            }
-            if ( leInfo.dynSymTab->ilocalsym + leInfo.dynSymTab->nlocalsym < leInfo.dynSymTab->ilocalsym  ) {
-                diag.error("in '%s' malformed mach-o image: indirect symbol table local symbol count wraps", path);
-                return false;
-            }
-            if ( (leInfo.dynSymTab->nextdefsym > leInfo.symTab->nsyms) || (leInfo.dynSymTab->iextdefsym > leInfo.symTab->nsyms) ) {
-                diag.error("in '%s' malformed mach-o image: indirect symbol table extern symbol count exceeds total symbols", path);
-                return false;
-            }
-            if ( leInfo.dynSymTab->iextdefsym + leInfo.dynSymTab->nextdefsym < leInfo.dynSymTab->iextdefsym  ) {
-                diag.error("in '%s' malformed mach-o image: indirect symbol table extern symbol count wraps", path);
-                return false;
-            }
-            if ( (leInfo.dynSymTab->nundefsym > leInfo.symTab->nsyms) || (leInfo.dynSymTab->iundefsym > leInfo.symTab->nsyms) ) {
-                diag.error("in '%s' malformed mach-o image: indirect symbol table undefined symbol count exceeds total symbols", path);
-                return false;
-            }
-            if ( leInfo.dynSymTab->iundefsym + leInfo.dynSymTab->nundefsym < leInfo.dynSymTab->iundefsym  ) {
-                diag.error("in '%s' malformed mach-o image: indirect symbol table undefined symbol count wraps", path);
-                return false;
-            }
-        }
-    }
-
-    return true;
-}
-
-
-
-bool MachOAnalyzer::invalidRebaseState(Diagnostics& diag, const char* opcodeName, const char* path, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                                      bool segIndexSet, uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset, Rebase kind) const
-{
-    if ( !segIndexSet ) {
-        diag.error("in '%s' %s missing preceding REBASE_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB", path, opcodeName);
-        return true;
-    }
-    if ( segmentIndex >= leInfo.layout.linkeditSegIndex )  {
-        diag.error("in '%s' %s segment index %d too large", path, opcodeName, segmentIndex);
-        return true;
-    }
-    if ( segmentOffset > (segments[segmentIndex].vmSize-ptrSize) ) {
-        diag.error("in '%s' %s current segment offset 0x%08llX beyond segment size (0x%08llX)", path, opcodeName, segmentOffset, segments[segmentIndex].vmSize);
-        return true;
-    }
-    switch ( kind )  {
-        case Rebase::pointer32:
-        case Rebase::pointer64:
-            if ( !segments[segmentIndex].writable() && enforceFormat(Malformed::writableData) ) {
-                diag.error("in '%s' %s pointer rebase is in non-writable segment", path, opcodeName);
-                return true;
-            }
-            if ( segments[segmentIndex].executable() && enforceFormat(Malformed::executableData) ) {
-                diag.error("in '%s' %s pointer rebase is in executable segment", path, opcodeName);
-                return true;
-            }
-            break;
-        case Rebase::textAbsolute32:
-        case Rebase::textPCrel32:
-            if ( !segments[segmentIndex].textRelocs ) {
-                diag.error("in '%s' %s text rebase is in segment that does not support text relocations", path, opcodeName);
-                return true;
-            }
-            if ( segments[segmentIndex].writable() ) {
-                diag.error("in '%s' %s text rebase is in writable segment", path, opcodeName);
-                return true;
-            }
-            if ( !segments[segmentIndex].executable() ) {
-                diag.error("in '%s' %s pointer rebase is in non-executable segment", path, opcodeName);
-                return true;
-            }
-            break;
-        case Rebase::unknown:
-            diag.error("in '%s' %s unknown rebase type", path, opcodeName);
-            return true;
-    }
-    return false;
-}
-
-
-void MachOAnalyzer::getAllSegmentsInfos(Diagnostics& diag, SegmentInfo segments[]) const
-{
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        segments[info.segIndex] = info;
-    });
-}
-
-
-bool MachOAnalyzer::validRebaseInfo(Diagnostics& diag, const char* path) const
-{
-    forEachRebase(diag, ^(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                          bool segIndexSet, uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset, Rebase kind, bool& stop) {
-        if ( invalidRebaseState(diag, opcodeName, path, leInfo, segments, segIndexSet, ptrSize, segmentIndex, segmentOffset, kind) )
-            stop = true;
-    });
-    return diag.noError();
-}
-
-
-void MachOAnalyzer::forEachTextRebase(Diagnostics& diag, void (^handler)(uint64_t runtimeOffset, bool& stop)) const
-{
-    __block bool     startVmAddrSet = false;
-    __block uint64_t startVmAddr    = 0;
-    forEachRebase(diag, ^(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                          bool segIndexSet, uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset, Rebase kind, bool& stop) {
-        if ( kind != Rebase::textAbsolute32 )
-            return;
-        if ( !startVmAddrSet ) {
-            for (int i=0; i <= segmentIndex; ++i) {
-                if ( strcmp(segments[i].segName, "__TEXT") == 0 ) {
-                    startVmAddr = segments[i].vmAddr;
-                    startVmAddrSet = true;
-                    break;
-               }
-            }
-        }
-        uint64_t rebaseVmAddr  = segments[segmentIndex].vmAddr + segmentOffset;
-        uint64_t runtimeOffset = rebaseVmAddr - startVmAddr;
-        handler(runtimeOffset, stop);
-    });
-}
-
-void MachOAnalyzer::forEachRebase(Diagnostics& diag, void (^callback)(uint64_t runtimeOffset, bool isLazyPointerRebase, bool& stop)) const
-{
-    __block bool     startVmAddrSet = false;
-    __block uint64_t startVmAddr    = 0;
-    __block uint64_t lpVmAddr       = 0;
-    __block uint64_t lpEndVmAddr    = 0;
-    __block uint64_t shVmAddr       = 0;
-    __block uint64_t shEndVmAddr    = 0;
-    forEachSection(^(const dyld3::MachOAnalyzer::SectionInfo& info, bool malformedSectionRange, bool &stop) {
-        if ( (info.sectFlags & SECTION_TYPE) == S_LAZY_SYMBOL_POINTERS ) {
-            lpVmAddr    = info.sectAddr;
-            lpEndVmAddr = info.sectAddr + info.sectSize;
-        }
-        else if ( (info.sectFlags & S_ATTR_PURE_INSTRUCTIONS) && (strcmp(info.sectName, "__stub_helper") == 0) ) {
-            shVmAddr    = info.sectAddr;
-            shEndVmAddr = info.sectAddr + info.sectSize;
-        }
-    });
-    forEachRebase(diag, ^(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                          bool segIndexSet, uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset, Rebase kind, bool& stop) {
-        switch ( kind ) {
-            case Rebase::unknown:
-                return;
-            case Rebase::pointer32:
-            case Rebase::pointer64:
-                // We only handle these kinds for now.
-                break;
-            case Rebase::textPCrel32:
-            case Rebase::textAbsolute32:
-                return;
-        }
-        if ( !startVmAddrSet ) {
-            for (int i=0; i < segmentIndex; ++i) {
-                if ( strcmp(segments[i].segName, "__TEXT") == 0 ) {
-                    startVmAddr = segments[i].vmAddr;
-                    startVmAddrSet = true;
-                    break;
-               }
-            }
-        }
-        uint64_t rebaseVmAddr  = segments[segmentIndex].vmAddr + segmentOffset;
-        bool isLazyPointerRebase = false;
-        if ( (rebaseVmAddr >= lpVmAddr) && (rebaseVmAddr < lpEndVmAddr) ) {
-            // rebase is in lazy pointer section
-            uint64_t lpValue = 0;
-            if ( ptrSize == 8 )
-                lpValue = *((uint64_t*)(rebaseVmAddr-startVmAddr+(uint8_t*)this));
-            else
-                lpValue = *((uint32_t*)(rebaseVmAddr-startVmAddr+(uint8_t*)this));
-            if ( (lpValue >= shVmAddr) && (lpValue < shEndVmAddr) ) {
-                // content is into stub_helper section
-                uint64_t lpTargetImageOffset = lpValue - startVmAddr;
-                const uint8_t* helperContent = (uint8_t*)this + lpTargetImageOffset;
-                bool isLazyStub = contentIsRegularStub(helperContent);
-                // ignore rebases for normal lazy pointers, but leave rebase for resolver helper stub
-                if ( isLazyStub )
-                    isLazyPointerRebase = true;
-            }
-            else {
-                // if lazy pointer does not point into stub_helper, then it points to weak-def symbol and we need rebase
-            }
-        }
-        uint64_t runtimeOffset = rebaseVmAddr - startVmAddr;
-        callback(runtimeOffset, isLazyPointerRebase, stop);
-    });
-}
-
-
-
-void MachOAnalyzer::forEachRebase(Diagnostics& diag, bool ignoreLazyPointers, void (^handler)(uint64_t runtimeOffset, bool& stop)) const
-{
-    forEachRebase(diag, ^(uint64_t runtimeOffset, bool isLazyPointerRebase, bool& stop) {
-        if ( isLazyPointerRebase && ignoreLazyPointers )
-            return;
-        handler(runtimeOffset, stop);
-    });
-}
-
-bool MachOAnalyzer::contentIsRegularStub(const uint8_t* helperContent) const
-{
-    switch (this->cputype) {
-        case CPU_TYPE_X86_64:
-            return ( (helperContent[0] == 0x68) && (helperContent[5] == 0xE9) ); // push $xxx / JMP pcRel
-        case CPU_TYPE_I386:
-            return ( (helperContent[0] == 0x68) && (helperContent[5] == 0xFF) && (helperContent[2] == 0x26) ); // push $xxx / JMP *pcRel
-        case CPU_TYPE_ARM:
-            return ( (helperContent[0] == 0x00) && (helperContent[1] == 0xC0) && (helperContent[2] == 0x9F) && (helperContent[3] == 0xE5) ); // ldr  ip, [pc, #0]
-        case CPU_TYPE_ARM64:
-            return ( (helperContent[0] == 0x50) && (helperContent[1] == 0x00) && (helperContent[2] == 0x00) && (helperContent[3] == 0x18) ); // ldr w16, L0
-
-    }
-    return false;
-}
-
-
-void MachOAnalyzer::forEachRebase(Diagnostics& diag,
-                                 void (^handler)(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                                                 bool segIndexSet, uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset,
-                                                 Rebase kind, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return;
-
-    const Rebase pointerRebaseKind = is64() ? Rebase::pointer64 : Rebase::pointer32;
-
-    if ( leInfo.dyldInfo != nullptr ) {
-        const uint8_t* const start = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->rebase_off);
-        const uint8_t* const end   = start + leInfo.dyldInfo->rebase_size;
-        const uint8_t* p           = start;
-        const uint32_t ptrSize     = pointerSize();
-        Rebase   kind = Rebase::unknown;
-        int      segIndex = 0;
-        uint64_t segOffset = 0;
-        uint64_t count;
-        uint64_t skip;
-        bool     segIndexSet = false;
-        bool     stop = false;
-        while ( !stop && diag.noError() && (p < end) ) {
-            uint8_t immediate = *p & REBASE_IMMEDIATE_MASK;
-            uint8_t opcode = *p & REBASE_OPCODE_MASK;
-            ++p;
-            switch (opcode) {
-                case REBASE_OPCODE_DONE:
-                    // Allow some padding, in case rebases were somehow aligned to 16-bytes in size
-                    if ( (end - p) > 15 )
-                        diag.error("rebase opcodes terminated early at offset %d of %d", (int)(p-start), (int)(end-start));
-                    stop = true;
-                    break;
-                case REBASE_OPCODE_SET_TYPE_IMM:
-                    switch ( immediate ) {
-                        case REBASE_TYPE_POINTER:
-                            kind = pointerRebaseKind;
-                            break;
-                        case REBASE_TYPE_TEXT_ABSOLUTE32:
-                            kind = Rebase::textAbsolute32;
-                            break;
-                        case REBASE_TYPE_TEXT_PCREL32:
-                            kind = Rebase::textPCrel32;
-                            break;
-                        default:
-                            kind = Rebase::unknown;
-                            break;
-                    }
-                    break;
-                case REBASE_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                    segIndex = immediate;
-                    segOffset = read_uleb128(diag, p, end);
-                    segIndexSet = true;
-                    break;
-                case REBASE_OPCODE_ADD_ADDR_ULEB:
-                    segOffset += read_uleb128(diag, p, end);
-                    break;
-                case REBASE_OPCODE_ADD_ADDR_IMM_SCALED:
-                    segOffset += immediate*ptrSize;
-                    break;
-                case REBASE_OPCODE_DO_REBASE_IMM_TIMES:
-                    for (int i=0; i < immediate; ++i) {
-                        handler("REBASE_OPCODE_DO_REBASE_IMM_TIMES", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                        segOffset += ptrSize;
-                        if ( stop )
-                            break;
-                    }
-                    break;
-                case REBASE_OPCODE_DO_REBASE_ULEB_TIMES:
-                    count = read_uleb128(diag, p, end);
-                    for (uint32_t i=0; i < count; ++i) {
-                        handler("REBASE_OPCODE_DO_REBASE_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                        segOffset += ptrSize;
-                        if ( stop )
-                            break;
-                    }
-                    break;
-                case REBASE_OPCODE_DO_REBASE_ADD_ADDR_ULEB:
-                    handler("REBASE_OPCODE_DO_REBASE_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                    segOffset += read_uleb128(diag, p, end) + ptrSize;
-                    break;
-                case REBASE_OPCODE_DO_REBASE_ULEB_TIMES_SKIPPING_ULEB:
-                    count = read_uleb128(diag, p, end);
-                    if ( diag.hasError() )
-                        break;
-                    skip = read_uleb128(diag, p, end);
-                    for (uint32_t i=0; i < count; ++i) {
-                        handler("REBASE_OPCODE_DO_REBASE_ULEB_TIMES_SKIPPING_ULEB", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                        segOffset += skip + ptrSize;
-                        if ( stop )
-                            break;
-                    }
-                    break;
-                default:
-                    diag.error("unknown rebase opcode 0x%02X", opcode);
-            }
-        }
-        return;
-    }
-
-    if ( leInfo.chainedFixups != nullptr ) {
-        // binary uses chained fixups, so do nothing
-        // The kernel collections need to support both chained and classic relocations
-        // If we are anything other than a kernel collection, then return here as we won't have
-        // anything else to do.
-        if ( !isFileSet() )
-            return;
-    }
-
-    if ( leInfo.dynSymTab != nullptr ) {
-        // old binary, walk relocations
-        const uint64_t                  relocsStartAddress = localRelocBaseAddress(segmentsInfo, leInfo.layout.linkeditSegIndex);
-        const relocation_info* const    relocsStart = (relocation_info*)getLinkEditContent(leInfo.layout, leInfo.dynSymTab->locreloff);
-        const relocation_info* const    relocsEnd   = &relocsStart[leInfo.dynSymTab->nlocrel];
-        bool                            stop = false;
-        const uint8_t                   relocSize = (is64() ? 3 : 2);
-        const uint8_t                   ptrSize   = pointerSize();
-        STACK_ALLOC_OVERFLOW_SAFE_ARRAY(relocation_info, relocs, 2048);
-        for (const relocation_info* reloc=relocsStart; (reloc < relocsEnd) && !stop; ++reloc) {
-            if ( reloc->r_length != relocSize ) {
-                bool shouldEmitError = true;
-#if BUILDING_APP_CACHE_UTIL
-                if ( usesClassicRelocationsInKernelCollection() && (reloc->r_length == 2) && (relocSize == 3) )
-                    shouldEmitError = false;
-#endif
-                if ( shouldEmitError ) {
-                    diag.error("local relocation has wrong r_length");
-                    break;
-                }
-            }
-            if ( reloc->r_type != 0 ) { // 0 == X86_64_RELOC_UNSIGNED == GENERIC_RELOC_VANILLA ==  ARM64_RELOC_UNSIGNED
-                diag.error("local relocation has wrong r_type");
-                break;
-            }
-            relocs.push_back(*reloc);
-        }
-        if ( !relocs.empty() ) {
-            sortRelocations(relocs);
-            for (relocation_info reloc : relocs) {
-                uint32_t addrOff = reloc.r_address;
-                uint32_t segIndex  = 0;
-                uint64_t segOffset = 0;
-                uint64_t addr = 0;
-#if BUILDING_APP_CACHE_UTIL
-                // xnu for x86_64 has __HIB mapped before __DATA, so offsets appear to be
-                // negative
-                if ( isStaticExecutable() || isFileSet() ) {
-                    addr = relocsStartAddress + (int32_t)addrOff;
-                } else {
-                    addr = relocsStartAddress + addrOff;
-                }
-#else
-                addr = relocsStartAddress + addrOff;
-#endif
-                if ( segIndexAndOffsetForAddress(addr, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-                    Rebase kind = (reloc.r_length == 2) ? Rebase::pointer32 : Rebase::pointer64;
-                    if ( this->cputype == CPU_TYPE_I386 ) {
-                        if ( segmentsInfo[segIndex].executable() )
-                            kind = Rebase::textAbsolute32;
-                    }
-                    handler("local relocation", leInfo, segmentsInfo, true, ptrSize, segIndex, segOffset, kind, stop);
-                }
-                else {
-                    diag.error("local relocation has out of range r_address");
-                    break;
-                }
-            }
-        }
-        // then process indirect symbols
-        forEachIndirectPointer(diag, false,
-                               ^(uint64_t address, bool bind, int bindLibOrdinal,
-                                 const char* bindSymbolName, bool bindWeakImport, bool bindLazy, bool selfModifyingStub, bool& indStop) {
-            if ( bind )
-               return;
-            uint32_t segIndex  = 0;
-            uint64_t segOffset = 0;
-            if ( segIndexAndOffsetForAddress(address, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-                handler("local relocation", leInfo, segmentsInfo, true, ptrSize, segIndex, segOffset, pointerRebaseKind, indStop);
-            }
-            else {
-                diag.error("local relocation has out of range r_address");
-                indStop = true;
-            }
-        });
-    }
-}
-
-bool MachOAnalyzer::segIndexAndOffsetForAddress(uint64_t addr, const SegmentInfo segmentsInfos[], uint32_t segCount, uint32_t& segIndex, uint64_t& segOffset) const
-{
-    for (uint32_t i=0; i < segCount; ++i) {
-        if ( (segmentsInfos[i].vmAddr <= addr) && (addr < segmentsInfos[i].vmAddr+segmentsInfos[i].vmSize) ) {
-            segIndex  = i;
-            segOffset = addr - segmentsInfos[i].vmAddr;
-            return true;
-        }
-    }
-    return false;
-}
-
-uint64_t MachOAnalyzer::localRelocBaseAddress(const SegmentInfo segmentsInfos[], uint32_t segCount) const
-{
-    if ( isArch("x86_64") || isArch("x86_64h") ) {
-#if BUILDING_APP_CACHE_UTIL
-        if ( isKextBundle() ) {
-            // for kext bundles the reloc base address starts at __TEXT segment
-            return segmentsInfos[0].vmAddr;
-        }
-#endif
-        // for all other kinds, the x86_64 reloc base address starts at first writable segment (usually __DATA)
-        for (uint32_t i=0; i < segCount; ++i) {
-            if ( segmentsInfos[i].writable() )
-                return segmentsInfos[i].vmAddr;
-        }
-    }
-    return segmentsInfos[0].vmAddr;
-}
-
-uint64_t MachOAnalyzer::externalRelocBaseAddress(const SegmentInfo segmentsInfos[], uint32_t segCount) const
-{
-    // Dyld caches are too large for a raw r_address, so everything is an offset from the base address
-    if ( inDyldCache() ) {
-        return preferredLoadAddress();
-    }
-
-#if BUILDING_APP_CACHE_UTIL
-    if ( isKextBundle() ) {
-        // for kext bundles the reloc base address starts at __TEXT segment
-        return preferredLoadAddress();
-    }
-#endif
-
-    if ( isArch("x86_64") || isArch("x86_64h") ) {
-        // for x86_64 reloc base address starts at first writable segment (usually __DATA)
-        for (uint32_t i=0; i < segCount; ++i) {
-            if ( segmentsInfos[i].writable() )
-                return segmentsInfos[i].vmAddr;
-        }
-    }
-    // For everyone else we start at 0
-    return 0;
-}
-
-
-
-void MachOAnalyzer::forEachIndirectPointer(Diagnostics& diag, bool supportPrivateExternsWorkaround,
-                                           void (^handler)(uint64_t pointerAddress, bool bind, int bindLibOrdinal, const char* bindSymbolName,
-                                                           bool bindWeakImport, bool bindLazy, bool selfModifyingStub, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    // find lazy and non-lazy pointer sections
-    const bool              is64Bit                  = is64();
-    const uint32_t* const   indirectSymbolTable      = (uint32_t*)getLinkEditContent(leInfo.layout, leInfo.dynSymTab->indirectsymoff);
-    const uint32_t          indirectSymbolTableCount = leInfo.dynSymTab->nindirectsyms;
-    const uint32_t          ptrSize                  = pointerSize();
-    const void*             symbolTable              = getLinkEditContent(leInfo.layout, leInfo.symTab->symoff);
-    const struct nlist_64*  symbols64                = (nlist_64*)symbolTable;
-    const struct nlist*     symbols32                = (struct nlist*)symbolTable;
-    const char*             stringPool               = (char*)getLinkEditContent(leInfo.layout, leInfo.symTab->stroff);
-    uint32_t                symCount                 = leInfo.symTab->nsyms;
-    uint32_t                poolSize                 = leInfo.symTab->strsize;
-    __block bool            stop                     = false;
-
-    // Old kexts put S_LAZY_SYMBOL_POINTERS on the __got section, even if they didn't have indirect symbols to prcess.
-    // In that case, skip the loop as there shouldn't be anything to process
-    if ( (indirectSymbolTableCount == 0) && isKextBundle() )
-        return;
-
-    forEachSection(^(const dyld3::MachOAnalyzer::SectionInfo& sectInfo, bool malformedSectionRange, bool& sectionStop) {
-        uint8_t  sectionType  = (sectInfo.sectFlags & SECTION_TYPE);
-        bool selfModifyingStub = (sectionType == S_SYMBOL_STUBS) && (sectInfo.sectFlags & S_ATTR_SELF_MODIFYING_CODE) && (sectInfo.reserved2 == 5) && (this->cputype == CPU_TYPE_I386);
-        if ( (sectionType != S_LAZY_SYMBOL_POINTERS) && (sectionType != S_NON_LAZY_SYMBOL_POINTERS) && !selfModifyingStub )
-            return;
-        if ( (flags & S_ATTR_SELF_MODIFYING_CODE) && !selfModifyingStub ) {
-            diag.error("S_ATTR_SELF_MODIFYING_CODE section type only valid in old i386 binaries");
-            sectionStop = true;
-            return;
-        }
-        uint32_t elementSize = selfModifyingStub ? sectInfo.reserved2 : ptrSize;
-        uint32_t elementCount = (uint32_t)(sectInfo.sectSize/elementSize);
-        if ( greaterThanAddOrOverflow(sectInfo.reserved1, elementCount, indirectSymbolTableCount) ) {
-            diag.error("section %s overflows indirect symbol table", sectInfo.sectName);
-            sectionStop = true;
-            return;
-        }
-
-        for (uint32_t i=0; (i < elementCount) && !stop; ++i) {
-            uint32_t symNum = indirectSymbolTable[sectInfo.reserved1 + i];
-            if ( symNum == INDIRECT_SYMBOL_ABS )
-                continue;
-            if ( symNum == INDIRECT_SYMBOL_LOCAL ) {
-                handler(sectInfo.sectAddr+i*elementSize, false, 0, "", false, false, false, stop);
-                continue;
-            }
-            if ( symNum > symCount ) {
-                diag.error("indirect symbol[%d] = %d which is invalid symbol index", sectInfo.reserved1 + i, symNum);
-                sectionStop = true;
-                return;
-            }
-            uint16_t n_desc = is64Bit ? symbols64[symNum].n_desc : symbols32[symNum].n_desc;
-            uint8_t  n_type     = is64Bit ? symbols64[symNum].n_type : symbols32[symNum].n_type;
-            uint32_t libOrdinal = libOrdinalFromDesc(n_desc);
-            uint32_t strOffset = is64Bit ? symbols64[symNum].n_un.n_strx : symbols32[symNum].n_un.n_strx;
-            if ( strOffset > poolSize ) {
-               diag.error("symbol[%d] string offset out of range", sectInfo.reserved1 + i);
-                sectionStop = true;
-                return;
-            }
-            const char* symbolName  = stringPool + strOffset;
-            bool        weakImport  = (n_desc & N_WEAK_REF);
-            bool        lazy        = (sectionType == S_LAZY_SYMBOL_POINTERS);
-#if BUILDING_DYLD && TARGET_OS_OSX && __x86_64__
-            if ( lazy && ((n_type & N_PEXT) != 0) ) {
-                // don't know why the static linker did not eliminate the internal reference to a private extern definition
-                // As this is private extern, we know the symbol lookup will fail.  We also know that this is a lazy-bind, and so
-                // there is a corresponding rebase.  The rebase will be run later, and will slide whatever value is in here.
-                // So lets change the value in this slot, and let the existing rebase slide it for us
-                // Note we only want to change the value in memory once, before rebases are applied.  We don't want to accidentally
-                // change it again later.
-                if ( supportPrivateExternsWorkaround ) {
-                    uintptr_t* ptr = (uintptr_t*)((uint8_t*)(sectInfo.sectAddr+i*elementSize) + this->getSlide());
-                    uint64_t n_value = is64Bit ? symbols64[symNum].n_value : symbols32[symNum].n_value;
-                    *ptr = (uintptr_t)n_value;
-                }
-                continue;
-            }
-#endif
-            // Handle defined weak def symbols which need to get a special ordinal
-            if ( ((n_type & N_TYPE) == N_SECT) && ((n_type & N_EXT) != 0) && ((n_desc & N_WEAK_DEF) != 0) )
-                libOrdinal = BIND_SPECIAL_DYLIB_WEAK_LOOKUP;
-            handler(sectInfo.sectAddr+i*elementSize, true, libOrdinal, symbolName, weakImport, lazy, selfModifyingStub, stop);
-        }
-        sectionStop = stop;
-    });
-}
-
-int MachOAnalyzer::libOrdinalFromDesc(uint16_t n_desc) const
-{
-    // -flat_namespace is always flat lookup
-    if ( (this->flags & MH_TWOLEVEL) == 0 )
-        return BIND_SPECIAL_DYLIB_FLAT_LOOKUP;
-
-    // extract byte from undefined symbol entry
-    int libIndex = GET_LIBRARY_ORDINAL(n_desc);
-    switch ( libIndex ) {
-        case SELF_LIBRARY_ORDINAL:
-            return BIND_SPECIAL_DYLIB_SELF;
-
-        case DYNAMIC_LOOKUP_ORDINAL:
-            return BIND_SPECIAL_DYLIB_FLAT_LOOKUP;
-
-        case EXECUTABLE_ORDINAL:
-            return BIND_SPECIAL_DYLIB_MAIN_EXECUTABLE;
-    }
-
-    return libIndex;
-}
-
-bool MachOAnalyzer::validBindInfo(Diagnostics& diag, const char* path) const
-{
-    forEachBind(diag, ^(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                         bool segIndexSet, bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                         uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset,
-                         uint8_t type, const char* symbolName, bool weakImport, bool lazyBind, uint64_t addend, bool& stop) {
-        if ( invalidBindState(diag, opcodeName, path, leInfo, segments, segIndexSet, libraryOrdinalSet, dylibCount,
-                              libOrdinal, ptrSize, segmentIndex, segmentOffset, type, symbolName) ) {
-            stop = true;
-        }
-    }, ^(const char* symbolName) {
-    });
-    return diag.noError();
-}
-
-bool MachOAnalyzer::invalidBindState(Diagnostics& diag, const char* opcodeName, const char* path, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                                    bool segIndexSet,  bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal, uint32_t ptrSize,
-                                    uint8_t segmentIndex, uint64_t segmentOffset, uint8_t type, const char* symbolName) const
-{
-    if ( !segIndexSet ) {
-        diag.error("in '%s' %s missing preceding BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB", path, opcodeName);
-        return true;
-    }
-    if ( segmentIndex >= leInfo.layout.linkeditSegIndex )  {
-        diag.error("in '%s' %s segment index %d too large", path, opcodeName, segmentIndex);
-        return true;
-    }
-    if ( segmentOffset > (segments[segmentIndex].vmSize-ptrSize) ) {
-        diag.error("in '%s' %s current segment offset 0x%08llX beyond segment size (0x%08llX)", path, opcodeName, segmentOffset, segments[segmentIndex].vmSize);
-        return true;
-    }
-    if ( symbolName == NULL ) {
-        diag.error("in '%s' %s missing preceding BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM", path, opcodeName);
-        return true;
-    }
-    if ( !libraryOrdinalSet ) {
-        diag.error("in '%s' %s missing preceding BIND_OPCODE_SET_DYLIB_ORDINAL", path, opcodeName);
-        return true;
-    }
-    if ( libOrdinal > (int)dylibCount ) {
-        diag.error("in '%s' %s has library ordinal too large (%d) max (%d)", path, opcodeName, libOrdinal, dylibCount);
-        return true;
-    }
-    if ( libOrdinal < BIND_SPECIAL_DYLIB_WEAK_LOOKUP ) {
-        diag.error("in '%s' %s has unknown library special ordinal (%d)", path, opcodeName, libOrdinal);
-        return true;
-    }
-    switch ( type )  {
-        case BIND_TYPE_POINTER:
-            if ( !segments[segmentIndex].writable() ) {
-                diag.error("in '%s' %s pointer bind is in non-writable segment", path, opcodeName);
-                return true;
-            }
-            if ( segments[segmentIndex].executable() && enforceFormat(Malformed::executableData) ) {
-                diag.error("in '%s' %s pointer bind is in executable segment", path, opcodeName);
-                return true;
-            }
-            break;
-        case BIND_TYPE_TEXT_ABSOLUTE32:
-        case BIND_TYPE_TEXT_PCREL32: {
-            // Text relocations are permitted in x86_64 kexts
-            bool forceAllowTextRelocs = false;
-#if BUILDING_APP_CACHE_UTIL
-            if ( isKextBundle() && (isArch("x86_64") || isArch("x86_64h")) )
-                forceAllowTextRelocs = true;
-#endif
-            if ( !forceAllowTextRelocs && !segments[segmentIndex].textRelocs ) {
-                diag.error("in '%s' %s text bind is in segment that does not support text relocations", path, opcodeName);
-                return true;
-            }
-            if ( segments[segmentIndex].writable() ) {
-                diag.error("in '%s' %s text bind is in writable segment", path, opcodeName);
-                return true;
-            }
-            if ( !segments[segmentIndex].executable() ) {
-                diag.error("in '%s' %s pointer bind is in non-executable segment", path, opcodeName);
-                return true;
-            }
-            break;
-        }
-        default:
-            diag.error("in '%s' %s unknown bind type %d", path, opcodeName, type);
-            return true;
-    }
-    return false;
-}
-
-void MachOAnalyzer::forEachBind(Diagnostics& diag, void (^handler)(uint64_t runtimeOffset, int libOrdinal, uint8_t type, const char* symbolName,
-                                                                  bool weakImport, bool lazyBind, uint64_t addend, bool& stop),
-                                                   void (^strongHandler)(const char* symbolName)) const
-{
-    __block bool     startVmAddrSet = false;
-    __block uint64_t startVmAddr    = 0;
-    forEachBind(diag, ^(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                        bool segIndexSet, bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                        uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset,
-                        uint8_t type, const char* symbolName, bool weakImport, bool lazyBind, uint64_t addend, bool& stop) {
-       if ( !startVmAddrSet ) {
-            for (int i=0; i <= segmentIndex; ++i) {
-                if ( strcmp(segments[i].segName, "__TEXT") == 0 ) {
-                    startVmAddr = segments[i].vmAddr;
-                    startVmAddrSet = true;
-                    break;
-               }
-            }
-        }
-        uint64_t bindVmOffset  = segments[segmentIndex].vmAddr + segmentOffset;
-        uint64_t runtimeOffset = bindVmOffset - startVmAddr;
-        handler(runtimeOffset, libOrdinal, type, symbolName, weakImport, lazyBind, addend, stop);
-    }, ^(const char* symbolName) {
-        strongHandler(symbolName);
-    });
-}
-
-void MachOAnalyzer::forEachBind(Diagnostics& diag, void (^handler)(uint64_t runtimeOffset, int libOrdinal, const char* symbolName,
-                                                                  bool weakImport, bool lazyBind, uint64_t addend, bool& stop),
-                                                   void (^strongHandler)(const char* symbolName)) const
-{
-    forEachBind(diag, ^(uint64_t runtimeOffset, int libOrdinal, uint8_t type, const char* symbolName,
-                        bool weakImport, bool lazyBind, uint64_t addend, bool &stop) {
-        handler(runtimeOffset, libOrdinal, symbolName, weakImport, lazyBind, addend, stop);
-    }, strongHandler);
-}
-
-void MachOAnalyzer::forEachBind(Diagnostics& diag,
-                                 void (^handler)(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo segments[],
-                                                 bool segIndexSet,  bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                                                 uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset, uint8_t type,
-                                                 const char* symbolName, bool weakImport, bool lazyBind, uint64_t addend, bool& stop),
-                                 void (^strongHandler)(const char* symbolName)) const
-{
-    const uint32_t  ptrSize = this->pointerSize();
-    bool            stop    = false;
-
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return;
-
-
-
-    const uint32_t dylibCount = dependentDylibCount();
-
-    if ( leInfo.dyldInfo != nullptr ) {
-        // process bind opcodes
-        const uint8_t*  p    = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->bind_off);
-        const uint8_t*  end  = p + leInfo.dyldInfo->bind_size;
-        uint8_t         type = 0;
-        uint64_t        segmentOffset = 0;
-        uint8_t         segmentIndex = 0;
-        const char*     symbolName = NULL;
-        int             libraryOrdinal = 0;
-        bool            segIndexSet = false;
-        bool            libraryOrdinalSet = false;
-
-        int64_t         addend = 0;
-        uint64_t        count;
-        uint64_t        skip;
-        bool            weakImport = false;
-        while ( !stop && diag.noError() && (p < end) ) {
-            uint8_t immediate = *p & BIND_IMMEDIATE_MASK;
-            uint8_t opcode = *p & BIND_OPCODE_MASK;
-            ++p;
-            switch (opcode) {
-                case BIND_OPCODE_DONE:
-                    stop = true;
-                    break;
-                case BIND_OPCODE_SET_DYLIB_ORDINAL_IMM:
-                    libraryOrdinal = immediate;
-                    libraryOrdinalSet = true;
-                    break;
-                case BIND_OPCODE_SET_DYLIB_ORDINAL_ULEB:
-                    libraryOrdinal = (int)read_uleb128(diag, p, end);
-                    libraryOrdinalSet = true;
-                    break;
-                case BIND_OPCODE_SET_DYLIB_SPECIAL_IMM:
-                    // the special ordinals are negative numbers
-                    if ( immediate == 0 )
-                        libraryOrdinal = 0;
-                    else {
-                        int8_t signExtended = BIND_OPCODE_MASK | immediate;
-                        libraryOrdinal = signExtended;
-                    }
-                    libraryOrdinalSet = true;
-                    break;
-                case BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM:
-                    weakImport = ( (immediate & BIND_SYMBOL_FLAGS_WEAK_IMPORT) != 0 );
-                    symbolName = (char*)p;
-                    while (*p != '\0')
-                        ++p;
-                    ++p;
-                    break;
-                case BIND_OPCODE_SET_TYPE_IMM:
-                    type = immediate;
-                    break;
-                case BIND_OPCODE_SET_ADDEND_SLEB:
-                    addend = read_sleb128(diag, p, end);
-                    break;
-                case BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                    segmentIndex = immediate;
-                    segmentOffset = read_uleb128(diag, p, end);
-                    segIndexSet = true;
-                    break;
-                case BIND_OPCODE_ADD_ADDR_ULEB:
-                    segmentOffset += read_uleb128(diag, p, end);
-                    break;
-                case BIND_OPCODE_DO_BIND:
-                    handler("BIND_OPCODE_DO_BIND", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                            ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                    segmentOffset += ptrSize;
-                    break;
-                case BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB:
-                    handler("BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                            ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                    segmentOffset += read_uleb128(diag, p, end) + ptrSize;
-                    break;
-                case BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED:
-                    handler("BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                            ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                    segmentOffset += immediate*ptrSize + ptrSize;
-                    break;
-                case BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB:
-                    count = read_uleb128(diag, p, end);
-                    skip = read_uleb128(diag, p, end);
-                    for (uint32_t i=0; i < count; ++i) {
-                        handler("BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                                ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                        segmentOffset += skip + ptrSize;
-                        if ( stop )
-                            break;
-                    }
-                    break;
-                default:
-                    diag.error("bad bind opcode 0x%02X", *p);
-            }
-        }
-        if ( diag.hasError() )
-            return;
-
-        // process lazy bind opcodes
-        uint32_t lazyDoneCount = 0;
-        uint32_t lazyBindCount = 0;
-        if ( leInfo.dyldInfo->lazy_bind_size != 0 ) {
-            p               = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->lazy_bind_off);
-            end             = p + leInfo.dyldInfo->lazy_bind_size;
-            type            = BIND_TYPE_POINTER;
-            segmentOffset   = 0;
-            segmentIndex    = 0;
-            symbolName      = NULL;
-            libraryOrdinal  = 0;
-            segIndexSet     = false;
-            libraryOrdinalSet= false;
-            addend          = 0;
-            weakImport      = false;
-            stop            = false;
-            while (  !stop && diag.noError() && (p < end) ) {
-                uint8_t immediate = *p & BIND_IMMEDIATE_MASK;
-                uint8_t opcode = *p & BIND_OPCODE_MASK;
-                ++p;
-                switch (opcode) {
-                    case BIND_OPCODE_DONE:
-                        // this opcode marks the end of each lazy pointer binding
-                        ++lazyDoneCount;
-                        break;
-                    case BIND_OPCODE_SET_DYLIB_ORDINAL_IMM:
-                        libraryOrdinal = immediate;
-                        libraryOrdinalSet = true;
-                        break;
-                    case BIND_OPCODE_SET_DYLIB_ORDINAL_ULEB:
-                        libraryOrdinal = (int)read_uleb128(diag, p, end);
-                        libraryOrdinalSet = true;
-                        break;
-                    case BIND_OPCODE_SET_DYLIB_SPECIAL_IMM:
-                        // the special ordinals are negative numbers
-                        if ( immediate == 0 )
-                            libraryOrdinal = 0;
-                        else {
-                            int8_t signExtended = BIND_OPCODE_MASK | immediate;
-                            libraryOrdinal = signExtended;
-                        }
-                        libraryOrdinalSet = true;
-                        break;
-                    case BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM:
-                        weakImport = ( (immediate & BIND_SYMBOL_FLAGS_WEAK_IMPORT) != 0 );
-                        symbolName = (char*)p;
-                        while (*p != '\0')
-                            ++p;
-                        ++p;
-                        break;
-                    case BIND_OPCODE_SET_ADDEND_SLEB:
-                        addend = read_sleb128(diag, p, end);
-                        break;
-                    case BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                        segmentIndex = immediate;
-                        segmentOffset = read_uleb128(diag, p, end);
-                        segIndexSet = true;
-                        break;
-                    case BIND_OPCODE_DO_BIND:
-                        handler("BIND_OPCODE_DO_BIND", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                                ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, true, addend, stop);
-                        segmentOffset += ptrSize;
-                        ++lazyBindCount;
-                        break;
-                    case BIND_OPCODE_SET_TYPE_IMM:
-                    case BIND_OPCODE_ADD_ADDR_ULEB:
-                    case BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB:
-                    case BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED:
-                    case BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB:
-                    default:
-                        diag.error("bad lazy bind opcode 0x%02X", opcode);
-                        break;
-                }
-            }
-            if ( lazyDoneCount > lazyBindCount+7 ) {
-                // diag.error("lazy bind opcodes missing binds");
-            }
-        }
-        if ( diag.hasError() )
-            return;
-
-        // process weak bind info
-        if ( leInfo.dyldInfo->weak_bind_size != 0 ) {
-            p               = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->weak_bind_off);
-            end             = p + leInfo.dyldInfo->weak_bind_size;
-            type            = BIND_TYPE_POINTER;
-            segmentOffset   = 0;
-            segmentIndex    = 0;
-            symbolName      = NULL;
-            libraryOrdinal  = BIND_SPECIAL_DYLIB_WEAK_LOOKUP;
-            segIndexSet     = false;
-            libraryOrdinalSet= true;
-            addend          = 0;
-            weakImport      = false;
-            stop            = false;
-            while ( !stop && diag.noError() && (p < end) ) {
-                uint8_t immediate = *p & BIND_IMMEDIATE_MASK;
-                uint8_t opcode = *p & BIND_OPCODE_MASK;
-                ++p;
-                switch (opcode) {
-                    case BIND_OPCODE_DONE:
-                        stop = true;
-                        break;
-                    case BIND_OPCODE_SET_DYLIB_ORDINAL_IMM:
-                    case BIND_OPCODE_SET_DYLIB_ORDINAL_ULEB:
-                    case BIND_OPCODE_SET_DYLIB_SPECIAL_IMM:
-                        diag.error("unexpected dylib ordinal in weak_bind");
-                        break;
-                    case BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM:
-                        weakImport = ( (immediate & BIND_SYMBOL_FLAGS_WEAK_IMPORT) != 0 );
-                        symbolName = (char*)p;
-                        while (*p != '\0')
-                            ++p;
-                        ++p;
-                        if ( immediate & BIND_SYMBOL_FLAGS_NON_WEAK_DEFINITION ) {
-                            strongHandler(symbolName);
-                        }
-                        break;
-                    case BIND_OPCODE_SET_TYPE_IMM:
-                        type = immediate;
-                        break;
-                    case BIND_OPCODE_SET_ADDEND_SLEB:
-                        addend = read_sleb128(diag, p, end);
-                        break;
-                    case BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                        segmentIndex = immediate;
-                        segmentOffset = read_uleb128(diag, p, end);
-                        segIndexSet = true;
-                        break;
-                    case BIND_OPCODE_ADD_ADDR_ULEB:
-                        segmentOffset += read_uleb128(diag, p, end);
-                        break;
-                    case BIND_OPCODE_DO_BIND:
-                        handler("BIND_OPCODE_DO_BIND", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                                ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                        segmentOffset += ptrSize;
-                        break;
-                    case BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB:
-                        handler("BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                                ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                        segmentOffset += read_uleb128(diag, p, end) + ptrSize;
-                        break;
-                    case BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED:
-                        handler("BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                                ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                        segmentOffset += immediate*ptrSize + ptrSize;
-                        break;
-                    case BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB:
-                        count = read_uleb128(diag, p, end);
-                        skip = read_uleb128(diag, p, end);
-                        for (uint32_t i=0; i < count; ++i) {
-                            handler("BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                                    ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, stop);
-                            segmentOffset += skip + ptrSize;
-                            if ( stop )
-                                break;
-                        }
-                        break;
-                    default:
-                        diag.error("bad bind opcode 0x%02X", *p);
-                }
-            }
-        }
-    }
-    else if ( leInfo.chainedFixups != nullptr ) {
-        // binary uses chained fixups, so do nothing
-    }
-    else if ( leInfo.dynSymTab != nullptr ) {
-        // old binary, process external relocations
-        const uint64_t                  relocsStartAddress = externalRelocBaseAddress(segmentsInfo, leInfo.layout.linkeditSegIndex);
-        const relocation_info* const    relocsStart = (relocation_info*)getLinkEditContent(leInfo.layout, leInfo.dynSymTab->extreloff);
-        const relocation_info* const    relocsEnd   = &relocsStart[leInfo.dynSymTab->nextrel];
-        bool                            is64Bit     = is64() ;
-        const uint8_t                   relocSize   = (is64Bit ? 3 : 2);
-        const void*                     symbolTable = getLinkEditContent(leInfo.layout, leInfo.symTab->symoff);
-        const struct nlist_64*          symbols64   = (nlist_64*)symbolTable;
-        const struct nlist*             symbols32   = (struct nlist*)symbolTable;
-        const char*                     stringPool  = (char*)getLinkEditContent(leInfo.layout, leInfo.symTab->stroff);
-        uint32_t                        symCount    = leInfo.symTab->nsyms;
-        uint32_t                        poolSize    = leInfo.symTab->strsize;
-        for (const relocation_info* reloc=relocsStart; (reloc < relocsEnd) && !stop; ++reloc) {
-            bool isBranch = false;
-#if BUILDING_APP_CACHE_UTIL
-            if ( isKextBundle() ) {
-                // kext's may have other kinds of relocations, eg, branch relocs.  Skip them
-                if ( isArch("x86_64") || isArch("x86_64h") ) {
-                    if ( reloc->r_type == X86_64_RELOC_BRANCH ) {
-                        if ( reloc->r_length != 2 ) {
-                            diag.error("external relocation has wrong r_length");
-                            break;
-                        }
-                        if ( reloc->r_pcrel != true ) {
-                            diag.error("external relocation should be pcrel");
-                            break;
-                        }
-                        isBranch = true;
-                    }
-                }
-            }
-#endif
-
-            if ( !isBranch ) {
-                if ( reloc->r_length != relocSize ) {
-                    diag.error("external relocation has wrong r_length");
-                    break;
-                }
-                if ( reloc->r_type != 0 ) { // 0 == X86_64_RELOC_UNSIGNED == GENERIC_RELOC_VANILLA == ARM64_RELOC_UNSIGNED
-                    diag.error("external relocation has wrong r_type");
-                    break;
-                }
-            }
-            uint32_t segIndex  = 0;
-            uint64_t segOffset = 0;
-            if ( segIndexAndOffsetForAddress(relocsStartAddress+reloc->r_address, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-                uint32_t symbolIndex = reloc->r_symbolnum;
-                if ( symbolIndex > symCount ) {
-                    diag.error("external relocation has out of range r_symbolnum");
-                    break;
-                }
-                else {
-                    uint32_t strOffset  = is64Bit ? symbols64[symbolIndex].n_un.n_strx : symbols32[symbolIndex].n_un.n_strx;
-                    uint16_t n_desc     = is64Bit ? symbols64[symbolIndex].n_desc : symbols32[symbolIndex].n_desc;
-                    uint8_t  n_type     = is64Bit ? symbols64[symbolIndex].n_type : symbols32[symbolIndex].n_type;
-                    uint32_t libOrdinal = libOrdinalFromDesc(n_desc);
-                    if ( strOffset >= poolSize ) {
-                        diag.error("external relocation has r_symbolnum=%d which has out of range n_strx", symbolIndex);
-                        break;
-                    }
-                    else {
-                        const char*     symbolName = stringPool + strOffset;
-                        bool            weakImport = (n_desc & N_WEAK_REF);
-                        const uint8_t*  content    = (uint8_t*)this + segmentsInfo[segIndex].vmAddr - leInfo.layout.textUnslidVMAddr + segOffset;
-                        uint64_t        addend     = (reloc->r_length == 3) ? *((uint64_t*)content) : *((uint32_t*)content);
-                        // Handle defined weak def symbols which need to get a special ordinal
-                        if ( ((n_type & N_TYPE) == N_SECT) && ((n_type & N_EXT) != 0) && ((n_desc & N_WEAK_DEF) != 0) )
-                            libOrdinal = BIND_SPECIAL_DYLIB_WEAK_LOOKUP;
-                        uint8_t type = isBranch ? BIND_TYPE_TEXT_PCREL32 : BIND_TYPE_POINTER;
-                        handler("external relocation", leInfo, segmentsInfo, true, true, dylibCount, libOrdinal,
-                                ptrSize, segIndex, segOffset, type, symbolName, weakImport, false, addend, stop);
-                    }
-                }
-            }
-            else {
-                diag.error("local relocation has out of range r_address");
-                break;
-            }
-        }
-        // then process indirect symbols
-        forEachIndirectPointer(diag, false,
-                               ^(uint64_t address, bool bind, int bindLibOrdinal,
-                                 const char* bindSymbolName, bool bindWeakImport, bool bindLazy, bool selfModifyingStub, bool& indStop) {
-            if ( !bind )
-               return;
-            uint32_t segIndex  = 0;
-            uint64_t segOffset = 0;
-            if ( segIndexAndOffsetForAddress(address, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-                handler("indirect symbol", leInfo, segmentsInfo, true, true, dylibCount, bindLibOrdinal,
-                         ptrSize, segIndex, segOffset, BIND_TYPE_POINTER, bindSymbolName, bindWeakImport, bindLazy, 0, indStop);
-            }
-            else {
-                diag.error("indirect symbol has out of range address");
-                indStop = true;
-            }
-        });
-    }
-
-}
-
-bool MachOAnalyzer::validChainedFixupsInfo(Diagnostics& diag, const char* path) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return false;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return false;
-
-    // validate dyld_chained_fixups_header
-    const dyld_chained_fixups_header* chainsHeader = (dyld_chained_fixups_header*)getLinkEditContent(leInfo.layout, leInfo.chainedFixups->dataoff);
-    if ( chainsHeader->fixups_version != 0 ) {
-        diag.error("chained fixups, unknown header version");
-        return false;
-    }
-    if ( chainsHeader->starts_offset >= leInfo.chainedFixups->datasize )  {
-        diag.error("chained fixups, starts_offset exceeds LC_DYLD_CHAINED_FIXUPS size");
-        return false;
-    }
-    if ( chainsHeader->imports_offset > leInfo.chainedFixups->datasize )  {
-        diag.error("chained fixups, imports_offset exceeds LC_DYLD_CHAINED_FIXUPS size");
-        return false;
-    }
-   
-    uint32_t formatEntrySize;
-    switch ( chainsHeader->imports_format ) {
-        case DYLD_CHAINED_IMPORT:
-            formatEntrySize = sizeof(dyld_chained_import);
-            break;
-        case DYLD_CHAINED_IMPORT_ADDEND:
-            formatEntrySize = sizeof(dyld_chained_import_addend);
-            break;
-        case DYLD_CHAINED_IMPORT_ADDEND64:
-            formatEntrySize = sizeof(dyld_chained_import_addend64);
-            break;
-        default:
-            diag.error("chained fixups, unknown imports_format");
-            return false;
-    }
-    if ( greaterThanAddOrOverflow(chainsHeader->imports_offset, (formatEntrySize * chainsHeader->imports_count), chainsHeader->symbols_offset) ) {
-         diag.error("chained fixups, imports array overlaps symbols");
-         return false;
-    }
-    if ( chainsHeader->symbols_format != 0 )  {
-         diag.error("chained fixups, symbols_format unknown");
-         return false;
-    }
-
-    // validate dyld_chained_starts_in_image
-    const dyld_chained_starts_in_image* startsInfo = (dyld_chained_starts_in_image*)((uint8_t*)chainsHeader + chainsHeader->starts_offset);
-    if ( startsInfo->seg_count != leInfo.layout.linkeditSegIndex+1 ) {
-        // We can have fewer segments than the count, so long as those we are missing have no relocs
-        // This can happen because __CTF is inserted by ctf_insert after linking, and between __DATA and __LINKEDIT, but has no relocs
-        // ctf_insert updates the load commands to put __CTF between __DATA and __LINKEDIT, but doesn't update the chained fixups data structures
-        if ( startsInfo->seg_count > (leInfo.layout.linkeditSegIndex + 1) ) {
-            diag.error("chained fixups, seg_count exceeds number of segments");
-            return false;
-        }
-
-        // We can have fewer segments than the count, so long as those we are missing have no relocs
-        uint32_t numNoRelocSegments = 0;
-        uint32_t numExtraSegments = (leInfo.layout.lastSegIndex + 1) - startsInfo->seg_count;
-        for (unsigned i = 0; i != numExtraSegments; ++i) {
-            // Check each extra segment before linkedit
-            const SegmentInfo& segInfo = segmentsInfo[leInfo.layout.linkeditSegIndex - (i + 1)];
-            if ( segInfo.vmSize == 0 )
-                ++numNoRelocSegments;
-        }
-
-        if ( numNoRelocSegments != numExtraSegments ) {
-            diag.error("chained fixups, seg_count does not match number of segments");
-            return false;
-        }
-    }
-    const uint64_t baseAddress = preferredLoadAddress();
-    uint32_t maxValidPointerSeen = 0;
-    uint16_t pointer_format_for_all = 0;
-    bool pointer_format_found = false;
-    const uint8_t* endOfStarts = (uint8_t*)chainsHeader + chainsHeader->imports_offset;
-    for (uint32_t i=0; i < startsInfo->seg_count; ++i) {
-        uint32_t segInfoOffset = startsInfo->seg_info_offset[i];
-        // 0 offset means this segment has no fixups
-        if ( segInfoOffset == 0 )
-            continue;
-        const dyld_chained_starts_in_segment* segInfo = (dyld_chained_starts_in_segment*)((uint8_t*)startsInfo + segInfoOffset);
-        if ( segInfo->size > (endOfStarts - (uint8_t*)segInfo) ) {
-             diag.error("chained fixups, dyld_chained_starts_in_segment for segment #%d overruns imports table", i);
-             return false;
-        }
-
-        // validate dyld_chained_starts_in_segment
-        if ( (segInfo->page_size != 0x1000) && (segInfo->page_size != 0x4000) ) {
-            diag.error("chained fixups, page_size not 4KB or 16KB in segment #%d", i);
-            return false;
-        }
-        if ( segInfo->pointer_format > 12 ) {
-            diag.error("chained fixups, unknown pointer_format in segment #%d", i);
-            return false;
-        }
-        if ( !pointer_format_found ) {
-            pointer_format_for_all = segInfo->pointer_format;
-            pointer_format_found = true;
-        }
-        if ( segInfo->pointer_format != pointer_format_for_all) {
-            diag.error("chained fixups, pointer_format not same for all segments %d and %d", segInfo->pointer_format, pointer_format_for_all);
-            return false;
-        }
-        if ( segInfo->segment_offset != (segmentsInfo[i].vmAddr - baseAddress) ) {
-            diag.error("chained fixups, segment_offset does not match vmaddr from LC_SEGMENT in segment #%d", i);
-            return false;
-        }
-        if ( segInfo->max_valid_pointer != 0 ) {
-            if ( maxValidPointerSeen == 0 ) {
-                // record max_valid_pointer values seen
-                maxValidPointerSeen = segInfo->max_valid_pointer;
-            }
-            else if ( maxValidPointerSeen != segInfo->max_valid_pointer ) {
-                diag.error("chained fixups, different max_valid_pointer values seen in different segments");
-                return false;
-            }
-        }
-        // validate starts table in segment
-        if ( offsetof(dyld_chained_starts_in_segment, page_start[segInfo->page_count]) > segInfo->size ) {
-            diag.error("chained fixups, page_start array overflows size");
-            return false;
-        }
-        uint32_t maxOverflowIndex = (uint32_t)(segInfo->size - offsetof(dyld_chained_starts_in_segment, page_start[segInfo->page_count]))/sizeof(uint16_t);
-        for (int pageIndex=0; pageIndex < segInfo->page_count; ++pageIndex) {
-            uint16_t offsetInPage = segInfo->page_start[pageIndex];
-            if ( offsetInPage == DYLD_CHAINED_PTR_START_NONE )
-                continue;
-            if ( (offsetInPage & DYLD_CHAINED_PTR_START_MULTI) == 0 ) {
-                // this is the offset into the page where the first fixup is
-                if ( offsetInPage > segInfo->page_size ) {
-                    diag.error("chained fixups, in segment #%d page_start[%d]=0x%04X exceeds page size", i, pageIndex, offsetInPage);
-                }
-            }
-            else {
-                // this is actually an index into chain_starts[]
-                uint32_t overflowIndex = offsetInPage & ~DYLD_CHAINED_PTR_START_MULTI;
-                // now verify all starts are within the page and in ascending order
-                uint16_t lastOffsetInPage = 0;
-                do {
-                    if ( overflowIndex > maxOverflowIndex )  {
-                        diag.error("chain overflow index out of range %d (max=%d) in segment %s", overflowIndex, maxOverflowIndex, segmentName(i));
-                        return false;
-                    }
-                    offsetInPage = (segInfo->page_start[overflowIndex] & ~DYLD_CHAINED_PTR_START_LAST);
-                    if ( offsetInPage > segInfo->page_size ) {
-                        diag.error("chained fixups, in segment #%d overflow page_start[%d]=0x%04X exceeds page size", i, overflowIndex, offsetInPage);
-                        return false;
-                    }
-                    if ( (offsetInPage <= lastOffsetInPage) && (lastOffsetInPage != 0) )  {
-                        diag.error("chained fixups, in segment #%d overflow page_start[%d]=0x%04X is before previous at 0x%04X\n", i, overflowIndex, offsetInPage, lastOffsetInPage);
-                        return false;
-                    }
-                    lastOffsetInPage = offsetInPage;
-                    ++overflowIndex;
-                } while ( (segInfo->page_start[overflowIndex] & DYLD_CHAINED_PTR_START_LAST) == 0 );
-           }
-        }
-
-    }
-    // validate import table size can fit
-    if ( chainsHeader->imports_count != 0 ) {
-        uint32_t maxBindOrdinal = 0;
-        switch (pointer_format_for_all) {
-            case DYLD_CHAINED_PTR_32:
-                maxBindOrdinal = 0x0FFFFF; // 20-bits
-                break;
-            case DYLD_CHAINED_PTR_ARM64E:
-            case DYLD_CHAINED_PTR_ARM64E_USERLAND:
-            case DYLD_CHAINED_PTR_ARM64E_OFFSET:
-                maxBindOrdinal = 0x00FFFF; // 16-bits
-                break;
-            case DYLD_CHAINED_PTR_64:
-            case DYLD_CHAINED_PTR_64_OFFSET:
-            case DYLD_CHAINED_PTR_ARM64E_USERLAND24:
-                maxBindOrdinal = 0xFFFFFF; // 24 bits
-                break;
-        }
-        if ( chainsHeader->imports_count >= maxBindOrdinal )  {
-            diag.error("chained fixups, imports_count (%d) exceeds max of %d", chainsHeader->imports_count, maxBindOrdinal);
-            return false;
-        }
-    }
-
-    // validate max_valid_pointer is larger than last segment
-    if ( (maxValidPointerSeen != 0) && !inDyldCache() ) {
-        uint64_t lastSegmentLastVMAddr = segmentsInfo[leInfo.layout.linkeditSegIndex-1].vmAddr + segmentsInfo[leInfo.layout.linkeditSegIndex-1].vmSize;
-        if ( maxValidPointerSeen < lastSegmentLastVMAddr ) {
-            diag.error("chained fixups, max_valid_pointer too small for image");
-            return false;
-        }
-    }
-
-    return diag.noError();
-}
-
-bool MachOAnalyzer::validChainedFixupsInfoOldArm64e(Diagnostics& diag, const char* path) const
-{
-    __block uint32_t maxTargetCount = 0;
-    __block uint32_t currentTargetCount = 0;
-    parseOrgArm64eChainedFixups(diag,
-        ^(uint32_t totalTargets, bool& stop) {
-            maxTargetCount = totalTargets;
-        },
-        ^(const LinkEditInfo& leInfo, const SegmentInfo segments[], bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal, uint8_t type, const char* symbolName, uint64_t addend, bool weakImport, bool& stop) {
-           if ( symbolName == NULL ) {
-                diag.error("in '%s' missing BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM", path);
-            }
-            else if ( !libraryOrdinalSet ) {
-                diag.error("in '%s' missing BIND_OPCODE_SET_DYLIB_ORDINAL", path);
-            }
-            else if ( libOrdinal > (int)dylibCount ) {
-                diag.error("in '%s' has library ordinal too large (%d) max (%d)", path, libOrdinal, dylibCount);
-            }
-            else if ( libOrdinal < BIND_SPECIAL_DYLIB_WEAK_LOOKUP ) {
-                diag.error("in '%s' has unknown library special ordinal (%d)", path, libOrdinal);
-            }
-            else if ( type != BIND_TYPE_POINTER ) {
-                diag.error("in '%s' unknown bind type %d", path, type);
-            }
-            else if ( currentTargetCount > maxTargetCount ) {
-                diag.error("in '%s' chained target counts exceeds BIND_SUBOPCODE_THREADED_SET_BIND_ORDINAL_TABLE_SIZE_ULEB", path);
-            }
-            ++currentTargetCount;
-            if ( diag.hasError() )
-                stop = true;
-        },
-        ^(const LinkEditInfo& leInfo, const SegmentInfo segments[], uint8_t segmentIndex, bool segIndexSet, uint64_t segmentOffset, uint16_t format, bool& stop) {
-           if ( !segIndexSet ) {
-                diag.error("in '%s' missing BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB", path);
-            }
-            else if ( segmentIndex >= leInfo.layout.linkeditSegIndex )  {
-                diag.error("in '%s' segment index %d too large", path, segmentIndex);
-            }
-            else if ( segmentOffset > (segments[segmentIndex].vmSize-8) ) {
-                diag.error("in '%s' current segment offset 0x%08llX beyond segment size (0x%08llX)", path, segmentOffset, segments[segmentIndex].vmSize);
-            }
-            else if ( !segments[segmentIndex].writable() ) {
-                diag.error("in '%s' pointer bind is in non-writable segment", path);
-            }
-            else if ( segments[segmentIndex].executable() ) {
-                diag.error("in '%s' pointer bind is in executable segment", path);
-            }
-            if ( diag.hasError() )
-                stop = true;
-        }
-    );
-
-    return diag.noError();
-}
-
-
-
-void MachOAnalyzer::parseOrgArm64eChainedFixups(Diagnostics& diag, void (^targetCount)(uint32_t totalTargets, bool& stop),
-                                                                   void (^addTarget)(const LinkEditInfo& leInfo, const SegmentInfo segments[], bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal, uint8_t type, const char* symbolName, uint64_t addend, bool weakImport, bool& stop),
-                                                                   void (^addChainStart)(const LinkEditInfo& leInfo, const SegmentInfo segments[], uint8_t segmentIndex, bool segIndexSet, uint64_t segmentOffset, uint16_t format, bool& stop)) const
-{
-    bool            stop    = false;
-
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return;
-
-    const uint32_t dylibCount = dependentDylibCount();
-
-    if ( leInfo.dyldInfo != nullptr ) {
-        // process bind opcodes
-        const uint8_t*  p    = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->bind_off);
-        const uint8_t*  end  = p + leInfo.dyldInfo->bind_size;
-        uint8_t         type = 0;
-        uint64_t        segmentOffset = 0;
-        uint8_t         segmentIndex = 0;
-        const char*     symbolName = NULL;
-        int             libraryOrdinal = 0;
-        bool            segIndexSet = false;
-        bool            libraryOrdinalSet = false;
-        uint64_t        targetTableCount;
-        uint64_t        addend = 0;
-        bool            weakImport = false;
-        while ( !stop && diag.noError() && (p < end) ) {
-            uint8_t immediate = *p & BIND_IMMEDIATE_MASK;
-            uint8_t opcode = *p & BIND_OPCODE_MASK;
-            ++p;
-            switch (opcode) {
-                case BIND_OPCODE_DONE:
-                    stop = true;
-                    break;
-                case BIND_OPCODE_SET_DYLIB_ORDINAL_IMM:
-                    libraryOrdinal = immediate;
-                    libraryOrdinalSet = true;
-                    break;
-                case BIND_OPCODE_SET_DYLIB_ORDINAL_ULEB:
-                    libraryOrdinal = (int)read_uleb128(diag, p, end);
-                    libraryOrdinalSet = true;
-                    break;
-                case BIND_OPCODE_SET_DYLIB_SPECIAL_IMM:
-                    // the special ordinals are negative numbers
-                    if ( immediate == 0 )
-                        libraryOrdinal = 0;
-                    else {
-                        int8_t signExtended = BIND_OPCODE_MASK | immediate;
-                        libraryOrdinal = signExtended;
-                    }
-                    libraryOrdinalSet = true;
-                    break;
-                case BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM:
-                    weakImport = ( (immediate & BIND_SYMBOL_FLAGS_WEAK_IMPORT) != 0 );
-                    symbolName = (char*)p;
-                    while (*p != '\0')
-                        ++p;
-                    ++p;
-                    break;
-                case BIND_OPCODE_SET_TYPE_IMM:
-                    type = immediate;
-                    break;
-                case BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                    segmentIndex = immediate;
-                    segmentOffset = read_uleb128(diag, p, end);
-                    segIndexSet = true;
-                    break;
-                case BIND_OPCODE_SET_ADDEND_SLEB:
-                    addend = read_sleb128(diag, p, end);
-                    break;
-                case BIND_OPCODE_DO_BIND:
-                    if ( addTarget )
-                        addTarget(leInfo, segmentsInfo, libraryOrdinalSet, dylibCount, libraryOrdinal, type, symbolName, addend, weakImport, stop);
-                    break;
-                case BIND_OPCODE_THREADED:
-                    switch (immediate) {
-                        case BIND_SUBOPCODE_THREADED_SET_BIND_ORDINAL_TABLE_SIZE_ULEB:
-                            targetTableCount = read_uleb128(diag, p, end);
-                            if ( targetTableCount > 65535 ) {
-                                diag.error("BIND_SUBOPCODE_THREADED_SET_BIND_ORDINAL_TABLE_SIZE_ULEB size too large");
-                                stop = true;
-                            }
-                            else {
-                                if ( targetCount )
-                                    targetCount((uint32_t)targetTableCount, stop);
-                            }
-                            break;
-                        case BIND_SUBOPCODE_THREADED_APPLY:
-                            if ( addChainStart )
-                                addChainStart(leInfo, segmentsInfo, segmentIndex, segIndexSet, segmentOffset, DYLD_CHAINED_PTR_ARM64E, stop);
-                            break;
-                        default:
-                            diag.error("bad BIND_OPCODE_THREADED sub-opcode 0x%02X", immediate);
-                    }
-                    break;
-                default:
-                    diag.error("bad bind opcode 0x%02X", immediate);
-            }
-        }
-        if ( diag.hasError() )
-            return;
-    }
-}
-
-void MachOAnalyzer::forEachChainedFixupTarget(Diagnostics& diag, void (^callback)(int libOrdinal, const char* symbolName, uint64_t addend, bool weakImport, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return;
-
-    bool stop    = false;
-    if ( leInfo.dyldInfo != nullptr ) {
-        parseOrgArm64eChainedFixups(diag, nullptr, ^(const LinkEditInfo& leInfo2, const SegmentInfo segments[], bool libraryOrdinalSet, uint32_t dylibCount,
-                                                    int libOrdinal, uint8_t type, const char* symbolName, uint64_t fixAddend, bool weakImport, bool& stopChain) {
-            callback(libOrdinal, symbolName, fixAddend, weakImport, stopChain);
-        }, nullptr);
-    }
-    else if ( leInfo.chainedFixups != nullptr ) {
-        const dyld_chained_fixups_header*  header = (dyld_chained_fixups_header*)getLinkEditContent(leInfo.layout, leInfo.chainedFixups->dataoff);
-        if ( (header->imports_offset > leInfo.chainedFixups->datasize) || (header->symbols_offset > leInfo.chainedFixups->datasize) ) {
-            diag.error("malformed import table");
-            return;
-        }
-        const dyld_chained_import*          imports;
-        const dyld_chained_import_addend*   importsA32;
-        const dyld_chained_import_addend64* importsA64;
-        const char*                         symbolsPool     = (char*)header + header->symbols_offset;
-        uint32_t                            maxSymbolOffset = leInfo.chainedFixups->datasize - header->symbols_offset;
-        int                                 libOrdinal;
-        switch (header->imports_format) {
-            case DYLD_CHAINED_IMPORT:
-                imports = (dyld_chained_import*)((uint8_t*)header + header->imports_offset);
-                for (uint32_t i=0; i < header->imports_count && !stop; ++i) {
-                    const char* symbolName = &symbolsPool[imports[i].name_offset];
-                    if ( imports[i].name_offset > maxSymbolOffset ) {
-                        diag.error("malformed import table, string overflow");
-                        return;
-                    }
-                    uint8_t libVal = imports[i].lib_ordinal;
-                    if ( libVal > 0xF0 )
-                        libOrdinal = (int8_t)libVal;
-                    else
-                        libOrdinal = libVal;
-                    callback(libOrdinal, symbolName, 0, imports[i].weak_import, stop);
-                }
-                break;
-            case DYLD_CHAINED_IMPORT_ADDEND:
-                importsA32 = (dyld_chained_import_addend*)((uint8_t*)header + header->imports_offset);
-                for (uint32_t i=0; i < header->imports_count && !stop; ++i) {
-                    const char* symbolName = &symbolsPool[importsA32[i].name_offset];
-                    if ( importsA32[i].name_offset > maxSymbolOffset ) {
-                        diag.error("malformed import table, string overflow");
-                        return;
-                    }
-                    uint8_t libVal = importsA32[i].lib_ordinal;
-                    if ( libVal > 0xF0 )
-                        libOrdinal = (int8_t)libVal;
-                    else
-                        libOrdinal = libVal;
-                    callback(libOrdinal, symbolName, importsA32[i].addend, importsA32[i].weak_import, stop);
-                }
-                break;
-            case DYLD_CHAINED_IMPORT_ADDEND64:
-                importsA64 = (dyld_chained_import_addend64*)((uint8_t*)header + header->imports_offset);
-                for (uint32_t i=0; i < header->imports_count && !stop; ++i) {
-                    const char* symbolName = &symbolsPool[importsA64[i].name_offset];
-                    if ( importsA64[i].name_offset > maxSymbolOffset ) {
-                        diag.error("malformed import table, string overflow");
-                        return;
-                    }
-                    uint16_t libVal = importsA64[i].lib_ordinal;
-                    if ( libVal > 0xFFF0 )
-                        libOrdinal = (int16_t)libVal;
-                    else
-                        libOrdinal = libVal;
-                    callback(libOrdinal, symbolName, importsA64[i].addend, importsA64[i].weak_import, stop);
-                }
-                break;
-           default:
-                diag.error("unknown imports format");
-                return;
-        }
-    }
-}
-
-uint32_t MachOAnalyzer::segmentCount() const
-{
-    __block uint32_t count   = 0;
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        ++count;
-    });
-    return count;
-}
-
-bool MachOAnalyzer::hasCodeSignature(uint32_t& fileOffset, uint32_t& size) const
-{
-    fileOffset = 0;
-    size = 0;
-
-    Diagnostics diag;
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        if ( cmd->cmd == LC_CODE_SIGNATURE ) {
-            const linkedit_data_command* sigCmd = (linkedit_data_command*)cmd;
-            fileOffset = sigCmd->dataoff;
-            size       = sigCmd->datasize;
-            stop = true;
-        }
-    });
-    diag.assertNoError();   // any malformations in the file should have been caught by earlier validate() call
-
-    // early exist if no LC_CODE_SIGNATURE
-    if ( fileOffset == 0 )
-        return false;
-
-    // <rdar://problem/13622786> ignore code signatures in macOS binaries built with pre-10.9 tools
-    if ( (this->cputype == CPU_TYPE_X86_64) || (this->cputype == CPU_TYPE_I386) ) {
-        __block bool foundPlatform = false;
-        __block bool badSignature  = false;
-        forEachSupportedPlatform(^(Platform platform, uint32_t minOS, uint32_t sdk) {
-            foundPlatform = true;
-            if ( (platform == Platform::macOS) && (sdk < 0x000A0900) )
-                badSignature = true;
-        });
-        return foundPlatform && !badSignature;
-    }
-
-    return true;
-}
-
-bool MachOAnalyzer::hasProgramVars(uint32_t& progVarsOffset, bool& crtRunsInitializers, DyldLookFunc*& dyldLookupFuncAddr) const
-{
-    crtRunsInitializers = false;
-    if ( this->filetype != MH_EXECUTE )
-        return false;
-
-    // macOS 10.8+              program uses LC_MAIN and ProgramVars are in libdyld.dylib
-    // macOS 10.6 -> 10.7       ProgramVars are in __program_vars section in main executable
-    // macOS 10.5               ProgramVars are in __dyld section in main executable and 7 pointers in size
-    // macOS 10.4 and earlier   ProgramVars need to be looked up by name in nlist of main executable
-
-    uint64_t offset;
-    bool     usesCRT;
-    if ( getEntry(offset, usesCRT) && usesCRT ) {
-        // is pre-10.8 program
-        uint64_t sectionSize;
-        if ( const void* progVarsSection = findSectionContent("__DATA", "__program_vars", sectionSize) ) {
-            // macOS 10.6 or 10.7 binary
-            progVarsOffset = (uint32_t)((uint8_t*)progVarsSection - (uint8_t*)this);
-            return true;
-        }
-        else if ( const void* dyldSection = findSectionContent("__DATA", "__dyld", sectionSize) ) {
-#if SUPPPORT_PRE_LC_MAIN
-            if ( sectionSize >= 2*pointerSize() ) {
-                dyldLookupFuncAddr = (DyldLookFunc*)((uint8_t*)dyldSection + pointerSize());
-            }
-#endif
-            if ( sectionSize >= 7*pointerSize() ) {
-                // macOS 10.5 binary
-                progVarsOffset = (uint32_t)((uint8_t*)dyldSection - (uint8_t*)this) + 2*pointerSize();
-                return true;
-            }
-            else {
-                // macOS 10.4 binary
-                crtRunsInitializers = true;
-            }
-        }
-        return false;
-    }
-    return false;
-}
-
-// Convert from a (possibly) live pointer to a vmAddr
-uint64_t MachOAnalyzer::VMAddrConverter::convertToVMAddr(uint64_t value) const {
-    if ( contentRebased ) {
-        if ( value == 0 )
-            return 0;
-        // The value may have been signed.  Strip the signature if that is the case
-#if __has_feature(ptrauth_calls)
-        value = (uint64_t)__builtin_ptrauth_strip((void*)value, ptrauth_key_asia);
-#endif
-        value -= slide;
-        return value;
-    }
-    if ( chainedPointerFormat != 0 ) {
-        // We try to only use the VMAddrConverter on locations which are pointers, but we don't know
-        // for sure if the location contains a rebase or not.  Eg, it can be called on a NULL Protocol ISA field.
-        // If we see a 0, then its extremely likely that this is not a rebase, as we only use VMAddrConverter
-        // for initializers, terminators, and objc.  None of those have any reason to point to offset 0 in the binary,
-        // ie, no reason to point to the mach_header
-        if ( value == 0 )
-            return 0;
-        auto* chainedValue = (MachOAnalyzer::ChainedFixupPointerOnDisk*)&value;
-        uint64_t targetRuntimeOffset;
-        if ( chainedValue->isRebase(chainedPointerFormat, preferredLoadAddress, targetRuntimeOffset) ) {
-            value = preferredLoadAddress + targetRuntimeOffset;
-        }
-        return value;
-    }
-
-#if !(BUILDING_LIBDYLD || BUILDING_DYLD)
-    typedef MachOAnalyzer::VMAddrConverter VMAddrConverter;
-    if ( sharedCacheChainedPointerFormat != VMAddrConverter::SharedCacheFormat::none ) {
-        switch ( sharedCacheChainedPointerFormat ) {
-            case VMAddrConverter::SharedCacheFormat::none:
-                assert(false);
-            case VMAddrConverter::SharedCacheFormat::v2_x86_64_tbi: {
-                const uint64_t   deltaMask    = 0x00FFFF0000000000;
-                const uint64_t   valueMask    = ~deltaMask;
-                const uint64_t   valueAdd     = preferredLoadAddress;
-                value = (value & valueMask);
-                if ( value != 0 ) {
-                    value += valueAdd;
-                }
-                break;
-            }
-            case VMAddrConverter::SharedCacheFormat::v3: {
-                // Just use the chained pointer format for arm64e
-                auto* chainedValue = (MachOAnalyzer::ChainedFixupPointerOnDisk*)&value;
-                uint64_t targetRuntimeOffset;
-                if ( chainedValue->isRebase(DYLD_CHAINED_PTR_ARM64E, preferredLoadAddress,
-                                            targetRuntimeOffset) ) {
-                    value = preferredLoadAddress + targetRuntimeOffset;
-                }
-                break;
-            }
-            case VMAddrConverter::SharedCacheFormat::v4: {
-                const uint64_t   deltaMask    = 0x00000000C0000000;
-                const uint64_t   valueMask    = ~deltaMask;
-                const uint64_t   valueAdd     = preferredLoadAddress;
-                value = (value & valueMask);
-                if ( value != 0 ) {
-                    value += valueAdd;
-                }
-                break;
-            }
-        }
-        return value;
-    }
-#endif
-
-    return value;
-}
-
-MachOAnalyzer::VMAddrConverter MachOAnalyzer::makeVMAddrConverter(bool contentRebased) const {
-    MachOAnalyzer::VMAddrConverter vmAddrConverter;
-    vmAddrConverter.preferredLoadAddress   = preferredLoadAddress();
-    vmAddrConverter.slide                  = getSlide();
-    vmAddrConverter.chainedPointerFormat   = hasChainedFixups() ? chainedPointerFormat() : 0;
-    vmAddrConverter.contentRebased         = contentRebased;
-    return vmAddrConverter;
-}
-
-bool MachOAnalyzer::hasInitializer(Diagnostics& diag) const
-{
-    __block bool result = false;
-
-    // if dylib linked with -init linker option, that initializer is first
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        if ( (cmd->cmd == LC_ROUTINES) || (cmd->cmd == LC_ROUTINES_64) ) {
-            result = true;
-            stop = true;
-        }
-    });
-
-    if ( result )
-        return true;
-
-    // next any function pointers in mod-init section
-    forEachInitializerPointerSection(diag, ^(uint32_t sectionOffset, uint32_t sectionSize, const uint8_t* content, bool& stop) {
-        result = true;
-        stop = true;
-    });
-
-    if ( result )
-        return true;
-
-    forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& stop) {
-        if ( (info.sectFlags & SECTION_TYPE) != S_INIT_FUNC_OFFSETS )
-            return;
-        result = true;
-        stop = true;
-    });
-
-    return result;
-}
-
-void MachOAnalyzer::forEachInitializerPointerSection(Diagnostics& diag, void (^callback)(uint32_t sectionOffset, uint32_t sectionSize, const uint8_t* content, bool& stop)) const
-{
-    const unsigned ptrSize     = pointerSize();
-    const uint64_t baseAddress = preferredLoadAddress();
-    const uint64_t slide       = (uint64_t)this - baseAddress;
-    forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& sectStop) {
-        if ( (info.sectFlags & SECTION_TYPE) == S_MOD_INIT_FUNC_POINTERS ) {
-            if ( (info.sectSize % ptrSize) != 0 ) {
-                diag.error("initializer section %s/%s has bad size", info.segInfo.segName, info.sectName);
-                sectStop = true;
-                return;
-            }
-            if ( malformedSectionRange ) {
-                diag.error("initializer section %s/%s extends beyond its segment", info.segInfo.segName, info.sectName);
-                sectStop = true;
-                return;
-            }
-            const uint8_t* content = (uint8_t*)(info.sectAddr + slide);
-            if ( ((long)content % ptrSize) != 0 ) {
-                diag.error("initializer section %s/%s is not pointer aligned", info.segInfo.segName, info.sectName);
-                sectStop = true;
-                return;
-            }
-            callback((uint32_t)(info.sectAddr - baseAddress), (uint32_t)info.sectSize, content, sectStop);
-        }
-    });
-}
-
-struct VIS_HIDDEN SegmentRanges
-{
-    struct SegmentRange {
-        uint64_t vmAddrStart;
-        uint64_t vmAddrEnd;
-        uint32_t fileSize;
-    };
-
-    bool contains(uint64_t vmAddr) const {
-        for (const SegmentRange& range : segments) {
-            if ( (range.vmAddrStart <= vmAddr) && (vmAddr < range.vmAddrEnd) )
-                return true;
-        }
-        return false;
-    }
-
-private:
-    SegmentRange localAlloc[1];
-
-public:
-    dyld3::OverflowSafeArray<SegmentRange> segments { localAlloc, sizeof(localAlloc) / sizeof(localAlloc[0]) };
-};
-
-void MachOAnalyzer::forEachInitializer(Diagnostics& diag, const VMAddrConverter& vmAddrConverter, void (^callback)(uint32_t offset), const void* dyldCache) const
-{
-    __block SegmentRanges executableSegments;
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        if ( (info.protections & VM_PROT_EXECUTE) != 0 ) {
-            executableSegments.segments.push_back({ info.vmAddr, info.vmAddr + info.vmSize, (uint32_t)info.fileSize });
-        }
-    });
-
-    if (executableSegments.segments.empty()) {
-        diag.error("no exeutable segments");
-        return;
-    }
-
-    uint64_t loadAddress = preferredLoadAddress();
-    intptr_t slide = getSlide();
-
-    // if dylib linked with -init linker option, that initializer is first
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        if ( cmd->cmd == LC_ROUTINES ) {
-            const routines_command* routines = (routines_command*)cmd;
-            uint64_t dashInit = routines->init_address;
-            if ( executableSegments.contains(dashInit) )
-                callback((uint32_t)(dashInit - loadAddress));
-            else
-                diag.error("-init does not point within __TEXT segment");
-        }
-        else if ( cmd->cmd == LC_ROUTINES_64 ) {
-            const routines_command_64* routines = (routines_command_64*)cmd;
-            uint64_t dashInit = routines->init_address;
-            if ( executableSegments.contains(dashInit) )
-                callback((uint32_t)(dashInit - loadAddress));
-            else
-                diag.error("-init does not point within __TEXT segment");
-        }
-    });
-
-    // next any function pointers in mod-init section
-    const unsigned ptrSize          = pointerSize();
-    forEachInitializerPointerSection(diag, ^(uint32_t sectionOffset, uint32_t sectionSize, const uint8_t* content, bool& stop) {
-        if ( ptrSize == 8 ) {
-            const uint64_t* initsStart = (uint64_t*)content;
-            const uint64_t* initsEnd   = (uint64_t*)((uint8_t*)content + sectionSize);
-            for (const uint64_t* p=initsStart; p < initsEnd; ++p) {
-                uint64_t anInit = vmAddrConverter.convertToVMAddr(*p);
-                if ( !executableSegments.contains(anInit) ) {
-                     diag.error("initializer 0x%0llX does not point within executable segment", anInit);
-                     stop = true;
-                     break;
-                }
-                callback((uint32_t)(anInit - loadAddress));
-            }
-        }
-        else {
-            const uint32_t* initsStart = (uint32_t*)content;
-            const uint32_t* initsEnd   = (uint32_t*)((uint8_t*)content + sectionSize);
-            for (const uint32_t* p=initsStart; p < initsEnd; ++p) {
-                uint32_t anInit = (uint32_t)vmAddrConverter.convertToVMAddr(*p);
-                if ( !executableSegments.contains(anInit) ) {
-                     diag.error("initializer 0x%0X does not point within executable segment", anInit);
-                     stop = true;
-                     break;
-                }
-                callback(anInit - (uint32_t)loadAddress);
-            }
-        }
-    });
-
-    forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& stop) {
-        if ( (info.sectFlags & SECTION_TYPE) != S_INIT_FUNC_OFFSETS )
-            return;
-        const uint8_t* content = (uint8_t*)(info.sectAddr + slide);
-        if ( info.segInfo.writable() ) {
-            diag.error("initializer offsets section %s/%s must be in read-only segment", info.segInfo.segName, info.sectName);
-            stop = true;
-            return;
-        }
-        if ( (info.sectSize % 4) != 0 ) {
-            diag.error("initializer offsets section %s/%s has bad size", info.segInfo.segName, info.sectName);
-            stop = true;
-            return;
-        }
-        if ( malformedSectionRange ) {
-            diag.error("initializer offsets section %s/%s extends beyond the end of the segment", info.segInfo.segName, info.sectName);
-            stop = true;
-            return;
-        }
-        if ( (info.sectAddr % 4) != 0 ) {
-            diag.error("initializer offsets section %s/%s is not 4-byte aligned", info.segInfo.segName, info.sectName);
-            stop = true;
-            return;
-        }
-        const uint32_t* initsStart = (uint32_t*)content;
-        const uint32_t* initsEnd   = (uint32_t*)((uint8_t*)content + info.sectSize);
-        for (const uint32_t* p=initsStart; p < initsEnd; ++p) {
-            uint32_t anInitOffset = *p;
-            if ( !executableSegments.contains(loadAddress + anInitOffset) ) {
-                 diag.error("initializer 0x%08X is not an offset to an executable segment", anInitOffset);
-                 stop = true;
-                 break;
-            }
-            callback(anInitOffset);
-        }
-    });
-}
-
-bool MachOAnalyzer::hasTerminators(Diagnostics& diag, const VMAddrConverter& vmAddrConverter) const
-{
-    __block bool result = false;
-    forEachTerminator(diag, vmAddrConverter, ^(uint32_t offset) {
-        result = true;
-    });
-    return result;
-}
-
-void MachOAnalyzer::forEachTerminator(Diagnostics& diag, const VMAddrConverter& vmAddrConverter, void (^callback)(uint32_t offset)) const
-{
-    __block SegmentRanges executableSegments;
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        if ( (info.protections & VM_PROT_EXECUTE) != 0 ) {
-            executableSegments.segments.push_back({ info.vmAddr, info.vmAddr + info.vmSize, (uint32_t)info.fileSize });
-        }
-    });
-
-    if (executableSegments.segments.empty()) {
-        diag.error("no exeutable segments");
-        return;
-    }
-
-    uint64_t loadAddress = preferredLoadAddress();
-    intptr_t slide = getSlide();
-
-    // next any function pointers in mod-term section
-    const unsigned ptrSize          = pointerSize();
-    forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& stop) {
-        if ( (info.sectFlags & SECTION_TYPE) == S_MOD_TERM_FUNC_POINTERS ) {
-            const uint8_t* content;
-            content = (uint8_t*)(info.sectAddr + slide);
-            if ( (info.sectSize % ptrSize) != 0 ) {
-                diag.error("terminator section %s/%s has bad size", info.segInfo.segName, info.sectName);
-                stop = true;
-                return;
-            }
-            if ( malformedSectionRange ) {
-                diag.error("terminator section %s/%s extends beyond its segment", info.segInfo.segName, info.sectName);
-                stop = true;
-                return;
-            }
-            if ( ((long)content % ptrSize) != 0 ) {
-                diag.error("terminator section %s/%s is not pointer aligned", info.segInfo.segName, info.sectName);
-                stop = true;
-                return;
-            }
-            if ( ptrSize == 8 ) {
-                const uint64_t* initsStart = (uint64_t*)content;
-                const uint64_t* initsEnd   = (uint64_t*)((uint8_t*)content + info.sectSize);
-                for (const uint64_t* p=initsStart; p < initsEnd; ++p) {
-                    uint64_t rawContent = *p;
-    #if __has_feature(ptrauth_calls)
-                    rawContent = (uint64_t)__builtin_ptrauth_strip((void*)rawContent, ptrauth_key_asia);
-    #endif
-                    uint64_t anInit = vmAddrConverter.convertToVMAddr(rawContent);
-                    if ( !executableSegments.contains(anInit) ) {
-                         diag.error("terminator 0x%0llX does not point within executable segment", anInit);
-                         stop = true;
-                         break;
-                    }
-                    callback((uint32_t)(anInit - loadAddress));
-                }
-            }
-            else {
-                const uint32_t* initsStart = (uint32_t*)content;
-                const uint32_t* initsEnd   = (uint32_t*)((uint8_t*)content + info.sectSize);
-                for (const uint32_t* p=initsStart; p < initsEnd; ++p) {
-                    uint32_t anInit = (uint32_t)vmAddrConverter.convertToVMAddr(*p);
-                    if ( !executableSegments.contains(anInit) ) {
-                         diag.error("terminator 0x%0X does not point within executable segment", anInit);
-                         stop = true;
-                         break;
-                    }
-                    callback(anInit - (uint32_t)loadAddress);
-                }
-            }
-        }
-    });
-}
-
-
-
-void MachOAnalyzer::forEachRPath(void (^callback)(const char* rPath, bool& stop)) const
-{
-    Diagnostics diag;
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-         if ( cmd->cmd == LC_RPATH ) {
-            const char* rpath = (char*)cmd + ((struct rpath_command*)cmd)->path.offset;
-            callback(rpath, stop);
-        }
-    });
-    diag.assertNoError();   // any malformations in the file should have been caught by earlier validate() call
-}
-
-
-bool MachOAnalyzer::hasObjC() const
-{
-    __block bool result = false;
-    forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& stop) {
-        if ( (strcmp(info.sectName, "__objc_imageinfo") == 0) && (strncmp(info.segInfo.segName, "__DATA", 6) == 0) ) {
-            result = true;
-            stop = true;
-        }
-        if ( (this->cputype == CPU_TYPE_I386) && (strcmp(info.sectName, "__image_info") == 0) && (strcmp(info.segInfo.segName, "__OBJC") == 0) ) {
-            result = true;
-            stop = true;
-        }
-    });
-    return result;
-}
-
-bool MachOAnalyzer::usesObjCGarbageCollection() const
-{
-    __block bool result = false;
-    forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& stop) {
-        if ( (strcmp(info.sectName, "__objc_imageinfo") == 0) && (strncmp(info.segInfo.segName, "__DATA", 6) == 0) ) {
-            const uint64_t  slide = (uint64_t)this - preferredLoadAddress();
-            const uint32_t* flags = (uint32_t*)(info.sectAddr + slide);
-            if ( flags[1] & 4 )
-                result = true;
-            stop = true;
-        }
-     });
-    return result;
-}
-
-
-bool MachOAnalyzer::hasPlusLoadMethod(Diagnostics& diag) const
-{
-    __block bool result = false;
-    if ( (this->cputype == CPU_TYPE_I386) && this->builtForPlatform(Platform::macOS) ) {
-        // old objc runtime has no special section for +load methods, scan for string
-        int64_t slide = getSlide();
-        forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& stop) {
-            if ( ( (info.sectFlags & SECTION_TYPE) == S_CSTRING_LITERALS ) ) {
-                if ( malformedSectionRange ) {
-                    diag.error("cstring section %s/%s extends beyond the end of the segment", info.segInfo.segName, info.sectName);
-                    stop = true;
-                    return;
-                }
-                const uint8_t* content = (uint8_t*)(info.sectAddr + slide);
-                const char* s   = (char*)content;
-                const char* end = s + info.sectSize;
-                while ( s < end ) {
-                    if ( strcmp(s, "load") == 0 ) {
-                        result = true;
-                        stop = true;
-                        return;
-                    }
-                    while (*s != '\0' )
-                        ++s;
-                    ++s;
-                }
-            }
-        });
-    }
-    else {
-        // in new objc runtime compiler puts classes/categories with +load method in specical section
-        forEachSection(^(const SectionInfo& info, bool malformedSectionRange, bool& stop) {
-            if ( strncmp(info.segInfo.segName, "__DATA", 6) != 0 )
-                return;
-            if ( (strcmp(info.sectName, "__objc_nlclslist") == 0) || (strcmp(info.sectName, "__objc_nlcatlist") == 0)) {
-                result = true;
-                stop = true;
-            }
-        });
-    }
-    return result;
-}
-
-bool MachOAnalyzer::isSwiftLibrary() const
-{
-    struct objc_image_info {
-        int32_t version;
-        uint32_t flags;
-    };
-    
-    int64_t slide = getSlide();
-    __block bool result = false;
-    forEachSection(^(const SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        if ( (strncmp(sectInfo.sectName, "__objc_imageinfo", 16) == 0) && (strncmp(sectInfo.segInfo.segName, "__DATA", 6) == 0) ) {
-            objc_image_info* info =  (objc_image_info*)((uint8_t*)sectInfo.sectAddr + slide);
-            uint32_t swiftVersion = ((info->flags >> 8) & 0xFF);
-            if ( swiftVersion )
-                result = true;
-            stop = true;
-        }
-    });
-    return result;
-}
-
-const void* MachOAnalyzer::getRebaseOpcodes(uint32_t& size) const
-{
-    Diagnostics diag;
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() || (leInfo.dyldInfo == nullptr) )
-        return nullptr;
-
-    size = leInfo.dyldInfo->rebase_size;
-    return getLinkEditContent(leInfo.layout, leInfo.dyldInfo->rebase_off);
-}
-
-const void* MachOAnalyzer::getBindOpcodes(uint32_t& size) const
-{
-    Diagnostics diag;
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() || (leInfo.dyldInfo == nullptr) )
-        return nullptr;
-
-    size = leInfo.dyldInfo->bind_size;
-    return getLinkEditContent(leInfo.layout, leInfo.dyldInfo->bind_off);
-}
-
-const void* MachOAnalyzer::getLazyBindOpcodes(uint32_t& size) const
-{
-    Diagnostics diag;
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() || (leInfo.dyldInfo == nullptr) )
-        return nullptr;
-
-    size = leInfo.dyldInfo->lazy_bind_size;
-    return getLinkEditContent(leInfo.layout, leInfo.dyldInfo->lazy_bind_off);
-}
-
-const void* MachOAnalyzer::getSplitSeg(uint32_t& size) const
-{
-    Diagnostics diag;
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() || (leInfo.splitSegInfo == nullptr) )
-        return nullptr;
-
-    size = leInfo.splitSegInfo->datasize;
-    return getLinkEditContent(leInfo.layout, leInfo.splitSegInfo->dataoff);
-}
-
-bool MachOAnalyzer::hasSplitSeg() const {
-    uint32_t splitSegSize = 0;
-    const void* splitSegStart = getSplitSeg(splitSegSize);
-    return splitSegStart != nullptr;
-}
-
-bool MachOAnalyzer::isSplitSegV1() const {
-    uint32_t splitSegSize = 0;
-    const void* splitSegStart = getSplitSeg(splitSegSize);
-    if (!splitSegStart)
-        return false;
-
-    return (*(const uint8_t*)splitSegStart) != DYLD_CACHE_ADJ_V2_FORMAT;
-}
-
-bool MachOAnalyzer::isSplitSegV2() const {
-    uint32_t splitSegSize = 0;
-    const void* splitSegStart = getSplitSeg(splitSegSize);
-    if (!splitSegStart)
-        return false;
-
-    return (*(const uint8_t*)splitSegStart) == DYLD_CACHE_ADJ_V2_FORMAT;
-}
-
-
-uint64_t MachOAnalyzer::segAndOffsetToRuntimeOffset(uint8_t targetSegIndex, uint64_t targetSegOffset) const
-{
-    __block uint64_t textVmAddr = 0;
-    __block uint64_t result     = 0;
-    forEachSegment(^(const SegmentInfo& info, bool& stop) {
-        if ( strcmp(info.segName, "__TEXT") == 0 )
-            textVmAddr = info.vmAddr;
-        if ( info.segIndex == targetSegIndex ) {
-            result = (info.vmAddr - textVmAddr) + targetSegOffset;
-        }
-    });
-    return result;
-}
-
-bool MachOAnalyzer::hasLazyPointers(uint32_t& runtimeOffset, uint32_t& size) const
-{
-    size = 0;
-    forEachSection(^(const dyld3::MachOAnalyzer::SectionInfo& info, bool malformedSectionRange, bool &stop) {
-        if ( (info.sectFlags & SECTION_TYPE) == S_LAZY_SYMBOL_POINTERS ) {
-            runtimeOffset = (uint32_t)(info.sectAddr - preferredLoadAddress());
-            size          = (uint32_t)info.sectSize;
-            stop = true;
-        }
-    });
-    return (size != 0);
-}
-
-
-bool MachOAnalyzer::getEntry(uint64_t& offset, bool& usesCRT) const
-{
-    Diagnostics diag;
-    offset = 0;
-    forEachLoadCommand(diag, ^(const load_command* cmd, bool& stop) {
-        if ( cmd->cmd == LC_MAIN ) {
-            entry_point_command* mainCmd = (entry_point_command*)cmd;
-            usesCRT = false;
-            offset = mainCmd->entryoff;
-            stop = true;
-        }
-        else if ( cmd->cmd == LC_UNIXTHREAD ) {
-            stop = true;
-            usesCRT = true;
-            uint64_t startAddress = entryAddrFromThreadCmd((thread_command*)cmd);
-            offset = startAddress - preferredLoadAddress();
-        }
-    });
-    return (offset != 0);
-}
-
-
-void MachOAnalyzer::forEachDOFSection(Diagnostics& diag, void (^callback)(uint32_t offset)) const
-{
-    forEachSection(^(const dyld3::MachOAnalyzer::SectionInfo& info, bool malformedSectionRange, bool &stop) {
-        if ( ( (info.sectFlags & SECTION_TYPE) == S_DTRACE_DOF ) && !malformedSectionRange ) {
-            callback((uint32_t)(info.sectAddr - info.segInfo.vmAddr));
-        }
-    });
-}
-
-void MachOAnalyzer::forEachCDHash(void (^handler)(const uint8_t cdHash[20])) const
-{
-    Diagnostics diag;
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() || (leInfo.codeSig == nullptr) )
-        return;
-
-    forEachCDHashOfCodeSignature(getLinkEditContent(leInfo.layout, leInfo.codeSig->dataoff),
-                                 leInfo.codeSig->datasize,
-                                 handler);
-}
-
-bool MachOAnalyzer::usesLibraryValidation() const
-{
-    Diagnostics diag;
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() || (leInfo.codeSig == nullptr) )
-        return false;
-
-    // check for CS_REQUIRE_LV in CS_CodeDirectory.flags
-    __block bool requiresLV = false;
-    forEachCodeDirectoryBlob(getLinkEditContent(leInfo.layout, leInfo.codeSig->dataoff),
-                             leInfo.codeSig->datasize,
-                             ^(const void *cdBuffer) {
-         const CS_CodeDirectory* cd = (const CS_CodeDirectory*)cdBuffer;
-         requiresLV |= (htonl(cd->flags) & CS_REQUIRE_LV);
-    });
-
-    return requiresLV;
-}
-
-bool MachOAnalyzer::canHavePrecomputedDlopenClosure(const char* path, void (^failureReason)(const char*)) const
-{
-    if (!MachOFile::canHavePrecomputedDlopenClosure(path, failureReason))
-        return false;
-
-    // prebuilt closures use the cdhash of the dylib to verify that the dylib is still the same
-    // at runtime as when the shared cache processed it.  We must have a code signature to record this information
-    uint32_t codeSigFileOffset;
-    uint32_t codeSigSize;
-    if ( !hasCodeSignature(codeSigFileOffset, codeSigSize) ) {
-        failureReason("no code signature");
-        return false;
-    }
-
-    __block bool retval = true;
-
-    // images that use dynamic_lookup, bundle_loader, or have weak-defs cannot have dlopen closure pre-computed
-    Diagnostics diag;
-    auto checkBind = ^(int libOrdinal, bool& stop) {
-        switch (libOrdinal) {
-            case BIND_SPECIAL_DYLIB_WEAK_LOOKUP:
-                failureReason("has weak externals");
-                retval = false;
-                stop = true;
-                break;
-            case BIND_SPECIAL_DYLIB_FLAT_LOOKUP:
-                failureReason("has dynamic_lookup binds");
-                retval = false;
-                stop = true;
-                break;
-            case BIND_SPECIAL_DYLIB_MAIN_EXECUTABLE:
-                failureReason("has reference to main executable (bundle loader)");
-                retval = false;
-                stop = true;
-                break;
-        }
-    };
-
-    if (hasChainedFixups()) {
-        forEachChainedFixupTarget(diag, ^(int libOrdinal, const char *symbolName, uint64_t addend, bool weakImport, bool &stop) {
-            checkBind(libOrdinal, stop);
-        });
-    } else {
-        forEachBind(diag, ^(uint64_t runtimeOffset, int libOrdinal, const char* symbolName, bool weakImport, bool lazyBind, uint64_t addend, bool& stop) {
-            checkBind(libOrdinal, stop);
-        },
-        ^(const char* symbolName) {
-        });
-    }
-
-    return retval;
-}
-
-
-bool MachOAnalyzer::hasUnalignedPointerFixups() const
-{
-    // only look at 64-bit architectures
-    if ( pointerSize() == 4 )
-        return false;
-
-    __block Diagnostics diag;
-    __block bool result = false;
-    if ( hasChainedFixups() ) {
-        withChainStarts(diag, chainStartsOffset(), ^(const dyld_chained_starts_in_image* startsInfo) {
-            forEachFixupInAllChains(diag, startsInfo, false, ^(MachOLoaded::ChainedFixupPointerOnDisk* fixupLoc, const dyld_chained_starts_in_segment* segInfo, bool& fixupsStop) {
-                if ( ((long)(fixupLoc) & 7) != 0 ) {
-                    result = true;
-                    fixupsStop = true;
-                }
-           });
-        });
-    }
-    else {
-        forEachBind(diag, ^(uint64_t runtimeOffset, int libOrdinal, const char* symbolName, bool weakImport, bool lazyBind, uint64_t addend, bool& stop) {
-            if ( (runtimeOffset & 7) != 0 ) {
-                result = true;
-                stop = true;
-            }
-        },
-        ^(const char* symbolName) {
-        });
-        forEachRebase(diag, true, ^(uint64_t runtimeOffset, bool& stop) {
-            if ( (runtimeOffset & 7) != 0 ) {
-                result = true;
-                stop = true;
-            }
-        });
-    }
-
-    return result;
-}
-
-void MachOAnalyzer::recurseTrie(Diagnostics& diag, const uint8_t* const start, const uint8_t* p, const uint8_t* const end,
-                                OverflowSafeArray<char>& cummulativeString, int curStrOffset, bool& stop, ExportsCallback callback) const
-{
-    if ( p >= end ) {
-        diag.error("malformed trie, node past end");
-        return;
-    }
-    const uint64_t terminalSize = read_uleb128(diag, p, end);
-    const uint8_t* children = p + terminalSize;
-    if ( terminalSize != 0 ) {
-        uint64_t    imageOffset = 0;
-        uint64_t    flags       = read_uleb128(diag, p, end);
-        uint64_t    other       = 0;
-        const char* importName  = nullptr;
-        if ( flags & EXPORT_SYMBOL_FLAGS_REEXPORT ) {
-            other = read_uleb128(diag, p, end); // dylib ordinal
-            importName = (char*)p;
-        }
-        else {
-            imageOffset = read_uleb128(diag, p, end);
-            if ( flags & EXPORT_SYMBOL_FLAGS_STUB_AND_RESOLVER )
-                other = read_uleb128(diag, p, end);
-            else
-                other = 0;
-        }
-        if ( diag.hasError() )
-            return;
-        callback(cummulativeString.begin(), imageOffset, flags, other, importName, stop);
-        if ( stop )
-            return;
-    }
-    if ( children > end ) {
-        diag.error("malformed trie, terminalSize extends beyond trie data");
-        return;
-    }
-    const uint8_t childrenCount = *children++;
-    const uint8_t* s = children;
-    for (uint8_t i=0; i < childrenCount; ++i) {
-        int edgeStrLen = 0;
-        while (*s != '\0') {
-            cummulativeString.resize(curStrOffset+edgeStrLen + 1);
-            cummulativeString[curStrOffset+edgeStrLen] = *s++;
-            ++edgeStrLen;
-            if ( s > end ) {
-                diag.error("malformed trie node, child node extends past end of trie\n");
-                return;
-            }
-       }
-        cummulativeString.resize(curStrOffset+edgeStrLen + 1);
-        cummulativeString[curStrOffset+edgeStrLen] = *s++;
-        uint64_t childNodeOffset = read_uleb128(diag, s, end);
-        if (childNodeOffset == 0) {
-            diag.error("malformed trie, childNodeOffset==0");
-            return;
-        }
-        recurseTrie(diag, start, start+childNodeOffset, end, cummulativeString, curStrOffset+edgeStrLen, stop, callback);
-        if ( diag.hasError() || stop )
-            return;
-    }
-}
-
-void MachOAnalyzer::forEachExportedSymbol(Diagnostics& diag, ExportsCallback callback) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-    uint64_t trieSize;
-    if ( const uint8_t* trieStart = getExportsTrie(leInfo, trieSize) ) {
-        const uint8_t* trieEnd   = trieStart + trieSize;
-        // We still emit empty export trie load commands just as a placeholder to show we have
-        // no exports.  In that case, don't start recursing as we'll immediately think we ran
-        // of the end of the buffer
-        if ( trieSize == 0 )
-            return;
-        bool stop = false;
-        STACK_ALLOC_OVERFLOW_SAFE_ARRAY(char, cummulativeString, 4096);
-        recurseTrie(diag, trieStart, trieStart, trieEnd, cummulativeString, 0, stop, callback);
-   }
-}
-
-bool MachOAnalyzer::neverUnload() const
-{
-    if ( this->isDylib() && this->hasObjC() ) {
-        return true;
-    }
-    else if ( this->hasThreadLocalVariables() ) {
-        return true;
-    }
-    else {
-        // record if image has DOF sections
-        __block bool  hasDOFs = false;
-        Diagnostics   diag;
-        this->forEachDOFSection(diag, ^(uint32_t offset) {
-            hasDOFs = true;
-        });
-        if ( diag.noError() && hasDOFs )
-            return true;
-    }
-    return false;
-}
-
-bool MachOAnalyzer::canBePlacedInDyldCache(const char* path, void (^failureReason)(const char*)) const
-{
-    if (!MachOFile::canBePlacedInDyldCache(path, failureReason))
-        return false;
-
-    // arm64e requires split seg v2 as the split seg code can't handle chained fixups for split seg v1
-    if ( isArch("arm64e") ) {
-        uint32_t splitSegSize = 0;
-        const uint8_t* infoStart = (const uint8_t*)getSplitSeg(splitSegSize);
-        if ( *infoStart != DYLD_CACHE_ADJ_V2_FORMAT ) {
-            failureReason("chained fixups requires split seg v2");
-            return false;
-        }
-    }
-
-    // <rdar://problem/57769033> dyld_cache_patchable_location only supports addend in range 0..31
-    const bool is64bit = is64();
-    __block Diagnostics diag;
-    __block bool addendTooLarge = false;
-    if ( this->hasChainedFixups() ) {
-        // with chained fixups, addends can be in the import table or embedded in a bind pointer
-        forEachChainedFixupTarget(diag, ^(int libOrdinal, const char* symbolName, uint64_t addend, bool weakImport, bool& stop) {
-            if ( is64bit )
-                addend &= 0x00FFFFFFFFFFFFFF; // ignore TBI
-            if ( addend > 31 ) {
-                addendTooLarge = true;
-                stop = true;
-            }
-        });
-        // check each pointer for embedded addend
-        withChainStarts(diag, 0, ^(const dyld_chained_starts_in_image* starts) {
-            forEachFixupInAllChains(diag, starts, false, ^(ChainedFixupPointerOnDisk* fixupLoc, const dyld_chained_starts_in_segment* segInfo, bool& stop) {
-                switch (segInfo->pointer_format) {
-                    case DYLD_CHAINED_PTR_ARM64E:
-                    case DYLD_CHAINED_PTR_ARM64E_USERLAND:
-                    case DYLD_CHAINED_PTR_ARM64E_USERLAND24:
-                        if ( fixupLoc->arm64e.bind.bind && !fixupLoc->arm64e.authBind.auth ) {
-                            if ( fixupLoc->arm64e.bind.addend > 31 ) {
-                                addendTooLarge = true;
-                                stop = true;
-                            }
-                        }
-                        break;
-                    case DYLD_CHAINED_PTR_64:
-                    case DYLD_CHAINED_PTR_64_OFFSET:
-                        if ( fixupLoc->generic64.rebase.bind ) {
-                            if ( fixupLoc->generic64.bind.addend > 31 ) {
-                                addendTooLarge = true;
-                                stop = true;
-                            }
-                        }
-                        break;
-                    case DYLD_CHAINED_PTR_32:
-                        if ( fixupLoc->generic32.bind.bind ) {
-                            if ( fixupLoc->generic32.bind.addend > 31 ) {
-                                addendTooLarge = true;
-                                stop = true;
-                            }
-                        }
-                        break;
-                }
-            });
-        });
-    }
-    else {
-        // scan bind opcodes for large addend
-        forEachBind(diag, ^(const char* opcodeName, const LinkEditInfo& leInfo, const SegmentInfo* segments, bool segIndexSet, bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                            uint32_t ptrSize, uint8_t segmentIndex, uint64_t segmentOffset, uint8_t type, const char* symbolName, bool weakImport, bool lazyBind, uint64_t addend, bool& stop) {
-            if ( is64bit )
-                addend &= 0x00FFFFFFFFFFFFFF; // ignore TBI
-            if ( addend > 31 ) {
-                addendTooLarge = true;
-                stop = true;
-            }
-        },
-        ^(const char* symbolName) {
-        });
-    }
-    if ( addendTooLarge ) {
-        failureReason("bind addend too large");
-        return false;
-    }
-
-    // evict swift dylibs with split seg v1 info
-    if ( this->isSwiftLibrary() && this->isSplitSegV1() )
-        return false;
-
-    if ( hasChainedFixups() ) {
-        // Chained fixups assumes split seg v2.  This is true for now as chained fixups is arm64e only
-        return true; // this->isSplitSegV2();
-    }
-
-    if ( !(isArch("x86_64") || isArch("x86_64h")) )
-        return true;
-
-    __block bool rebasesOk = true;
-    uint64_t startVMAddr = preferredLoadAddress();
-    uint64_t endVMAddr = startVMAddr + mappedSize();
-    forEachRebase(diag, false, ^(uint64_t runtimeOffset, bool &stop) {
-        // We allow TBI for x86_64 dylibs, but then require that the remainder of the offset
-        // is a 32-bit offset from the mach-header.
-        uint64_t value = *(uint64_t*)((uint8_t*)this + runtimeOffset);
-        value &= 0x00FFFFFFFFFFFFFFULL;
-        if ( (value < startVMAddr) || (value >= endVMAddr) ) {
-            failureReason("rebase value out of range of dylib");
-            rebasesOk = false;
-            stop = true;
-            return;
-        }
-
-        // Also error if the rebase location is anything other than 4/8 byte aligned
-        if ( (runtimeOffset & 0x3) != 0 ) {
-            failureReason("rebase value is not 4-byte aligned");
-            rebasesOk = false;
-            stop = true;
-            return;
-        }
-    });
-    return rebasesOk;
-}
-
-#if BUILDING_APP_CACHE_UTIL
-bool MachOAnalyzer::canBePlacedInKernelCollection(const char* path, void (^failureReason)(const char*)) const
-{
-    if (!MachOFile::canBePlacedInKernelCollection(path, failureReason))
-        return false;
-
-    // App caches reguire that everything be built with split seg v2
-    // This is because v1 can't move anything other than __TEXT and __DATA
-    // but kernels have __TEXT_EXEC and other segments
-    if ( isKextBundle() ) {
-        // x86_64 kext's might not have split seg
-        if ( !isArch("x86_64") && !isArch("x86_64h") ) {
-            if ( !isSplitSegV2() ) {
-                failureReason("Missing split seg v2");
-                return false;
-            }
-        }
-    } else if ( isStaticExecutable() ) {
-        // The kernel must always have split seg V2
-        if ( !isSplitSegV2() ) {
-            failureReason("Missing split seg v2");
-            return false;
-        }
-
-        // The kernel should have __TEXT and __TEXT_EXEC
-        __block bool foundText = false;
-        __block bool foundTextExec = false;
-        __block bool foundHIB = false;
-        __block uint64_t hibernateVMAddr = 0;
-        __block uint64_t hibernateVMSize = 0;
-        forEachSegment(^(const SegmentInfo &segmentInfo, bool &stop) {
-            if ( strcmp(segmentInfo.segName, "__TEXT") == 0 ) {
-                foundText = true;
-            }
-            if ( strcmp(segmentInfo.segName, "__TEXT_EXEC") == 0 ) {
-                foundTextExec = true;
-            }
-            if ( strcmp(segmentInfo.segName, "__HIB") == 0 ) {
-                foundHIB = true;
-                hibernateVMAddr = segmentInfo.vmAddr;
-                hibernateVMSize = segmentInfo.vmSize;
-            }
-        });
-        if (!foundText) {
-            failureReason("Expected __TEXT segment");
-            return false;
-        }
-        if ( foundTextExec && foundHIB ) {
-            failureReason("Expected __TEXT_EXEC or __HIB segment, but found both");
-            return false;
-        }
-        if ( !foundTextExec && !foundHIB ) {
-            failureReason("Expected __TEXT_EXEC or __HIB segment, but found neither");
-            return false;
-        }
-
-        // The hibernate segment should be mapped before the base address
-        if ( foundHIB ) {
-            uint64_t baseAddress = preferredLoadAddress();
-            if ( greaterThanAddOrOverflow(hibernateVMAddr, hibernateVMSize, baseAddress) ) {
-                failureReason("__HIB segment should be mapped before base address");
-                return false;
-            }
-        }
-    }
-
-    // Don't allow kext's to have load addresses
-    if ( isKextBundle() && (preferredLoadAddress() != 0) ) {
-        failureReason("Has load address");
-        return false;
-    }
-
-    if (hasChainedFixups()) {
-        if ( usesClassicRelocationsInKernelCollection() ) {
-            failureReason("Cannot use fixup chains with binary expecting classic relocations");
-            return false;
-        }
-
-        __block bool fixupsOk = true;
-        __block Diagnostics diag;
-        withChainStarts(diag, 0, ^(const dyld_chained_starts_in_image* starts) {
-            forEachFixupInAllChains(diag, starts, false, ^(dyld3::MachOLoaded::ChainedFixupPointerOnDisk* fixupLoc,
-                                                           const dyld_chained_starts_in_segment* segInfo, bool& stop) {
-                // We only support inputs from a few pointer format types, so that we don't need to handle them all later
-                switch (segInfo->pointer_format) {
-                    case DYLD_CHAINED_PTR_ARM64E:
-                    case DYLD_CHAINED_PTR_64:
-                    case DYLD_CHAINED_PTR_32:
-                    case DYLD_CHAINED_PTR_32_CACHE:
-                    case DYLD_CHAINED_PTR_32_FIRMWARE:
-                        failureReason("unsupported chained fixups pointer format");
-                        fixupsOk = false;
-                        stop = true;
-                        return;
-                    case DYLD_CHAINED_PTR_64_OFFSET:
-                        // arm64 kernel and kexts use this format
-                        break;
-                    case DYLD_CHAINED_PTR_ARM64E_KERNEL:
-                        // arm64e kexts use this format
-                        break;
-                    case DYLD_CHAINED_PTR_64_KERNEL_CACHE:
-                    case DYLD_CHAINED_PTR_X86_64_KERNEL_CACHE:
-                        failureReason("unsupported chained fixups pointer format");
-                        fixupsOk = false;
-                        stop = true;
-                        return;
-                    default:
-                        failureReason("unknown chained fixups pointer format");
-                        fixupsOk = false;
-                        stop = true;
-                        return;
-                }
-
-                uint64_t vmOffset = (uint8_t*)fixupLoc - (uint8_t*)this;
-                // Error if the fixup location is anything other than 4/8 byte aligned
-                if ( (vmOffset & 0x3) != 0 ) {
-                    failureReason("fixup value is not 4-byte aligned");
-                    fixupsOk = false;
-                    stop = true;
-                    return;
-                }
-
-                // We also must only need 30-bits for the chain format of the resulting cache
-                if ( vmOffset >= (1 << 30) ) {
-                    failureReason("fixup value does not fit in 30-bits");
-                    fixupsOk = false;
-                    stop = true;
-                    return;
-                }
-            });
-        });
-        if (!fixupsOk)
-            return false;
-    } else {
-        // x86_64 xnu will have unaligned text/data fixups and fixups inside __HIB __text.
-        // We allow these as xnu is emitted with classic relocations
-        bool canHaveUnalignedFixups = usesClassicRelocationsInKernelCollection();
-        canHaveUnalignedFixups |= ( isArch("x86_64") || isArch("x86_64h") );
-        __block bool rebasesOk = true;
-        Diagnostics diag;
-        forEachRebase(diag, false, ^(uint64_t runtimeOffset, bool &stop) {
-            // Error if the rebase location is anything other than 4/8 byte aligned
-            if ( !canHaveUnalignedFixups && ((runtimeOffset & 0x3) != 0) ) {
-                failureReason("rebase value is not 4-byte aligned");
-                rebasesOk = false;
-                stop = true;
-                return;
-            }
-
-#if BUILDING_APP_CACHE_UTIL
-            // xnu for x86_64 has __HIB mapped before __DATA, so offsets appear to be
-            // negative.  Adjust the fixups so that we don't think they are out of
-            // range of the number of bits we have
-            if ( isStaticExecutable() ) {
-                __block uint64_t baseAddress = ~0ULL;
-                forEachSegment(^(const SegmentInfo& sinfo, bool& segStop) {
-                    baseAddress = std::min(baseAddress, sinfo.vmAddr);
-                });
-                uint64_t textSegVMAddr = preferredLoadAddress();
-                runtimeOffset = (textSegVMAddr + runtimeOffset) - baseAddress;
-            }
-#endif
-
-            // We also must only need 30-bits for the chain format of the resulting cache
-            if ( runtimeOffset >= (1 << 30) ) {
-                failureReason("rebase value does not fit in 30-bits");
-                rebasesOk = false;
-                stop = true;
-                return;
-            }
-        });
-        if (!rebasesOk)
-            return false;
-
-        __block bool bindsOk = true;
-        forEachBind(diag, ^(uint64_t runtimeOffset, int libOrdinal, uint8_t type, const char *symbolName,
-                            bool weakImport, bool lazyBind, uint64_t addend, bool &stop) {
-
-            // Don't validate branch fixups as we'll turn then in to direct jumps instead
-            if ( type == BIND_TYPE_TEXT_PCREL32 )
-                return;
-
-            // Error if the bind location is anything other than 4/8 byte aligned
-            if ( !canHaveUnalignedFixups && ((runtimeOffset & 0x3) != 0) ) {
-                failureReason("bind value is not 4-byte aligned");
-                bindsOk = false;
-                stop = true;
-                return;
-            }
-
-            // We also must only need 30-bits for the chain format of the resulting cache
-            if ( runtimeOffset >= (1 << 30) ) {
-                failureReason("bind value does not fit in 30-bits");
-                rebasesOk = false;
-                stop = true;
-                return;
-            }
-        }, ^(const char *symbolName) {
-        });
-        if (!bindsOk)
-            return false;
-    }
-
-    return true;
-}
-
-#endif
-
-bool MachOAnalyzer::usesClassicRelocationsInKernelCollection() const {
-    // The xnu x86_64 static executable needs to do the i386->x86_64 transition
-    // so will be emitted with classic relocations
-    if ( isArch("x86_64") || isArch("x86_64h") ) {
-        return isStaticExecutable() || isFileSet();
-    }
-    return false;
-}
-
-uint64_t MachOAnalyzer::chainStartsOffset() const
-{
-    const dyld_chained_fixups_header* header = chainedFixupsHeader();
-    // old arm64e binary has no dyld_chained_fixups_header
-    if ( header == nullptr )
-        return 0;
-    return header->starts_offset + ((uint8_t*)header - (uint8_t*)this);
-}
-
-const dyld_chained_fixups_header* MachOAnalyzer::chainedFixupsHeader() const
-{
-    Diagnostics diag;
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() || (leInfo.chainedFixups == nullptr) )
-        return nullptr;
-
-    return (dyld_chained_fixups_header*)getLinkEditContent(leInfo.layout, leInfo.chainedFixups->dataoff);
-}
-
-uint16_t MachOAnalyzer::chainedPointerFormat(const dyld_chained_fixups_header* header)
-{
-    const dyld_chained_starts_in_image* startsInfo = (dyld_chained_starts_in_image*)((uint8_t*)header + header->starts_offset);
-    for (uint32_t i=0; i < startsInfo->seg_count; ++i) {
-        uint32_t segInfoOffset = startsInfo->seg_info_offset[i];
-        // 0 offset means this segment has no fixups
-        if ( segInfoOffset == 0 )
-            continue;
-        const dyld_chained_starts_in_segment* segInfo = (dyld_chained_starts_in_segment*)((uint8_t*)startsInfo + segInfoOffset);
-        if ( segInfo->page_count != 0 )
-            return segInfo->pointer_format;
-    }
-    return 0;  // no chains (perhaps no __DATA segment)
-}
-
-uint16_t MachOAnalyzer::chainedPointerFormat() const
-{
-    const dyld_chained_fixups_header* header = chainedFixupsHeader();
-    if ( header != nullptr ) {
-        // get pointer format from chain info struct in LINKEDIT
-        return chainedPointerFormat(header);
-    }
-    assert(this->cputype == CPU_TYPE_ARM64 && (this->maskedCpuSubtype() == CPU_SUBTYPE_ARM64E) && "chainedPointerFormat() called on non-chained binary");
-    return DYLD_CHAINED_PTR_ARM64E;
-}
-
-#if (BUILDING_DYLD || BUILDING_LIBDYLD) && !__arm64e__
-  #define SUPPORT_OLD_ARM64E_FORMAT 0
-#else
-  #define SUPPORT_OLD_ARM64E_FORMAT 1
-#endif
-
-// find dyld_chained_starts_in_image* in image
-// if old arm64e binary, synthesize dyld_chained_starts_in_image*
-void MachOAnalyzer::withChainStarts(Diagnostics& diag, uint64_t startsStructOffsetHint, void (^callback)(const dyld_chained_starts_in_image*)) const
-{
-    if ( startsStructOffsetHint != 0 ) {
-        // we have a pre-computed offset into LINKEDIT for dyld_chained_starts_in_image
-        callback((dyld_chained_starts_in_image*)((uint8_t*)this + startsStructOffsetHint));
-        return;
-    }
-
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    if ( leInfo.chainedFixups != nullptr ) {
-        // find dyld_chained_starts_in_image from dyld_chained_fixups_header
-        const dyld_chained_fixups_header* header = (dyld_chained_fixups_header*)getLinkEditContent(leInfo.layout, leInfo.chainedFixups->dataoff);
-        callback((dyld_chained_starts_in_image*)((uint8_t*)header + header->starts_offset));
-    }
-#if SUPPORT_OLD_ARM64E_FORMAT
-    // don't want this code in non-arm64e dyld because it causes a stack protector which dereferences a GOT pointer before GOT is set up
-    else if ( (leInfo.dyldInfo != nullptr) && (this->cputype == CPU_TYPE_ARM64) && (this->maskedCpuSubtype() == CPU_SUBTYPE_ARM64E) ) {
-        // old arm64e binary, create a dyld_chained_starts_in_image for caller
-        uint64_t baseAddress = preferredLoadAddress();
-        uint64_t imagePageCount = this->mappedSize()/0x4000;
-        size_t bufferSize = leInfo.dyldInfo->bind_size + (size_t)imagePageCount*sizeof(uint16_t) + 512;
-        BLOCK_ACCCESSIBLE_ARRAY(uint8_t, buffer, bufferSize);
-        uint8_t* bufferEnd = &buffer[bufferSize];
-        dyld_chained_starts_in_image* header = (dyld_chained_starts_in_image*)buffer;
-        header->seg_count = leInfo.layout.linkeditSegIndex;
-        for (uint32_t i=0; i < header->seg_count; ++i)
-            header->seg_info_offset[i] = 0;
-        __block uint8_t curSegIndex = 0;
-        __block dyld_chained_starts_in_segment* curSeg = (dyld_chained_starts_in_segment*)(&(header->seg_info_offset[header->seg_count]));
-        parseOrgArm64eChainedFixups(diag, nullptr, nullptr, ^(const LinkEditInfo& leInfo2, const SegmentInfo segments[], uint8_t segmentIndex,
-                                                              bool segIndexSet, uint64_t segmentOffset, uint16_t format, bool& stop) {
-            uint32_t pageIndex = (uint32_t)(segmentOffset/0x1000);
-            if ( segmentIndex != curSegIndex ) {
-                if ( curSegIndex == 0 ) {
-                    header->seg_info_offset[segmentIndex] = (uint32_t)((uint8_t*)curSeg - buffer);
-                }
-                else {
-                    header->seg_info_offset[segmentIndex] = (uint32_t)((uint8_t*)(&curSeg->page_start[curSeg->page_count]) - buffer);
-                    curSeg = (dyld_chained_starts_in_segment*)((uint8_t*)header+header->seg_info_offset[segmentIndex]);
-                    assert((uint8_t*)curSeg < bufferEnd);
-               }
-               curSeg->page_count = 0;
-               curSegIndex = segmentIndex;
-            }
-            while ( curSeg->page_count != pageIndex ) {
-                assert((uint8_t*)(&curSeg->page_start[curSeg->page_count]) < bufferEnd);
-                curSeg->page_start[curSeg->page_count] = 0xFFFF;
-                curSeg->page_count++;
-            }
-            curSeg->size                  = (uint32_t)((uint8_t*)(&curSeg->page_start[pageIndex]) - (uint8_t*)curSeg);
-            curSeg->page_size             = 0x1000; // old arm64e encoding used 4KB pages
-            curSeg->pointer_format        = DYLD_CHAINED_PTR_ARM64E;
-            curSeg->segment_offset        = segments[segmentIndex].vmAddr - baseAddress;
-            curSeg->max_valid_pointer     = 0;
-            curSeg->page_count            = pageIndex+1;
-            assert((uint8_t*)(&curSeg->page_start[pageIndex]) < bufferEnd);
-            curSeg->page_start[pageIndex] = segmentOffset & 0xFFF;
-            //fprintf(stderr, "segment_offset=0x%llX, vmAddr=0x%llX\n", curSeg->segment_offset, segments[segmentIndex].vmAddr );
-            //printf("segIndex=%d, segOffset=0x%08llX, page_start[%d]=0x%04X, page_start[%d]=0x%04X\n",
-            //        segmentIndex, segmentOffset, pageIndex, curSeg->page_start[pageIndex], pageIndex-1, pageIndex ? curSeg->page_start[pageIndex-1] : 0);
-        });
-        callback(header);
-    }
-#endif
-    else {
-        diag.error("image does not use chained fixups");
-    }
-}
-
-struct OldThreadsStartSection
-{
-    uint32_t        padding : 31,
-                    stride8 : 1;
-    uint32_t        chain_starts[1];
-};
-
-// ld64 can't sometimes determine the size of __thread_starts accurately,
-// because these sections have to be given a size before everything is laid out,
-// and you don't know the actual size of the chains until everything is
-// laid out. In order to account for this, the linker puts trailing 0xFFFFFFFF at
-// the end of the section, that must be ignored when walking the chains. This
-// patch adjust the section size accordingly.
-static uint32_t adjustStartsCount(uint32_t startsCount, const uint32_t* starts) {
-    for ( int i = startsCount; i > 0; --i )
-    {
-        if ( starts[i - 1] == 0xFFFFFFFF )
-            startsCount--;
-        else
-            break;
-    }
-    return startsCount;
-}
-
-bool MachOAnalyzer::hasFirmwareChainStarts(uint16_t* pointerFormat, uint32_t* startsCount, const uint32_t** starts) const
-{
-    if ( !this->isPreload() && !this->isStaticExecutable() )
-        return false;
-
-    uint64_t sectionSize;
-    if (const dyld_chained_starts_offsets* sect = (dyld_chained_starts_offsets*)this->findSectionContent("__TEXT", "__chain_starts", sectionSize) ) {
-        *pointerFormat = sect->pointer_format;
-        *startsCount   = sect->starts_count;
-        *starts        = &sect->chain_starts[0];
-        return true;
-    }
-    if (const OldThreadsStartSection* sect = (OldThreadsStartSection*)this->findSectionContent("__TEXT", "__thread_starts", sectionSize) ) {
-        *pointerFormat = sect->stride8 ? DYLD_CHAINED_PTR_ARM64E : DYLD_CHAINED_PTR_ARM64E_FIRMWARE;
-        *startsCount   = adjustStartsCount((uint32_t)(sectionSize/4) - 1, sect->chain_starts);
-        *starts        = sect->chain_starts;
-        return true;
-    }
-    return false;
-}
-
-
-MachOAnalyzer::ObjCInfo MachOAnalyzer::getObjCInfo() const
-{
-    __block ObjCInfo result;
-    result.selRefCount      = 0;
-    result.classDefCount    = 0;
-    result.protocolDefCount = 0;
-
-    const uint32_t ptrSize  = pointerSize();
-    forEachSection(^(const SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        if ( strncmp(sectInfo.segInfo.segName, "__DATA", 6) == 0 ) {
-            if ( strcmp(sectInfo.sectName, "__objc_selrefs") == 0 )
-                result.selRefCount += (sectInfo.sectSize/ptrSize);
-            else if ( strcmp(sectInfo.sectName, "__objc_classlist") == 0 )
-                result.classDefCount += (sectInfo.sectSize/ptrSize);
-            else if ( strcmp(sectInfo.sectName, "__objc_protolist") == 0 )
-                result.protocolDefCount += (sectInfo.sectSize/ptrSize);
-        }
-        else if ( (this->cputype == CPU_TYPE_I386) && (strcmp(sectInfo.segInfo.segName, "__OBJC") == 0) ) {
-            if ( strcmp(sectInfo.sectName, "__message_refs") == 0 )
-                result.selRefCount += (sectInfo.sectSize/4);
-            else if ( strcmp(sectInfo.sectName, "__class") == 0 )
-                result.classDefCount += (sectInfo.sectSize/48);
-            else if ( strcmp(sectInfo.sectName, "__protocol") == 0 )
-                result.protocolDefCount += (sectInfo.sectSize/20);
-        }
-   });
-
-    return result;
-}
-
-uint64_t MachOAnalyzer::ObjCClassInfo::getReadOnlyDataField(ObjCClassInfo::ReadOnlyDataField field, uint32_t pointerSize) const {
-    if (pointerSize == 8) {
-        typedef uint64_t PtrTy;
-        struct class_ro_t {
-            uint32_t flags;
-            uint32_t instanceStart;
-            // Note there is 4-bytes of alignment padding between instanceSize and ivarLayout
-            // on 64-bit archs, but no padding on 32-bit archs.
-            // This union is a way to model that.
-            union {
-                uint32_t   instanceSize;
-                PtrTy   pad;
-            } instanceSize;
-            PtrTy ivarLayoutVMAddr;
-            PtrTy nameVMAddr;
-            PtrTy baseMethodsVMAddr;
-            PtrTy baseProtocolsVMAddr;
-            PtrTy ivarsVMAddr;
-            PtrTy weakIvarLayoutVMAddr;
-            PtrTy basePropertiesVMAddr;
-        };
-        const class_ro_t* classData = (const class_ro_t*)(dataVMAddr + vmAddrConverter.slide);
-        switch (field) {
-        case ObjCClassInfo::ReadOnlyDataField::name:
-            return vmAddrConverter.convertToVMAddr(classData->nameVMAddr);
-        case ObjCClassInfo::ReadOnlyDataField::baseProtocols:
-            return vmAddrConverter.convertToVMAddr(classData->baseProtocolsVMAddr);
-        case ObjCClassInfo::ReadOnlyDataField::baseMethods:
-            return vmAddrConverter.convertToVMAddr(classData->baseMethodsVMAddr);
-        case ObjCClassInfo::ReadOnlyDataField::baseProperties:
-            return vmAddrConverter.convertToVMAddr(classData->basePropertiesVMAddr);
-        case ObjCClassInfo::ReadOnlyDataField::flags:
-            return classData->flags;
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        struct class_ro_t {
-            uint32_t flags;
-            uint32_t instanceStart;
-            // Note there is 4-bytes of alignment padding between instanceSize and ivarLayout
-            // on 64-bit archs, but no padding on 32-bit archs.
-            // This union is a way to model that.
-            union {
-                uint32_t   instanceSize;
-                PtrTy   pad;
-            } instanceSize;
-            PtrTy ivarLayoutVMAddr;
-            PtrTy nameVMAddr;
-            PtrTy baseMethodsVMAddr;
-            PtrTy baseProtocolsVMAddr;
-            PtrTy ivarsVMAddr;
-            PtrTy weakIvarLayoutVMAddr;
-            PtrTy basePropertiesVMAddr;
-        };
-        const class_ro_t* classData = (const class_ro_t*)(dataVMAddr + vmAddrConverter.slide);
-        switch (field) {
-            case ObjCClassInfo::ReadOnlyDataField::name:
-                return vmAddrConverter.convertToVMAddr(classData->nameVMAddr);
-            case ObjCClassInfo::ReadOnlyDataField::baseProtocols:
-                return vmAddrConverter.convertToVMAddr(classData->baseProtocolsVMAddr);
-            case ObjCClassInfo::ReadOnlyDataField::baseMethods:
-                return vmAddrConverter.convertToVMAddr(classData->baseMethodsVMAddr);
-            case ObjCClassInfo::ReadOnlyDataField::baseProperties:
-                return vmAddrConverter.convertToVMAddr(classData->basePropertiesVMAddr);
-            case ObjCClassInfo::ReadOnlyDataField::flags:
-                return classData->flags;
-        }
-    }
-}
-
-const char* MachOAnalyzer::getPrintableString(uint64_t stringVMAddr, MachOAnalyzer::PrintableStringResult& result) const {
-    uint32_t fairplayTextOffsetStart;
-    uint32_t fairplayTextOffsetEnd;
-    uint32_t fairplaySize;
-    if ( isFairPlayEncrypted(fairplayTextOffsetStart, fairplaySize) ) {
-        fairplayTextOffsetEnd = fairplayTextOffsetStart + fairplaySize;
-    } else {
-        fairplayTextOffsetEnd = 0;
-    }
-
-    result = PrintableStringResult::UnknownSection;
-    forEachSection(^(const MachOAnalyzer::SectionInfo &sectInfo, bool malformedSectionRange, bool &stop) {
-        if ( stringVMAddr < sectInfo.sectAddr ) {
-            return;
-        }
-        if ( stringVMAddr >= ( sectInfo.sectAddr + sectInfo.sectSize) ) {
-            return;
-        }
-
-        // We can't scan this section if its protected
-        if ( sectInfo.segInfo.isProtected ) {
-            result = PrintableStringResult::ProtectedSection;
-            stop = true;
-            return;
-        }
-
-        // We can't scan this section if it overlaps with the fairplay range
-        if ( fairplayTextOffsetEnd < sectInfo.sectFileOffset ) {
-            // Fairplay range ends before section
-        } else if ( fairplayTextOffsetStart > (sectInfo.sectFileOffset + sectInfo.sectSize) ) {
-            // Fairplay range starts after section
-        } else {
-            // Must overlap
-            result = PrintableStringResult::FairPlayEncrypted;
-            stop = true;
-            return;
-        }
-    
-        result = PrintableStringResult::CanPrint;
-        stop = true;
-    });
-
-#if BUILDING_SHARED_CACHE_UTIL || BUILDING_DYLDINFO
-    // The shared cache coalesces strings in to their own section.
-    // Assume its a valid pointer
-    if (result == PrintableStringResult::UnknownSection) {
-        result = PrintableStringResult::CanPrint;
-        return (const char*)(stringVMAddr + getSlide());
-    }
-#endif
-
-    if (result == PrintableStringResult::CanPrint)
-        return (const char*)(stringVMAddr + getSlide());
-    return nullptr;
-}
-
-void MachOAnalyzer::forEachObjCClass(uint64_t classListRuntimeOffset, uint64_t classListCount,
-                                     const VMAddrConverter& vmAddrConverter, ClassCallback& callback) const {
-    const uint64_t ptrSize = pointerSize();
-    const uint8_t* arrayBase = (uint8_t*)this + classListRuntimeOffset;
-    if ( ptrSize == 8 ) {
-        typedef uint64_t PtrTy;
-        __block bool stop = false;
-        for ( uint64_t i = 0; i != classListCount; ++i ) {
-            uint64_t classVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(arrayBase + (i * sizeof(PtrTy))));
-            parseObjCClass(vmAddrConverter, classVMAddr, ^(uint64_t classSuperclassVMAddr, uint64_t classDataVMAddr, const ObjCClassInfo& objcClass) {
-                callback(classVMAddr, classSuperclassVMAddr, classDataVMAddr, objcClass, false, stop);
-                if ( stop )
-                    return;
-
-                // Then parse and call for the metaclass
-                uint64_t isaVMAddr = objcClass.isaVMAddr;
-                parseObjCClass(vmAddrConverter, isaVMAddr, ^(uint64_t metaclassSuperclassVMAddr, uint64_t metaclassDataVMAddr, const ObjCClassInfo& objcMetaclass) {
-                    callback(isaVMAddr, metaclassSuperclassVMAddr, metaclassDataVMAddr, objcMetaclass, true, stop);
-                });
-            });
-            if ( stop )
-                break;
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        __block bool stop = false;
-        for ( uint64_t i = 0; i != classListCount; ++i ) {
-            uint64_t classVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(arrayBase + (i * sizeof(PtrTy))));
-            parseObjCClass(vmAddrConverter, classVMAddr, ^(uint64_t classSuperclassVMAddr, uint64_t classDataVMAddr, const ObjCClassInfo& objcClass) {
-                callback(classVMAddr, classSuperclassVMAddr, classDataVMAddr, objcClass, false, stop);
-                if ( stop )
-                    return;
-
-                // Then parse and call for the metaclass
-                uint64_t isaVMAddr = objcClass.isaVMAddr;
-                parseObjCClass(vmAddrConverter, isaVMAddr, ^(uint64_t metaclassSuperclassVMAddr, uint64_t metaclassDataVMAddr, const ObjCClassInfo& objcMetaclass) {
-                    callback(isaVMAddr, metaclassSuperclassVMAddr, metaclassDataVMAddr, objcMetaclass, true, stop);
-                });
-            });
-            if ( stop )
-                break;
-        }
-    }
-}
-
-void MachOAnalyzer::forEachObjCClass(Diagnostics& diag, const VMAddrConverter& vmAddrConverter,
-                                     ClassCallback& callback) const {
-    uint64_t classListRuntimeOffset;
-    uint64_t classListSize;
-    bool foundSection = findObjCDataSection("__objc_classlist", classListRuntimeOffset, classListSize);
-    if ( !foundSection )
-        return;
-
-    const uint64_t ptrSize = pointerSize();
-    if ( (classListSize % ptrSize) != 0 ) {
-        diag.error("Invalid objc class section size");
-        return;
-    }
-
-    forEachObjCClass(classListRuntimeOffset, classListSize / ptrSize, vmAddrConverter, callback);
-}
-
-void MachOAnalyzer::parseObjCClass(const VMAddrConverter& vmAddrConverter,
-                                   uint64_t classVMAddr,
-                                   void (^handler)(uint64_t classSuperclassVMAddr,
-                                                   uint64_t classDataVMAddr,
-                                                   const ObjCClassInfo& objcClass)) const {
-    const uint64_t ptrSize = pointerSize();
-    intptr_t slide = getSlide();
-
-    uint64_t classSuperclassVMAddr = 0;
-    uint64_t classDataVMAddr       = 0;
-    ObjCClassInfo objcClass;
-
-    if ( ptrSize == 8 ) {
-       struct objc_class_t {
-           uint64_t isaVMAddr;
-           uint64_t superclassVMAddr;
-           uint64_t methodCacheBuckets;
-           uint64_t methodCacheProperties;
-           uint64_t dataVMAddrAndFastFlags;
-       };
-        // This matches "struct TargetClassMetadata" from Metadata.h in Swift
-        struct swift_class_metadata_t : objc_class_t {
-            uint32_t swiftClassFlags;
-        };
-        enum : uint64_t {
-            FAST_DATA_MASK = 0x00007ffffffffff8ULL
-        };
-        classSuperclassVMAddr = classVMAddr + offsetof(objc_class_t, superclassVMAddr);
-        classDataVMAddr       = classVMAddr + offsetof(objc_class_t, dataVMAddrAndFastFlags);
-
-        // First call the handler on the class
-        const objc_class_t*           classPtr      = (const objc_class_t*)(classVMAddr + slide);
-        const swift_class_metadata_t* swiftClassPtr = (const swift_class_metadata_t*)classPtr;
-        objcClass.isaVMAddr         = vmAddrConverter.convertToVMAddr(classPtr->isaVMAddr);
-        objcClass.superclassVMAddr  = vmAddrConverter.convertToVMAddr(classPtr->superclassVMAddr);
-        objcClass.methodCacheVMAddr  = classPtr->methodCacheProperties == 0 ? 0 : vmAddrConverter.convertToVMAddr(classPtr->methodCacheProperties);
-        objcClass.dataVMAddr        = vmAddrConverter.convertToVMAddr(classPtr->dataVMAddrAndFastFlags) & FAST_DATA_MASK;
-        objcClass.vmAddrConverter   = vmAddrConverter;
-        objcClass.isSwiftLegacy     = classPtr->dataVMAddrAndFastFlags & ObjCClassInfo::FAST_IS_SWIFT_LEGACY;
-        objcClass.isSwiftStable     = classPtr->dataVMAddrAndFastFlags & ObjCClassInfo::FAST_IS_SWIFT_STABLE;
-        // The Swift class flags are only present if the class is swift
-        objcClass.swiftClassFlags   = (objcClass.isSwiftLegacy || objcClass.isSwiftStable) ? swiftClassPtr->swiftClassFlags : 0;
-    } else {
-        struct objc_class_t {
-            uint32_t isaVMAddr;
-            uint32_t superclassVMAddr;
-            uint32_t methodCacheBuckets;
-            uint32_t methodCacheProperties;
-            uint32_t dataVMAddrAndFastFlags;
-        };
-        // This matches "struct TargetClassMetadata" from Metadata.h in Swift
-        struct swift_class_metadata_t : objc_class_t {
-            uint32_t swiftClassFlags;
-        };
-        enum : uint32_t {
-            FAST_DATA_MASK = 0xfffffffcUL
-        };
-        classSuperclassVMAddr = classVMAddr + offsetof(objc_class_t, superclassVMAddr);
-        classDataVMAddr       = classVMAddr + offsetof(objc_class_t, dataVMAddrAndFastFlags);
-
-        // First call the handler on the class
-        const objc_class_t*           classPtr      = (const objc_class_t*)(classVMAddr + slide);
-        const swift_class_metadata_t* swiftClassPtr = (const swift_class_metadata_t*)classPtr;
-        objcClass.isaVMAddr         = vmAddrConverter.convertToVMAddr(classPtr->isaVMAddr);
-        objcClass.superclassVMAddr  = vmAddrConverter.convertToVMAddr(classPtr->superclassVMAddr);
-        objcClass.methodCacheVMAddr  = classPtr->methodCacheProperties == 0 ? 0 : vmAddrConverter.convertToVMAddr(classPtr->methodCacheProperties);
-        objcClass.dataVMAddr        = vmAddrConverter.convertToVMAddr(classPtr->dataVMAddrAndFastFlags) & FAST_DATA_MASK;
-        objcClass.vmAddrConverter   = vmAddrConverter;
-        objcClass.isSwiftLegacy     = classPtr->dataVMAddrAndFastFlags & ObjCClassInfo::FAST_IS_SWIFT_LEGACY;
-        objcClass.isSwiftStable     = classPtr->dataVMAddrAndFastFlags & ObjCClassInfo::FAST_IS_SWIFT_STABLE;
-        // The Swift class flags are only present if the class is swift
-        objcClass.swiftClassFlags   = (objcClass.isSwiftLegacy || objcClass.isSwiftStable) ? swiftClassPtr->swiftClassFlags : 0;
-    }
-                                       
-    handler(classSuperclassVMAddr, classDataVMAddr, objcClass);
-}
-
-void MachOAnalyzer::forEachObjCCategory(uint64_t categoryListRuntimeOffset, uint64_t categoryListCount,
-                                        const VMAddrConverter& vmAddrConverter, CategoryCallback& callback) const
-{
-    const uint64_t ptrSize = pointerSize();
-    intptr_t slide = getSlide();
-    const uint8_t* arrayBase = (uint8_t*)this + categoryListRuntimeOffset;
-    if ( ptrSize == 8 ) {
-        typedef uint64_t PtrTy;
-        struct objc_category_t {
-            PtrTy nameVMAddr;
-            PtrTy clsVMAddr;
-            PtrTy instanceMethodsVMAddr;
-            PtrTy classMethodsVMAddr;
-            PtrTy protocolsVMAddr;
-            PtrTy instancePropertiesVMAddr;
-        };
-        __block bool stop = false;
-        for ( uint64_t i = 0; i != categoryListCount; ++i ) {
-            uint64_t categoryVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(arrayBase + (i * sizeof(PtrTy))));
-            const objc_category_t* categoryPtr = (const objc_category_t*)(categoryVMAddr + slide);
-            ObjCCategory objCCategory;
-            objCCategory.nameVMAddr                 = vmAddrConverter.convertToVMAddr(categoryPtr->nameVMAddr);
-            objCCategory.clsVMAddr                  = vmAddrConverter.convertToVMAddr(categoryPtr->clsVMAddr);
-            objCCategory.instanceMethodsVMAddr      = vmAddrConverter.convertToVMAddr(categoryPtr->instanceMethodsVMAddr);
-            objCCategory.classMethodsVMAddr         = vmAddrConverter.convertToVMAddr(categoryPtr->classMethodsVMAddr);
-            objCCategory.protocolsVMAddr            = vmAddrConverter.convertToVMAddr(categoryPtr->protocolsVMAddr);
-            objCCategory.instancePropertiesVMAddr   = vmAddrConverter.convertToVMAddr(categoryPtr->instancePropertiesVMAddr);
-            callback(categoryVMAddr, objCCategory, stop);
-            if ( stop )
-                break;
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        struct objc_category_t {
-            PtrTy nameVMAddr;
-            PtrTy clsVMAddr;
-            PtrTy instanceMethodsVMAddr;
-            PtrTy classMethodsVMAddr;
-            PtrTy protocolsVMAddr;
-            PtrTy instancePropertiesVMAddr;
-        };
-        __block bool stop = false;
-        for ( uint64_t i = 0; i != categoryListCount; ++i ) {
-            uint64_t categoryVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(arrayBase + (i * sizeof(PtrTy))));
-            const objc_category_t* categoryPtr = (const objc_category_t*)(categoryVMAddr + slide);
-            ObjCCategory objCCategory;
-            objCCategory.nameVMAddr                 = vmAddrConverter.convertToVMAddr(categoryPtr->nameVMAddr);
-            objCCategory.clsVMAddr                  = vmAddrConverter.convertToVMAddr(categoryPtr->clsVMAddr);
-            objCCategory.instanceMethodsVMAddr      = vmAddrConverter.convertToVMAddr(categoryPtr->instanceMethodsVMAddr);
-            objCCategory.classMethodsVMAddr         = vmAddrConverter.convertToVMAddr(categoryPtr->classMethodsVMAddr);
-            objCCategory.protocolsVMAddr            = vmAddrConverter.convertToVMAddr(categoryPtr->protocolsVMAddr);
-            objCCategory.instancePropertiesVMAddr   = vmAddrConverter.convertToVMAddr(categoryPtr->instancePropertiesVMAddr);
-            callback(categoryVMAddr, objCCategory, stop);
-            if ( stop )
-                break;
-        }
-    }
-}
-
-void MachOAnalyzer::forEachObjCCategory(Diagnostics& diag, const VMAddrConverter& vmAddrConverter,
-                                        CategoryCallback& callback) const {
-    uint64_t categoryListRuntimeOffset;
-    uint64_t categoryListSize;
-    bool foundSection = findObjCDataSection("__objc_catlist", categoryListRuntimeOffset, categoryListSize);
-    if ( !foundSection )
-        return;
-
-    const uint64_t ptrSize = pointerSize();
-    if ( (categoryListSize % ptrSize) != 0 ) {
-        diag.error("Invalid objc category section size");
-        return;
-    }
-
-    forEachObjCCategory(categoryListRuntimeOffset, categoryListSize / ptrSize, vmAddrConverter, callback);
-}
-
-void MachOAnalyzer::forEachObjCProtocol(uint64_t protocolListRuntimeOffset, uint64_t protocolListCount,
-                                        const VMAddrConverter& vmAddrConverter,
-                                        ProtocolCallback& callback) const
-{
-    const uint64_t ptrSize = pointerSize();
-    intptr_t slide = getSlide();
-    const uint8_t* arrayBase = (uint8_t*)this + protocolListRuntimeOffset;
-    if ( ptrSize == 8 ) {
-        typedef uint64_t PtrTy;
-        struct protocol_t {
-            PtrTy    isaVMAddr;
-            PtrTy    nameVMAddr;
-            PtrTy    protocolsVMAddr;
-            PtrTy    instanceMethodsVMAddr;
-            PtrTy    classMethodsVMAddr;
-            PtrTy    optionalInstanceMethodsVMAddr;
-            PtrTy    optionalClassMethodsVMAddr;
-            PtrTy    instancePropertiesVMAddr;
-            uint32_t size;
-            uint32_t flags;
-            // Fields below this point are not always present on disk.
-            PtrTy    extendedMethodTypesVMAddr;
-            PtrTy    demangledNameVMAddr;
-            PtrTy    classPropertiesVMAddr;
-        };
-        __block bool stop = false;
-        for ( uint64_t i = 0; i != protocolListCount; ++i ) {
-            uint64_t protocolVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(arrayBase + (i * sizeof(PtrTy))));
-            const protocol_t* protocolPtr = (const protocol_t*)(protocolVMAddr + slide);
-            ObjCProtocol objCProtocol;
-            objCProtocol.isaVMAddr                          = vmAddrConverter.convertToVMAddr(protocolPtr->isaVMAddr);
-            objCProtocol.nameVMAddr                         = vmAddrConverter.convertToVMAddr(protocolPtr->nameVMAddr);
-            objCProtocol.protocolsVMAddr                    = vmAddrConverter.convertToVMAddr(protocolPtr->protocolsVMAddr);
-            objCProtocol.instanceMethodsVMAddr              = vmAddrConverter.convertToVMAddr(protocolPtr->instanceMethodsVMAddr);
-            objCProtocol.classMethodsVMAddr                 = vmAddrConverter.convertToVMAddr(protocolPtr->classMethodsVMAddr);
-            objCProtocol.optionalInstanceMethodsVMAddr      = vmAddrConverter.convertToVMAddr(protocolPtr->optionalInstanceMethodsVMAddr);
-            objCProtocol.optionalClassMethodsVMAddr         = vmAddrConverter.convertToVMAddr(protocolPtr->optionalClassMethodsVMAddr);
-            callback(protocolVMAddr, objCProtocol, stop);
-            if ( stop )
-                break;
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        struct protocol_t {
-            PtrTy    isaVMAddr;
-            PtrTy    nameVMAddr;
-            PtrTy    protocolsVMAddr;
-            PtrTy    instanceMethodsVMAddr;
-            PtrTy    classMethodsVMAddr;
-            PtrTy    optionalInstanceMethodsVMAddr;
-            PtrTy    optionalClassMethodsVMAddr;
-            PtrTy    instancePropertiesVMAddr;
-            uint32_t size;
-            uint32_t flags;
-            // Fields below this point are not always present on disk.
-            PtrTy    extendedMethodTypesVMAddr;
-            PtrTy    demangledNameVMAddr;
-            PtrTy    classPropertiesVMAddr;
-        };
-        __block bool stop = false;
-        for ( uint64_t i = 0; i != protocolListCount; ++i ) {
-            uint64_t protocolVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(arrayBase + (i * sizeof(PtrTy))));
-            const protocol_t* protocolPtr = (const protocol_t*)(protocolVMAddr + slide);
-            ObjCProtocol objCProtocol;
-            objCProtocol.isaVMAddr                          = vmAddrConverter.convertToVMAddr(protocolPtr->isaVMAddr);
-            objCProtocol.nameVMAddr                         = vmAddrConverter.convertToVMAddr(protocolPtr->nameVMAddr);
-            objCProtocol.protocolsVMAddr                    = vmAddrConverter.convertToVMAddr(protocolPtr->protocolsVMAddr);
-            objCProtocol.instanceMethodsVMAddr              = vmAddrConverter.convertToVMAddr(protocolPtr->instanceMethodsVMAddr);
-            objCProtocol.classMethodsVMAddr                 = vmAddrConverter.convertToVMAddr(protocolPtr->classMethodsVMAddr);
-            objCProtocol.optionalInstanceMethodsVMAddr      = vmAddrConverter.convertToVMAddr(protocolPtr->optionalInstanceMethodsVMAddr);
-            objCProtocol.optionalClassMethodsVMAddr         = vmAddrConverter.convertToVMAddr(protocolPtr->optionalClassMethodsVMAddr);
-            callback(protocolVMAddr, objCProtocol, stop);
-            if ( stop )
-                break;
-        }
-    }
-}
-
-void MachOAnalyzer::forEachObjCProtocol(Diagnostics& diag, const VMAddrConverter& vmAddrConverter,
-                                        ProtocolCallback& callback) const {
-    uint64_t protocolListRuntimeOffset;
-    uint64_t protocolListSize;
-    bool foundSection = findObjCDataSection("__objc_protolist", protocolListRuntimeOffset, protocolListSize);
-    if ( !foundSection )
-        return;
-
-    const uint64_t ptrSize = pointerSize();
-    if ( (protocolListSize % ptrSize) != 0 ) {
-        diag.error("Invalid objc protocol section size");
-        return;
-    }
-
-    forEachObjCProtocol(protocolListRuntimeOffset, protocolListSize / ptrSize, vmAddrConverter, callback);
-}
-
-void MachOAnalyzer::forEachObjCMethod(uint64_t methodListVMAddr, const VMAddrConverter& vmAddrConverter,
-                                      uint64_t sharedCacheRelativeSelectorBaseVMAddress,
-                                      void (^handler)(uint64_t methodVMAddr, const ObjCMethod& method, bool& stop)) const
-{
-    if ( methodListVMAddr == 0 )
-        return;
-
-    const uint64_t ptrSize = pointerSize();
-    intptr_t slide = getSlide();
-
-    if ( ptrSize == 8 ) {
-        typedef uint64_t PtrTy;
-        struct method_list_t {
-            uint32_t    entsize;
-            uint32_t    count;
-            PtrTy       methodArrayBase; // Note this is the start the array method_t[0]
-
-            uint32_t getEntsize() const {
-                return entsize & ObjCMethodList::methodListSizeMask;
-            }
-
-            bool usesDirectOffsetsToSelectors() const {
-                return (entsize & 0x40000000) != 0;
-            }
-
-            bool usesRelativeOffsets() const {
-                return (entsize & 0x80000000) != 0;
-            }
-        };
-
-        struct method_t {
-            PtrTy nameVMAddr;   // SEL
-            PtrTy typesVMAddr;  // const char *
-            PtrTy impVMAddr;    // IMP
-        };
-
-        struct relative_method_t {
-            int32_t nameOffset;   // SEL*
-            int32_t typesOffset;  // const char *
-            int32_t impOffset;    // IMP
-        };
-
-        const method_list_t* methodList = (const method_list_t*)(methodListVMAddr + slide);
-        if ( methodList == nullptr )
-            return;
-        bool relativeMethodListsAreOffsetsToSelectors = methodList->usesDirectOffsetsToSelectors();
-        uint64_t methodListArrayBaseVMAddr = methodListVMAddr + offsetof(method_list_t, methodArrayBase);
-        for (unsigned i = 0; i != methodList->count; ++i) {
-            uint64_t methodEntryOffset = i * methodList->getEntsize();
-            uint64_t methodVMAddr = methodListArrayBaseVMAddr + methodEntryOffset;
-            ObjCMethod method;
-            if ( methodList->usesRelativeOffsets() ) {
-                const relative_method_t* methodPtr = (const relative_method_t*)(methodVMAddr + slide);
-                if ( relativeMethodListsAreOffsetsToSelectors ) {
-                    if ( sharedCacheRelativeSelectorBaseVMAddress != 0 ) {
-                        // // New shared caches use an offset from a magic selector for relative method lists
-                        method.nameVMAddr = sharedCacheRelativeSelectorBaseVMAddress + methodPtr->nameOffset;
-                    } else {
-                        method.nameVMAddr  = methodVMAddr + offsetof(relative_method_t, nameOffset) + methodPtr->nameOffset;
-                    }
-                } else {
-                    PtrTy* nameLocation = (PtrTy*)((uint8_t*)&methodPtr->nameOffset + methodPtr->nameOffset);
-                    method.nameVMAddr   = vmAddrConverter.convertToVMAddr(*nameLocation);
-                }
-                method.typesVMAddr  = methodVMAddr + offsetof(relative_method_t, typesOffset) + methodPtr->typesOffset;
-                method.impVMAddr    = methodVMAddr + offsetof(relative_method_t, impOffset) + methodPtr->impOffset;
-                method.nameLocationVMAddr = methodVMAddr + offsetof(relative_method_t, nameOffset) + methodPtr->nameOffset;
-            } else {
-                const method_t* methodPtr = (const method_t*)(methodVMAddr + slide);
-                method.nameVMAddr   = vmAddrConverter.convertToVMAddr(methodPtr->nameVMAddr);
-                method.typesVMAddr  = vmAddrConverter.convertToVMAddr(methodPtr->typesVMAddr);
-                method.impVMAddr    = vmAddrConverter.convertToVMAddr(methodPtr->impVMAddr);
-                method.nameLocationVMAddr = methodVMAddr + offsetof(method_t, nameVMAddr);
-            }
-            bool stop = false;
-            handler(methodVMAddr, method, stop);
-            if ( stop )
-                break;
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        struct method_list_t {
-            uint32_t    entsize;
-            uint32_t    count;
-            PtrTy       methodArrayBase; // Note this is the start the array method_t[0]
-
-            uint32_t getEntsize() const {
-                return entsize & ObjCMethodList::methodListSizeMask;
-            }
-
-            bool usesDirectOffsetsToSelectors() const {
-                return (entsize & 0x40000000) != 0;
-            }
-
-            bool usesRelativeOffsets() const {
-                return (entsize & 0x80000000) != 0;
-            }
-        };
-
-        struct method_t {
-            PtrTy nameVMAddr;   // SEL
-            PtrTy typesVMAddr;  // const char *
-            PtrTy impVMAddr;    // IMP
-        };
-
-        struct relative_method_t {
-            int32_t nameOffset;   // SEL*
-            int32_t typesOffset;  // const char *
-            int32_t impOffset;    // IMP
-        };
-
-        const method_list_t* methodList = (const method_list_t*)(methodListVMAddr + slide);
-        if ( methodList == nullptr )
-            return;
-        bool relativeMethodListsAreOffsetsToSelectors = methodList->usesDirectOffsetsToSelectors();
-        uint64_t methodListArrayBaseVMAddr = methodListVMAddr + offsetof(method_list_t, methodArrayBase);
-        for (unsigned i = 0; i != methodList->count; ++i) {
-            uint64_t methodEntryOffset = i * methodList->getEntsize();
-            uint64_t methodVMAddr = methodListArrayBaseVMAddr + methodEntryOffset;
-            ObjCMethod method;
-            if ( methodList->usesRelativeOffsets() ) {
-                const relative_method_t* methodPtr = (const relative_method_t*)(methodVMAddr + slide);
-                if ( relativeMethodListsAreOffsetsToSelectors ) {
-                    if ( sharedCacheRelativeSelectorBaseVMAddress != 0 ) {
-                        // // New shared caches use an offset from a magic selector for relative method lists
-                        method.nameVMAddr = sharedCacheRelativeSelectorBaseVMAddress + methodPtr->nameOffset;
-                    } else {
-                        method.nameVMAddr  = methodVMAddr + offsetof(relative_method_t, nameOffset) + methodPtr->nameOffset;
-                    }
-                } else {
-                    PtrTy* nameLocation = (PtrTy*)((uint8_t*)&methodPtr->nameOffset + methodPtr->nameOffset);
-                    method.nameVMAddr   = vmAddrConverter.convertToVMAddr(*nameLocation);
-                }
-                method.typesVMAddr  = methodVMAddr + offsetof(relative_method_t, typesOffset) + methodPtr->typesOffset;
-                method.impVMAddr    = methodVMAddr + offsetof(relative_method_t, impOffset) + methodPtr->impOffset;
-                method.nameLocationVMAddr = methodVMAddr + offsetof(relative_method_t, nameOffset) + methodPtr->nameOffset;
-            } else {
-                const method_t* methodPtr = (const method_t*)(methodVMAddr + slide);
-                method.nameVMAddr   = vmAddrConverter.convertToVMAddr(methodPtr->nameVMAddr);
-                method.typesVMAddr  = vmAddrConverter.convertToVMAddr(methodPtr->typesVMAddr);
-                method.impVMAddr    = vmAddrConverter.convertToVMAddr(methodPtr->impVMAddr);
-                method.nameLocationVMAddr = methodVMAddr + offsetof(method_t, nameVMAddr);
-            }
-            bool stop = false;
-            handler(methodVMAddr, method, stop);
-            if ( stop )
-                break;
-        }
-    }
-}
-
-bool MachOAnalyzer::objcMethodListIsRelative(uint64_t methodListRuntimeOffset) const
-{
-    if ( methodListRuntimeOffset == 0 )
-        return false;
-
-    struct method_list_t {
-        uint32_t    entsize;
-        uint32_t    count;
-
-        bool usesRelativeOffsets() const {
-            return (entsize & 0x80000000) != 0;
-        }
-    };
-
-    const method_list_t* methodList = (const method_list_t*)((const uint8_t*)this + methodListRuntimeOffset);
-    return methodList->usesRelativeOffsets();
-}
-
-void MachOAnalyzer::forEachObjCProperty(uint64_t propertyListVMAddr, const VMAddrConverter& vmAddrConverter,
-                                        void (^handler)(uint64_t propertyVMAddr, const ObjCProperty& property)) const {
-    if ( propertyListVMAddr == 0 )
-        return;
-
-    const uint64_t ptrSize = pointerSize();
-    intptr_t slide = getSlide();
-
-    if ( ptrSize == 8 ) {
-        typedef uint64_t PtrTy;
-        struct property_list_t {
-            uint32_t    entsize;
-            uint32_t    count;
-            PtrTy       propertyArrayBase; // Note this is the start the array property_t[0]
-
-            uint32_t getEntsize() const {
-                return (entsize) & ~(uint32_t)3;
-            }
-        };
-
-        struct property_t {
-            PtrTy nameVMAddr;   // SEL
-            PtrTy attributesVMAddr;  // const char *
-        };
-
-        const property_list_t* propertyList = (const property_list_t*)(propertyListVMAddr + slide);
-        uint64_t propertyListArrayBaseVMAddr = propertyListVMAddr + offsetof(property_list_t, propertyArrayBase);
-        for (unsigned i = 0; i != propertyList->count; ++i) {
-            uint64_t propertyEntryOffset = i * propertyList->getEntsize();
-            uint64_t propertyVMAddr = propertyListArrayBaseVMAddr + propertyEntryOffset;
-            const property_t* propertyPtr = (const property_t*)(propertyVMAddr + slide);
-            ObjCProperty property;
-            property.nameVMAddr         = vmAddrConverter.convertToVMAddr(propertyPtr->nameVMAddr);
-            property.attributesVMAddr   = vmAddrConverter.convertToVMAddr(propertyPtr->attributesVMAddr);
-            handler(propertyVMAddr, property);
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        struct property_list_t {
-            uint32_t    entsize;
-            uint32_t    count;
-            PtrTy       propertyArrayBase; // Note this is the start the array property_t[0]
-
-            uint32_t getEntsize() const {
-                return (entsize) & ~(uint32_t)3;
-            }
-        };
-
-        struct property_t {
-            PtrTy nameVMAddr;   // SEL
-            PtrTy attributesVMAddr;  // const char *
-        };
-
-        const property_list_t* propertyList = (const property_list_t*)(propertyListVMAddr + slide);
-        uint64_t propertyListArrayBaseVMAddr = propertyListVMAddr + offsetof(property_list_t, propertyArrayBase);
-        for (unsigned i = 0; i != propertyList->count; ++i) {
-            uint64_t propertyEntryOffset = i * propertyList->getEntsize();
-            uint64_t propertyVMAddr = propertyListArrayBaseVMAddr + propertyEntryOffset;
-            const property_t* propertyPtr = (const property_t*)(propertyVMAddr + slide);
-            ObjCProperty property;
-            property.nameVMAddr         = vmAddrConverter.convertToVMAddr(propertyPtr->nameVMAddr);
-            property.attributesVMAddr   = vmAddrConverter.convertToVMAddr(propertyPtr->attributesVMAddr);
-            handler(propertyVMAddr, property);
-        }
-    }
-}
-
-void MachOAnalyzer::forEachObjCProtocol(uint64_t protocolListVMAddr, const VMAddrConverter& vmAddrConverter,
-                                        void (^handler)(uint64_t protocolRefVMAddr, const ObjCProtocol&)) const
-{
-    if ( protocolListVMAddr == 0 )
-        return;
-
-    auto ptrSize = pointerSize();
-    intptr_t slide = getSlide();
-
-    if ( ptrSize == 8 ) {
-        typedef uint64_t PtrTy;
-        struct protocol_ref_t {
-            PtrTy       refVMAddr;
-        };
-        struct protocol_list_t {
-            PtrTy           count;
-            protocol_ref_t  array[];
-        };
-        struct protocol_t {
-            PtrTy    isaVMAddr;
-            PtrTy    nameVMAddr;
-            PtrTy    protocolsVMAddr;
-            PtrTy    instanceMethodsVMAddr;
-            PtrTy    classMethodsVMAddr;
-            PtrTy    optionalInstanceMethodsVMAddr;
-            PtrTy    optionalClassMethodsVMAddr;
-            PtrTy    instancePropertiesVMAddr;
-            uint32_t size;
-            uint32_t flags;
-            // Fields below this point are not always present on disk.
-            PtrTy    extendedMethodTypesVMAddr;
-            PtrTy    demangledNameVMAddr;
-            PtrTy    classPropertiesVMAddr;
-        };
-
-        const protocol_list_t* protoList = (const protocol_list_t*)(protocolListVMAddr + slide);
-        for (PtrTy i = 0; i != protoList->count; ++i) {
-            uint64_t protocolVMAddr = vmAddrConverter.convertToVMAddr(protoList->array[i].refVMAddr);
-
-            const protocol_t* protocolPtr = (const protocol_t*)(protocolVMAddr + slide);
-            ObjCProtocol objCProtocol;
-            objCProtocol.isaVMAddr                          = vmAddrConverter.convertToVMAddr(protocolPtr->isaVMAddr);
-            objCProtocol.nameVMAddr                         = vmAddrConverter.convertToVMAddr(protocolPtr->nameVMAddr);
-            objCProtocol.protocolsVMAddr                    = vmAddrConverter.convertToVMAddr(protocolPtr->protocolsVMAddr);
-            objCProtocol.instanceMethodsVMAddr              = vmAddrConverter.convertToVMAddr(protocolPtr->instanceMethodsVMAddr);
-            objCProtocol.classMethodsVMAddr                 = vmAddrConverter.convertToVMAddr(protocolPtr->classMethodsVMAddr);
-            objCProtocol.optionalInstanceMethodsVMAddr      = vmAddrConverter.convertToVMAddr(protocolPtr->optionalInstanceMethodsVMAddr);
-            objCProtocol.optionalClassMethodsVMAddr         = vmAddrConverter.convertToVMAddr(protocolPtr->optionalClassMethodsVMAddr);
-
-            handler(protocolVMAddr, objCProtocol);
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        struct protocol_ref_t {
-            PtrTy       refVMAddr;
-        };
-        struct protocol_list_t {
-            PtrTy           count;
-            protocol_ref_t  array[];
-        };
-        struct protocol_t {
-            PtrTy    isaVMAddr;
-            PtrTy    nameVMAddr;
-            PtrTy    protocolsVMAddr;
-            PtrTy    instanceMethodsVMAddr;
-            PtrTy    classMethodsVMAddr;
-            PtrTy    optionalInstanceMethodsVMAddr;
-            PtrTy    optionalClassMethodsVMAddr;
-            PtrTy    instancePropertiesVMAddr;
-            uint32_t size;
-            uint32_t flags;
-            // Fields below this point are not always present on disk.
-            PtrTy    extendedMethodTypesVMAddr;
-            PtrTy    demangledNameVMAddr;
-            PtrTy    classPropertiesVMAddr;
-        };
-
-        const protocol_list_t* protoList = (const protocol_list_t*)(protocolListVMAddr + slide);
-        for (PtrTy i = 0; i != protoList->count; ++i) {
-            uint64_t protocolVMAddr = vmAddrConverter.convertToVMAddr(protoList->array[i].refVMAddr);
-
-            const protocol_t* protocolPtr = (const protocol_t*)(protocolVMAddr + slide);
-            ObjCProtocol objCProtocol;
-            objCProtocol.isaVMAddr                          = vmAddrConverter.convertToVMAddr(protocolPtr->isaVMAddr);
-            objCProtocol.nameVMAddr                         = vmAddrConverter.convertToVMAddr(protocolPtr->nameVMAddr);
-            objCProtocol.protocolsVMAddr                    = vmAddrConverter.convertToVMAddr(protocolPtr->protocolsVMAddr);
-            objCProtocol.instanceMethodsVMAddr              = vmAddrConverter.convertToVMAddr(protocolPtr->instanceMethodsVMAddr);
-            objCProtocol.classMethodsVMAddr                 = vmAddrConverter.convertToVMAddr(protocolPtr->classMethodsVMAddr);
-            objCProtocol.optionalInstanceMethodsVMAddr      = vmAddrConverter.convertToVMAddr(protocolPtr->optionalInstanceMethodsVMAddr);
-            objCProtocol.optionalClassMethodsVMAddr         = vmAddrConverter.convertToVMAddr(protocolPtr->optionalClassMethodsVMAddr);
-
-            handler(protocolVMAddr, objCProtocol);
-        }
-    }
-}
-
-
-void MachOAnalyzer::forEachObjCSelectorReference(uint64_t selRefsRuntimeOffset, uint64_t selRefsCount, const VMAddrConverter& vmAddrConverter,
-                                                 void (^handler)(uint64_t selRefVMAddr, uint64_t selRefTargetVMAddr, bool& stop)) const
-{
-    uint64_t baseAddress = preferredLoadAddress();
-    const uint64_t ptrSize = pointerSize();
-    const uint8_t* selRefs = (uint8_t*)this + selRefsRuntimeOffset;
-    if ( ptrSize == 8 ) {
-        typedef uint64_t PtrTy;
-        bool stop = false;
-        for ( uint64_t i = 0; i != selRefsCount; ++i ) {
-            uint64_t selRefVMAddr = baseAddress + selRefsRuntimeOffset + (i * sizeof(PtrTy));
-            uint64_t selRefTargetVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(selRefs + (i * sizeof(PtrTy))));
-            handler(selRefVMAddr, selRefTargetVMAddr, stop);
-            if ( stop )
-                break;
-        }
-    } else {
-        typedef uint32_t PtrTy;
-        bool stop = false;
-        for ( uint64_t i = 0; i != selRefsCount; ++i ) {
-            uint64_t selRefVMAddr = baseAddress + selRefsRuntimeOffset + (i * sizeof(PtrTy));
-            uint64_t selRefTargetVMAddr = vmAddrConverter.convertToVMAddr(*(PtrTy*)(selRefs + (i * sizeof(PtrTy))));
-            handler(selRefVMAddr, selRefTargetVMAddr, stop);
-            if ( stop )
-                break;
-        }
-    }
-}
-
-bool MachOAnalyzer::findObjCDataSection(const char *sectionName, uint64_t& sectionRuntimeOffset, uint64_t& sectionSize) const
-{
-    uint64_t baseAddress = preferredLoadAddress();
-
-    __block bool foundSection = false;
-    forEachSection(^(const SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        if ( (strcmp(sectInfo.segInfo.segName, "__DATA") != 0) &&
-             (strcmp(sectInfo.segInfo.segName, "__DATA_CONST") != 0) &&
-             (strcmp(sectInfo.segInfo.segName, "__DATA_DIRTY") != 0) )
-            return;
-        if ( strcmp(sectInfo.sectName, sectionName) != 0 )
-            return;
-        foundSection         = true;
-        sectionRuntimeOffset = sectInfo.sectAddr - baseAddress;
-        sectionSize          = sectInfo.sectSize;
-        stop                 = true;
-    });
-    return foundSection;
-}
-
-
-void MachOAnalyzer::forEachObjCSelectorReference(Diagnostics& diag, const VMAddrConverter& vmAddrConverter,
-                                                 void (^handler)(uint64_t selRefVMAddr, uint64_t selRefTargetVMAddr, bool& stop)) const
-{
-    uint64_t selRefsRuntimeOffset;
-    uint64_t selRefsSize;
-    bool foundSection = findObjCDataSection("__objc_selrefs", selRefsRuntimeOffset, selRefsSize);
-    if ( !foundSection )
-        return;
-
-    const uint64_t ptrSize = pointerSize();
-    if ( (selRefsSize % ptrSize) != 0 ) {
-        diag.error("Invalid sel ref section size");
-        return;
-    }
-
-    forEachObjCSelectorReference(selRefsRuntimeOffset, selRefsSize / ptrSize, vmAddrConverter, handler);
-}
-
-void MachOAnalyzer::forEachObjCMethodName(void (^handler)(const char* methodName)) const {
-    intptr_t slide = getSlide();
-    forEachSection(^(const dyld3::MachOAnalyzer::SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        if ( strcmp(sectInfo.segInfo.segName, "__TEXT") != 0 )
-            return;
-        if ( strcmp(sectInfo.sectName, "__objc_methname") != 0 )
-            return;
-        if ( sectInfo.segInfo.isProtected || ( (sectInfo.sectFlags & SECTION_TYPE) != S_CSTRING_LITERALS ) ) {
-            stop = true;
-            return;
-        }
-        if ( malformedSectionRange ) {
-            stop = true;
-            return;
-        }
-
-        const char* content       = (const char*)(sectInfo.sectAddr + slide);
-        uint64_t    sectionSize   = sectInfo.sectSize;
-
-        const char* s   = (const char*)content;
-        const char* end = s + sectionSize;
-        while ( s < end ) {
-            handler(s);
-            s += strlen(s) + 1;
-        }
-    });
-}
-
-#if BUILDING_DYLD || BUILDING_UNIT_TESTS
-void MachOAnalyzer::forEachObjCDuplicateClassToIgnore(void (^handler)(const char* className)) const {
-    const uint32_t pointerSize = this->pointerSize();
-
-    uint64_t sectionSize = 0;
-    const void* section = findSectionContent("__DATA", "__objc_dupclass", sectionSize);
-
-    if ( !section )
-        return;
-
-    // Ignore sections which are the wrong size
-    if ( (sectionSize % pointerSize) != 0 )
-        return;
-
-    // Copied from objc-abi.h
-    typedef struct _objc_duplicate_class {
-        uint32_t version;
-        uint32_t flags;
-        const char name[64];
-    } objc_duplicate_class;
-
-    for (uint64_t offset = 0; offset != sectionSize; offset += pointerSize) {
-        uintptr_t pointerValue = *(uintptr_t*)((uint64_t)section + offset);
-        const objc_duplicate_class* duplicateClass = (const objc_duplicate_class*)pointerValue;
-        handler(duplicateClass->name);
-    }
-}
-#endif
-
-
-bool MachOAnalyzer::hasObjCMessageReferences() const {
-
-    __block bool foundSection = false;
-    forEachSection(^(const SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        if ( strncmp(sectInfo.segInfo.segName, "__DATA", 6) != 0 )
-            return;
-        if ( strcmp(sectInfo.sectName, "__objc_msgrefs") != 0 )
-            return;
-        foundSection = true;
-        stop = true;
-    });
-    return foundSection;
-}
-
-const MachOAnalyzer::ObjCImageInfo* MachOAnalyzer::objcImageInfo() const {
-    int64_t slide = getSlide();
-
-    __block bool foundInvalidObjCImageInfo = false;
-    __block const ObjCImageInfo* imageInfo = nullptr;
-    forEachSection(^(const dyld3::MachOAnalyzer::SectionInfo& sectionInfo, bool malformedSectionRange, bool& stop) {
-        if ( strncmp(sectionInfo.segInfo.segName, "__DATA", 6) != 0 )
-            return;
-        if (strcmp(sectionInfo.sectName, "__objc_imageinfo") != 0)
-            return;
-        if ( malformedSectionRange ) {
-            stop = true;
-            return;
-        }
-        if ( sectionInfo.sectSize != 8 ) {
-            stop = true;
-            return;
-        }
-        imageInfo = (const ObjCImageInfo*)(sectionInfo.sectAddr + slide);
-        if ( (imageInfo->flags & ObjCImageInfo::dyldPreoptimized) != 0 ) {
-            foundInvalidObjCImageInfo = true;
-            stop = true;
-            return;
-        }
-        stop = true;
-    });
-    if ( foundInvalidObjCImageInfo )
-        return nullptr;
-    return imageInfo;
-}
-
-// A 32-bit relative pointer to a value.  The offset value is either:
-// - a direct 32-bit offset to the value, if the low bit is 0, or
-// - an offset to a pointer sized slot, if the low bit is 1
-class SwiftRelativePointer
-{
-public:
-    uint64_t getTargetRuntimeOffset(const MachOAnalyzer* ma,
-                                    const MachOAnalyzer::VMAddrConverter& vmAddrConverter,
-                                    uint32_t ptrSize) const {
-        if ( (relativeOffset & 0x1) == 0 ) {
-            // Relative offset directly to the target value
-            uint64_t relativePointerRuntimeOffset = (uint64_t)this - (uint64_t)ma;
-            return relativePointerRuntimeOffset + (int64_t)relativeOffset;
-        } else {
-            // Relative offset to a pointer.  The pointer contains the target value
-            int32_t offset = relativeOffset & ~0x1ULL;
-            uint64_t relativePointerRuntimeOffset = (uint64_t)this - (uint64_t)ma;
-            relativePointerRuntimeOffset += (int64_t)offset;
-            uint64_t pointerValue = 0;
-            if ( ptrSize == 8 ) {
-                pointerValue = *(uint64_t*)((uint8_t*)ma + relativePointerRuntimeOffset);
-            } else {
-                pointerValue = *(uint32_t*)((uint8_t*)ma + relativePointerRuntimeOffset);
-            }
-            // Convert from the encoded pointer to a runtime offset
-            return vmAddrConverter.convertToVMAddr(pointerValue) - vmAddrConverter.preferredLoadAddress;
-        }
-    }
-
-private:
-    int32_t relativeOffset = 0;
-};
-
-class SwiftProtocolConformanceFlags
-{
-public:
-    // Taken from MetadataValues.h
-    enum class TypeMetadataKind : uint32_t {
-        mask    = 0x7 << 3, // 8 type reference kinds
-        shift   = 3,
-    };
-
-    // Taken from MetadataValues.h
-    enum class TypeReferenceKind : uint32_t {
-        // The conformance is for a nominal type referenced directly;
-        // getTypeDescriptor() points to the type context descriptor.
-        directTypeDescriptor = 0x00,
-
-        // The conformance is for a nominal type referenced indirectly;
-        // getTypeDescriptor() points to the type context descriptor.
-        indirectTypeDescriptor = 0x01,
-
-        // The conformance is for an Objective-C class that should be looked up
-        // by class name.
-        directObjCClassName = 0x02,
-
-        // The conformance is for an Objective-C class that has no nominal type
-        // descriptor.
-        // getIndirectObjCClass() points to a variable that contains the pointer to
-        // the class object, which then requires a runtime call to get metadata.
-        //
-        // On platforms without Objective-C interoperability, this case is
-        // unused.
-        indirectObjCClass = 0x03,
-    };
-
-    TypeReferenceKind typeReferenceKind() const {
-        return (TypeReferenceKind)((flags & (uint32_t)TypeMetadataKind::mask) >> (uint32_t)TypeMetadataKind::shift);
-    }
-private:
-    uint32_t flags = 0;
-};
-
-// This represents the type descriptors pointed to by protocol conformances
-class TypeContextDescriptor
-{
-public:
-
-    const SwiftRelativePointer& name() const {
-        return _name;
-    }
-
-    bool isForeignMetadata() const {
-        // The botton 2 bits have the flags
-        return (getKindSpecificFlags() & 0x3) == ForeignMetadataInitialization;
-    }
-
-    bool hasImportInfo() const {
-        // Bit 2 tells us if we have import info, ie, a name containing NULLs
-        return (getKindSpecificFlags() & (1 << 2)) != 0;
-    }
-
-private:
-
-    enum TypeContextDescriptorFlags : uint16_t {
-        ForeignMetadataInitialization = 0x2
-    };
-
-    // The most significant two bytes of the flags word, which can have
-    // kind-specific meaning.
-    uint16_t getKindSpecificFlags() const {
-        return (_flags >> 16u) & 0xFFFFu;
-    }
-
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wunused-private-field"
-    uint32_t                _flags;
-    int32_t                 _parent;
-    SwiftRelativePointer    _name;
-    int32_t                 _accessFunction;
-    int32_t                 _fields;
-#pragma clang diagnostic pop
-};
-
-// A 32-bit relative pointer to a value
-// The type the pointer depends on the SwiftProtocolConformanceFlags::TypeReferenceKind
-class SwiftTypeRefPointer
-{
-public:
-    // A type descriptor is one of the types described by TypeReferenceKind.
-    // This works out which of the give types we are, and populates the given out parameters to match
-    void getTypeDescriptor(SwiftProtocolConformanceFlags::TypeReferenceKind refKind,
-                           const MachOAnalyzer* ma,
-                           const MachOAnalyzer::VMAddrConverter& vmAddrConverter,
-                           uint32_t ptrSize,
-                           bool canAnalyzeIndirectPointers,
-                           uint64_t& descriptorRuntimeOffset,
-                           uint64_t& objcClassNameRuntimeOffset,
-                           uint64_t& objcClassRuntimeOffset,
-                           uint64_t& foreignMetadataNameRuntimeOffset,
-                           bool& foreignMetadataNameHasImportInfo) const {
-        descriptorRuntimeOffset = 0;
-        objcClassNameRuntimeOffset = 0;
-        objcClassRuntimeOffset = 0;
-        switch ( refKind ) {
-            case SwiftProtocolConformanceFlags::TypeReferenceKind::directTypeDescriptor: {
-                // Relative offset directly to the target value
-                uint64_t relativePointerRuntimeOffset = (uint64_t)this - (uint64_t)ma;
-                descriptorRuntimeOffset = relativePointerRuntimeOffset + (int64_t)relativeOffset;
-                break;
-            }
-            case SwiftProtocolConformanceFlags::TypeReferenceKind::indirectTypeDescriptor: {
-                // Relative offset to a pointer.  The pointer contains the target value
-                uint64_t relativePointerRuntimeOffset = (uint64_t)this - (uint64_t)ma;
-                relativePointerRuntimeOffset += (int64_t)relativeOffset;
-                uint64_t pointerValue = 0;
-                if ( ptrSize == 8 ) {
-                    pointerValue = *(uint64_t*)((uint8_t*)ma + relativePointerRuntimeOffset);
-                } else {
-                    pointerValue = *(uint32_t*)((uint8_t*)ma + relativePointerRuntimeOffset);
-                }
-                // This might be a pointer to a missing weak import.  If that is the case, just skip it
-                if ( pointerValue == 0 )
-                    break;
-                // Convert from the encoded pointer to a runtime offset
-                descriptorRuntimeOffset = vmAddrConverter.convertToVMAddr(pointerValue) - vmAddrConverter.preferredLoadAddress;
-                break;
-            }
-            case SwiftProtocolConformanceFlags::TypeReferenceKind::directObjCClassName: {
-                // Relative offset directly to the class name string
-                uint64_t relativePointerRuntimeOffset = (uint64_t)this - (uint64_t)ma;
-                objcClassNameRuntimeOffset = relativePointerRuntimeOffset + (int64_t)relativeOffset;
-                break;
-            }
-            case SwiftProtocolConformanceFlags::TypeReferenceKind::indirectObjCClass: {
-                // Relative offset to a pointer.  The pointer contains the target class value
-                uint64_t relativePointerRuntimeOffset = (uint64_t)this - (uint64_t)ma;
-                relativePointerRuntimeOffset += (int64_t)relativeOffset;
-                uint64_t pointerValue = 0;
-                if ( ptrSize == 8 ) {
-                    pointerValue = *(uint64_t*)((uint8_t*)ma + relativePointerRuntimeOffset);
-                } else {
-                    pointerValue = *(uint32_t*)((uint8_t*)ma + relativePointerRuntimeOffset);
-                }
-                // This might be a pointer to a missing weak import.  If that is the case, just skip it
-                if ( pointerValue == 0 )
-                    break;
-                // Convert from the encoded pointer to a runtime offset
-                objcClassRuntimeOffset = vmAddrConverter.convertToVMAddr(pointerValue) - vmAddrConverter.preferredLoadAddress;
-                break;
-            }
-        }
-
-        // Type descriptors might be foreign.  This means that the runtime needs to use their name to identify them
-        if ( descriptorRuntimeOffset != 0 ) {
-            // Once we've bound all dylibs in the shared cache, we can visit indirect type descriptors, which are binds.
-            // Prior to that, just add a placeholder value for foriegn metadata to suggest that it might be set.  We don't know
-            // for sure, but need to make pace for it
-            bool canAnalyze = canAnalyzeIndirectPointers;
-            if ( refKind == SwiftProtocolConformanceFlags::TypeReferenceKind::directTypeDescriptor )
-                canAnalyze = true;
-
-            if ( canAnalyze ) {
-                const TypeContextDescriptor* typeDesc = (const TypeContextDescriptor*)((uint8_t*)ma + descriptorRuntimeOffset);
-                if ( typeDesc->isForeignMetadata() ) {
-                    foreignMetadataNameRuntimeOffset = typeDesc->name().getTargetRuntimeOffset(ma, vmAddrConverter, ptrSize);
-                    foreignMetadataNameHasImportInfo = typeDesc->hasImportInfo();
-                    // printf("Name: %s\n", (const char*)ma + foreignMetadatNameRuntimeOffset);
-                }
-            } else {
-                // HACK: Add a placeholder
-                foreignMetadataNameRuntimeOffset = ~0ULL;
-            }
-        }
-    }
-
-private:
-    int32_t relativeOffset = 0;
-};
-
-void MachOAnalyzer::forEachSwiftProtocolConformance(uint64_t protocolConformanceListRuntimeOffset,
-                                                    uint64_t numProtocolConformances,
-                                                    const VMAddrConverter& vmAddrConverter,
-                                                    bool canAnalyzeIndirectPointers,
-                                                    void (^handler)(uint64_t protocolConformanceRuntimeOffset,
-                                                                    const SwiftProtocolConformance& protocolConformance,
-                                                                    bool& stopProtocolConformance)) const
-{
-    // Protocol conformances are a 32-bit offset from the list entry
-    const int32_t* protocolConformanceList = (const int32_t*)((uint8_t*)this + protocolConformanceListRuntimeOffset);
-
-    struct Conformance
-    {
-        const SwiftRelativePointer          protocolRelativePointer;
-        const SwiftTypeRefPointer           typeRef;
-        int32_t                             witnessTable;
-        const SwiftProtocolConformanceFlags flags;
-    };
-
-    const uint32_t ptrSize = MachOFile::pointerSize();
-    for (uint64_t i = 0; i != numProtocolConformances; ++i) {
-        int32_t relativeOffset = protocolConformanceList[i];
-        uint64_t listEntryRuntimeOffset = protocolConformanceListRuntimeOffset + (i * 4);
-        uint64_t protocolConformanceRuntimeOffset = listEntryRuntimeOffset + (int64_t)relativeOffset;
-
-        const Conformance* conformance = (const Conformance*)((uint8_t*)this + protocolConformanceRuntimeOffset);
-
-        SwiftProtocolConformance swiftProtocolConformance;
-        swiftProtocolConformance.protocolRuntimeOffset = conformance->protocolRelativePointer.getTargetRuntimeOffset(this, vmAddrConverter, ptrSize);
-        conformance->typeRef.getTypeDescriptor(conformance->flags.typeReferenceKind(), this, vmAddrConverter, ptrSize,
-                                               canAnalyzeIndirectPointers,
-                                               swiftProtocolConformance.typeConformanceRuntimeOffset,
-                                               swiftProtocolConformance.typeConformanceObjCClassNameRuntimeOffset,
-                                               swiftProtocolConformance.typeConformanceObjCClassRuntimeOffset,
-                                               swiftProtocolConformance.foreignMetadataNameRuntimeOffset,
-                                               swiftProtocolConformance.foreignMetadataNameHasImportInfo);
-
-        bool stopProtocolConformance = false;
-        handler(protocolConformanceRuntimeOffset, swiftProtocolConformance, stopProtocolConformance);
-        if ( stopProtocolConformance )
-            break;
-    }
-}
-
-void MachOAnalyzer::forEachSwiftProtocolConformance(Diagnostics& diag, const VMAddrConverter& vmAddrConverter,
-                                                    bool canAnalyzeIndirectPointers,
-                                                    void (^handler)(uint64_t protocolConformanceRuntimeOffset,
-                                                                    const SwiftProtocolConformance& protocolConformance,
-                                                                    bool& stopProtocolConformance)) const
-{
-    uint64_t protocolListRuntimeOffset;
-    uint64_t protocolListSize;
-    const void* sectionContent = findSectionContent("__TEXT", "__swift5_proto", protocolListSize);
-    if ( sectionContent == nullptr )
-        return;
-    protocolListRuntimeOffset = (uint64_t)sectionContent - (uint64_t)this;
-
-    // Each entry is a 32-bit offset to a protocol conformance
-    const uint32_t entrySize = 4;
-    if ( (protocolListSize % entrySize) != 0 ) {
-        diag.error("Invalid Swift protocol conformance section size");
-        return;
-    }
-
-    forEachSwiftProtocolConformance(protocolListRuntimeOffset, protocolListSize / entrySize, vmAddrConverter, canAnalyzeIndirectPointers, handler);
-}
-
-uint32_t MachOAnalyzer::loadCommandsFreeSpace() const
-{
-    __block uint32_t firstSectionFileOffset = 0;
-    __block uint32_t firstSegmentFileOffset = 0;
-    forEachSection(^(const SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        firstSectionFileOffset = sectInfo.sectFileOffset;
-        firstSegmentFileOffset = (uint32_t)sectInfo.segInfo.fileOffset;
-        stop = true;
-    });
-
-    uint32_t headerSize = (this->magic == MH_MAGIC_64) ? sizeof(mach_header_64) : sizeof(mach_header);
-    uint32_t existSpaceUsed = this->sizeofcmds + headerSize;
-    return firstSectionFileOffset - firstSegmentFileOffset - existSpaceUsed;
-}
-
-void MachOAnalyzer::forEachWeakDef(Diagnostics& diag,
-                                   void (^handler)(const char* symbolName, uint64_t imageOffset, bool isFromExportTrie)) const {
-    uint64_t baseAddress = preferredLoadAddress();
-    forEachGlobalSymbol(diag, ^(const char *symbolName, uint64_t n_value, uint8_t n_type, uint8_t n_sect, uint16_t n_desc, bool &stop) {
-        if ( (n_desc & N_WEAK_DEF) != 0 ) {
-            handler(symbolName, n_value - baseAddress, false);
-        }
-    });
-    forEachExportedSymbol(diag, ^(const char *symbolName, uint64_t imageOffset, uint64_t flags, uint64_t other, const char *importName, bool &stop) {
-        if ( (flags & EXPORT_SYMBOL_FLAGS_WEAK_DEFINITION ) == 0 )
-            return;
-        // Skip resolvers and re-exports
-        if ( (flags & EXPORT_SYMBOL_FLAGS_REEXPORT ) != 0 )
-            return;
-        if ( (flags & EXPORT_SYMBOL_FLAGS_STUB_AND_RESOLVER ) != 0 )
-            return;
-        handler(symbolName, imageOffset, true);
-    });
-}
-
-uint32_t MachOAnalyzer::getFixupsLoadCommandFileOffset() const
-{
-    Diagnostics diag;
-    LinkEditInfo linkeditInfo;
-    getLinkEditLoadCommands(diag, linkeditInfo);
-    if ( diag.hasError() )
-        return 0;
-    if ( linkeditInfo.dyldInfo )
-        return (uint32_t)( (uint8_t*)linkeditInfo.dyldInfo - (uint8_t*)this );
-    else if ( linkeditInfo.chainedFixups )
-        return (uint32_t)( (uint8_t*)linkeditInfo.chainedFixups - (uint8_t*)this );
-    return 0;
-}
-
-
-bool MachOAnalyzer::inCodeSection(uint32_t runtimeOffset) const
-{
-    // only needed for arm64e code to know to sign pointers
-    if ( (this->cputype != CPU_TYPE_ARM64) || (this->maskedCpuSubtype() != CPU_SUBTYPE_ARM64E) )
-        return false;
-
-    __block bool result = false;
-    uint64_t baseAddress = this->preferredLoadAddress();
-    this->forEachSection(^(const SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        if ( ((sectInfo.sectAddr-baseAddress) <= runtimeOffset) && (runtimeOffset < (sectInfo.sectAddr+sectInfo.sectSize-baseAddress)) ) {
-            result = ( (sectInfo.sectFlags & S_ATTR_PURE_INSTRUCTIONS) || (sectInfo.sectFlags & S_ATTR_SOME_INSTRUCTIONS) );
-            stop = true;
-        }
-    });
-    return result;
-}
-
-// These are mangled symbols for all the variants of operator new and delete
-// which a main executable can define (non-weak) and override the
-// weak-def implementation in the OS.
-static const char* const sTreatAsWeak[] = {
-    "__Znwm", "__ZnwmRKSt9nothrow_t",
-    "__Znam", "__ZnamRKSt9nothrow_t",
-    "__ZdlPv", "__ZdlPvRKSt9nothrow_t", "__ZdlPvm",
-    "__ZdaPv", "__ZdaPvRKSt9nothrow_t", "__ZdaPvm",
-    "__ZnwmSt11align_val_t", "__ZnwmSt11align_val_tRKSt9nothrow_t",
-    "__ZnamSt11align_val_t", "__ZnamSt11align_val_tRKSt9nothrow_t",
-    "__ZdlPvSt11align_val_t", "__ZdlPvSt11align_val_tRKSt9nothrow_t", "__ZdlPvmSt11align_val_t",
-    "__ZdaPvSt11align_val_t", "__ZdaPvSt11align_val_tRKSt9nothrow_t", "__ZdaPvmSt11align_val_t"
-};
-
-void MachOAnalyzer::forEachTreatAsWeakDef(void (^handler)(const char* symbolName))
-{
-    for (const char*  sym : sTreatAsWeak)
-        handler(sym);
-}
-
-MachOAnalyzer::TLV_InitialContent MachOAnalyzer::forEachThreadLocalVariable(Diagnostics& diag, void (^handler)(TLV_Thunk& slot)) const
-{
-    __block TLV_InitialContent result = { 0, 0 };
-
-    // most images don't use thread locals, so quickly skip those
-    if ( !this->hasThreadLocalVariables() )
-        return result;
-
-    uintptr_t  baseAddress = (uintptr_t)this->preferredLoadAddress();
-    intptr_t   slide       = (uintptr_t)this - baseAddress;
-    this->forEachSection(^(const SectionInfo& sectInfo, bool malformedSectionRange, bool& stop) {
-        switch (sectInfo.sectFlags & SECTION_TYPE) {
-            case S_THREAD_LOCAL_VARIABLES:
-                {
-                    unsigned count = (unsigned)(sectInfo.sectSize / sizeof(TLV_Thunk));
-                    const uint8_t* content = (uint8_t*)(sectInfo.sectAddr + slide);
-                    TLV_Thunk* slotsStart = (TLV_Thunk*)content;
-                    TLV_Thunk* slotsEnd   =  &slotsStart[count];
-                    for (TLV_Thunk* p=slotsStart; p < slotsEnd; ++p) {
-                        handler(*p);
-                    }
-                }
-                break;
-            case S_THREAD_LOCAL_ZEROFILL:
-            case S_THREAD_LOCAL_REGULAR:
-                if ( result.runtimeOffset == 0 ) {
-                    // first of N contiguous TLV template sections, record as if this was only section
-                    result.runtimeOffset = sectInfo.sectAddr - baseAddress;
-                    result.size          = sectInfo.sectSize;
-                }
-                else {
-                    // non-first of N contiguous TLV template sections, accumlate values
-                    result.size = sectInfo.sectAddr + sectInfo.sectSize - baseAddress - result.runtimeOffset;
-                }
-                break;
-        }
-    });
-
-    return result;
-}
-
-
-
-void MachOAnalyzer::forEachBindTarget(Diagnostics& diag, bool allowLazyBinds,
-                                      void (^handler)(const BindTargetInfo& info, bool& stop),
-                                      void (^overrideHandler)(const BindTargetInfo& info, bool& stop)) const
-{
-    if ( this->hasChainedFixups() )
-        this->forEachBindTarget_ChainedFixups(diag, handler);
-    else if ( this->hasOpcodeFixups() )
-        this->forEachBindTarget_Opcodes(diag, allowLazyBinds, handler, overrideHandler);
-    else
-        this->forEachBindTarget_Relocations(diag, handler);
-}
-
-struct WeakBindInfo
-{
-    uint64_t    segIndex  :  8,
-                segOffset : 56;
-};
-
-// walk through all binds, unifying weak, lazy, and regular binds
-void MachOAnalyzer::forEachBindUnified_Opcodes(Diagnostics& diag, bool allowLazyBinds,
-                                               void (^handler)(uint64_t runtimeOffset, const BindTargetInfo& targetInfo, bool& stop),
-                                               void (^overrideHandler)(uint64_t runtimeOffset, const BindTargetInfo& targetInfo, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return;
-
-    {
-        __block unsigned         targetIndex = 0;
-        __block BindTargetInfo   targetInfo;
-        BindDetailedHandler binder =  ^(const char* opcodeName, const LinkEditInfo&, const SegmentInfo segments[],
-                                        bool segIndexSet,  bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                                        uint32_t pointerSize, uint8_t segmentIndex, uint64_t segmentOffset,
-                                        uint8_t type, const char* symbolName, bool weakImport, bool lazyBind,
-                                        uint64_t addend, bool targetOrAddendChanged, bool& stop) {
-            uint64_t bindVmOffset  = segments[segmentIndex].vmAddr + segmentOffset;
-            uint64_t runtimeOffset = bindVmOffset - leInfo.layout.textUnslidVMAddr;
-            if ( targetOrAddendChanged ) {
-                targetInfo.targetIndex = targetIndex++;
-                targetInfo.libOrdinal  = libOrdinal;
-                targetInfo.symbolName  = symbolName;
-                targetInfo.addend      = addend;
-                targetInfo.weakImport  = weakImport;
-                targetInfo.lazyBind    = lazyBind && allowLazyBinds;
-            }
-            handler(runtimeOffset, targetInfo, stop);
-        };
-        bool stopped = this->forEachBind_OpcodesRegular(diag, leInfo, segmentsInfo, binder);
-        if ( stopped )
-            return;
-        stopped = this->forEachBind_OpcodesLazy(diag, leInfo, segmentsInfo, binder);
-        if ( stopped )
-            return;
-    }
-
-    // Opcode based weak-binds effectively override other binds/rebases.  Process them last
-    // To match dyld2, they are allowed to fail to find a target, in which case the normal rebase/bind will
-    // not be overridden.
-    {
-        __block unsigned         weakTargetIndex = 0;
-        __block BindTargetInfo   weakTargetInfo;
-        BindDetailedHandler weakBinder =  ^(const char* opcodeName, const LinkEditInfo&, const SegmentInfo segments[],
-                                            bool segIndexSet,  bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                                            uint32_t pointerSize, uint8_t segmentIndex, uint64_t segmentOffset,
-                                            uint8_t type, const char* symbolName, bool weakImport, bool lazyBind,
-                                            uint64_t addend, bool targetOrAddendChanged, bool& stop) {
-            
-            uint64_t bindVmOffset  = segmentsInfo[segmentIndex].vmAddr + segmentOffset;
-            uint64_t runtimeOffset = bindVmOffset - leInfo.layout.textUnslidVMAddr;
-            if ( (symbolName != weakTargetInfo.symbolName) || (strcmp(symbolName, weakTargetInfo.symbolName) != 0) || (weakTargetInfo.addend != addend) ) {
-                weakTargetInfo.targetIndex = weakTargetIndex++;
-                weakTargetInfo.libOrdinal  = BIND_SPECIAL_DYLIB_WEAK_LOOKUP;
-                weakTargetInfo.symbolName  = symbolName;
-                weakTargetInfo.addend      = addend;
-                weakTargetInfo.weakImport  = false;
-                weakTargetInfo.lazyBind    = false;
-            }
-            overrideHandler(runtimeOffset, weakTargetInfo, stop);
-        };
-        auto strongHandler = ^(const char* strongName) { };
-        this->forEachBind_OpcodesWeak(diag, leInfo, segmentsInfo, weakBinder, strongHandler);
-    }
-}
-
-void MachOAnalyzer::forEachBindTarget_Opcodes(Diagnostics& diag, bool allowLazyBinds,
-                                              void (^handler)(const BindTargetInfo& info, bool& stop),
-                                              void (^overrideHandler)(const BindTargetInfo& info, bool& stop)) const
-{
-    __block unsigned lastTargetIndex = -1;
-    __block unsigned lastWeakBindTargetIndex = -1;
-    this->forEachBindUnified_Opcodes(diag, allowLazyBinds, ^(uint64_t runtimeOffset, const BindTargetInfo& targetInfo, bool& stop) {
-        // Regular/lazy binds
-        if ( lastTargetIndex != targetInfo.targetIndex) {
-            handler(targetInfo, stop);
-            lastTargetIndex = targetInfo.targetIndex;
-        }
-    }, ^(uint64_t runtimeOffset, const BindTargetInfo& targetInfo, bool& stop) {
-        // Weak binds
-        if ( lastWeakBindTargetIndex != targetInfo.targetIndex) {
-            overrideHandler(targetInfo, stop);
-            lastWeakBindTargetIndex = targetInfo.targetIndex;
-        }
-    });
-}
-
-void MachOAnalyzer::forEachBindTarget_ChainedFixups(Diagnostics& diag, void (^handler)(const BindTargetInfo& info, bool& stop)) const
-{
-    __block unsigned targetIndex = 0;
-    this->forEachChainedFixupTarget(diag, ^(int libOrdinal, const char* symbolName, uint64_t addend, bool weakImport, bool& stop)  {
-        BindTargetInfo info;
-        info.targetIndex = targetIndex;
-        info.libOrdinal  = libOrdinal;
-        info.symbolName  = symbolName;
-        info.addend      = addend;
-        info.weakImport  = weakImport;
-        info.lazyBind    = false;
-        handler(info, stop);
-       ++targetIndex;
-    });
-
-    // The C++ spec says main executables can define non-weak functions which override weak-defs in dylibs
-    // This happens automatically for anything bound at launch, but the dyld cache is pre-bound so we need
-    // to patch any binds that are overridden by this non-weak in the main executable.
-    if ( diag.noError() && this->isMainExecutable() && this->hasWeakDefs() ) {
-        MachOAnalyzer::forEachTreatAsWeakDef(^(const char* symbolName) {
-            BindTargetInfo info;
-            info.targetIndex = targetIndex;
-            info.libOrdinal  = BIND_SPECIAL_DYLIB_WEAK_LOOKUP;
-            info.symbolName  = symbolName;
-            info.addend      = 0;
-            info.weakImport  = false;
-            info.lazyBind    = false;
-            bool stop = false;
-            handler(info, stop);
-           ++targetIndex;
-        });
-    }
-}
-
-
-// old binary, walk external relocations and indirect symbol table
-void MachOAnalyzer::forEachBindTarget_Relocations(Diagnostics& diag, void (^handler)(const BindTargetInfo& info, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return;
-
-    __block unsigned targetIndex = 0;
-    this->forEachBind_Relocations(diag, leInfo, segmentsInfo, true,
-                                  ^(const char* opcodeName, const LinkEditInfo&, const SegmentInfo segments[],
-                                    bool segIndexSet,  bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                                    uint32_t pointerSize, uint8_t segmentIndex, uint64_t segmentOffset,
-                                    uint8_t type, const char* symbolName, bool weakImport, bool lazyBind,
-                                    uint64_t addend, bool targetOrAddendChanged, bool& stop) {
-        if ( targetOrAddendChanged ) {
-            BindTargetInfo info;
-            info.targetIndex = targetIndex;
-            info.libOrdinal  = libOrdinal;
-            info.symbolName  = symbolName;
-            info.addend      = addend;
-            info.weakImport  = weakImport;
-            info.lazyBind    = lazyBind;
-            handler(info, stop);
-           ++targetIndex;
-        }
-    });
-}
-
-
-void MachOAnalyzer::forEachBindLocation_Relocations(Diagnostics& diag, void (^handler)(uint64_t runtimeOffset, unsigned targetIndex, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return;
-
-    __block int targetIndex = -1;
-    this->forEachBind_Relocations(diag, leInfo, segmentsInfo, false,
-                                  ^(const char* opcodeName, const LinkEditInfo&, const SegmentInfo segments[],
-                                    bool segIndexSet,  bool libraryOrdinalSet, uint32_t dylibCount, int libOrdinal,
-                                    uint32_t pointerSize, uint8_t segmentIndex, uint64_t segmentOffset,
-                                    uint8_t type, const char* symbolName, bool weakImport, bool lazyBind,
-                                    uint64_t addend, bool targetOrAddendChanged, bool& stop) {
-        if ( targetOrAddendChanged )
-            ++targetIndex;
-        uint64_t bindVmOffset  = segments[segmentIndex].vmAddr + segmentOffset;
-        uint64_t runtimeOffset = bindVmOffset - leInfo.layout.textUnslidVMAddr;
-        handler(runtimeOffset, targetIndex, stop);
-    });
-}
-
-
-bool MachOAnalyzer::forEachBind_Relocations(Diagnostics& diag, const LinkEditInfo& leInfo, const SegmentInfo segmentsInfo[],
-                                            bool supportPrivateExternsWorkaround, BindDetailedHandler handler) const
-{
-    const uint64_t                  relocsStartAddress = externalRelocBaseAddress(segmentsInfo, leInfo.layout.linkeditSegIndex);
-    const relocation_info* const    relocsStart = (relocation_info*)getLinkEditContent(leInfo.layout, leInfo.dynSymTab->extreloff);
-    const relocation_info* const    relocsEnd   = &relocsStart[leInfo.dynSymTab->nextrel];
-    bool                            is64Bit     = is64() ;
-    const uint32_t                  ptrSize     = pointerSize();
-    const uint32_t                  dylibCount  = dependentDylibCount();
-    const uint8_t                   relocSize   = (is64Bit ? 3 : 2);
-    const void*                     symbolTable = getLinkEditContent(leInfo.layout, leInfo.symTab->symoff);
-    const struct nlist_64*          symbols64   = (nlist_64*)symbolTable;
-    const struct nlist*             symbols32   = (struct nlist*)symbolTable;
-    const char*                     stringPool  = (char*)getLinkEditContent(leInfo.layout, leInfo.symTab->stroff);
-    uint32_t                        symCount    = leInfo.symTab->nsyms;
-    uint32_t                        poolSize    = leInfo.symTab->strsize;
-    uint32_t                        lastSymIndx = -1;
-    uint64_t                        lastAddend  = 0;
-    bool                            stop        = false;
-    for (const relocation_info* reloc=relocsStart; (reloc < relocsEnd) && !stop; ++reloc) {
-        bool isBranch = false;
-#if BUILDING_APP_CACHE_UTIL
-        if ( isKextBundle() ) {
-            // kext's may have other kinds of relocations, eg, branch relocs.  Skip them
-            if ( isArch("x86_64") || isArch("x86_64h") ) {
-                if ( reloc->r_type == X86_64_RELOC_BRANCH ) {
-                    if ( reloc->r_length != 2 ) {
-                        diag.error("external relocation has wrong r_length");
-                        break;
-                    }
-                    if ( reloc->r_pcrel != true ) {
-                        diag.error("external relocation should be pcrel");
-                        break;
-                    }
-                    isBranch = true;
-                }
-            }
-        }
-#endif
-        if ( !isBranch ) {
-            if ( reloc->r_length != relocSize ) {
-                diag.error("external relocation has wrong r_length");
-                break;
-            }
-            if ( reloc->r_type != 0 ) { // 0 == X86_64_RELOC_UNSIGNED == GENERIC_RELOC_VANILLA == ARM64_RELOC_UNSIGNED
-                diag.error("external relocation has wrong r_type");
-                break;
-            }
-        }
-        uint32_t segIndex  = 0;
-        uint64_t segOffset = 0;
-        if ( segIndexAndOffsetForAddress(relocsStartAddress+reloc->r_address, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-            uint32_t symbolIndex = reloc->r_symbolnum;
-            if ( symbolIndex > symCount ) {
-                diag.error("external relocation has out of range r_symbolnum");
-                break;
-            }
-            else {
-                uint32_t strOffset  = is64Bit ? symbols64[symbolIndex].n_un.n_strx : symbols32[symbolIndex].n_un.n_strx;
-                uint16_t n_desc     = is64Bit ? symbols64[symbolIndex].n_desc : symbols32[symbolIndex].n_desc;
-                uint8_t  n_type     = is64Bit ? symbols64[symbolIndex].n_type : symbols32[symbolIndex].n_type;
-                uint32_t libOrdinal = libOrdinalFromDesc(n_desc);
-                if ( strOffset >= poolSize ) {
-                    diag.error("external relocation has r_symbolnum=%d which has out of range n_strx", symbolIndex);
-                    break;
-                }
-                else {
-                    const char*     symbolName = stringPool + strOffset;
-                    bool            weakImport = (n_desc & N_WEAK_REF);
-                    const uint8_t*  content    = (uint8_t*)this + segmentsInfo[segIndex].vmAddr - leInfo.layout.textUnslidVMAddr + segOffset;
-                    uint64_t        addend     = (reloc->r_length == 3) ? *((uint64_t*)content) : *((uint32_t*)content);
-                    // Handle defined weak def symbols which need to get a special ordinal
-                    if ( ((n_type & N_TYPE) == N_SECT) && ((n_type & N_EXT) != 0) && ((n_desc & N_WEAK_DEF) != 0) )
-                        libOrdinal = BIND_SPECIAL_DYLIB_WEAK_LOOKUP;
-                    uint8_t type = isBranch ? BIND_TYPE_TEXT_PCREL32 : BIND_TYPE_POINTER;
-                    bool targetOrAddendChanged = (lastSymIndx != symbolIndex) || (lastAddend != addend);
-                    handler("external relocation", leInfo, segmentsInfo, true, true, dylibCount, libOrdinal,
-                             ptrSize, segIndex, segOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                    lastSymIndx = symbolIndex;
-                    lastAddend  = addend;
-                }
-            }
-        }
-        else {
-            diag.error("local relocation has out of range r_address");
-            break;
-        }
-    }
-    // then process indirect symbols
-    forEachIndirectPointer(diag, supportPrivateExternsWorkaround,
-                           ^(uint64_t address, bool bind, int bindLibOrdinal,
-                             const char* bindSymbolName, bool bindWeakImport, bool bindLazy, bool selfModifyingStub, bool& indStop) {
-        if ( !bind )
-           return;
-        uint32_t segIndex  = 0;
-        uint64_t segOffset = 0;
-        if ( segIndexAndOffsetForAddress(address, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-            handler("indirect symbol", leInfo, segmentsInfo, true, true, dylibCount, bindLibOrdinal,
-                     ptrSize, segIndex, segOffset, BIND_TYPE_POINTER, bindSymbolName, bindWeakImport, bindLazy, 0, true, indStop);
-        }
-        else {
-            diag.error("indirect symbol has out of range address");
-            indStop = true;
-        }
-    });
-
-    return false;
-}
-
-void MachOAnalyzer::forEachBindLocation_Opcodes(Diagnostics& diag, void (^handler)(uint64_t runtimeOffset, unsigned targetIndex, bool& stop),
-                                                void (^overrideHandler)(uint64_t runtimeOffset, unsigned overrideBindTargetIndex, bool& stop)) const
-{
-    this->forEachBindUnified_Opcodes(diag, false, ^(uint64_t runtimeOffset, const BindTargetInfo& targetInfo, bool& stop) {
-        handler(runtimeOffset, targetInfo.targetIndex, stop);
-    }, ^(uint64_t runtimeOffset, const BindTargetInfo& weakTargetInfo, bool& stop) {
-        overrideHandler(runtimeOffset, weakTargetInfo.targetIndex, stop);
-    });
-}
-
-bool MachOAnalyzer::forEachBind_OpcodesLazy(Diagnostics& diag, const LinkEditInfo& leInfo, const SegmentInfo segmentsInfo[], BindDetailedHandler handler) const
-{
-    if ( (leInfo.dyldInfo == nullptr) || (leInfo.dyldInfo->lazy_bind_size == 0) )
-        return false;
-
-    uint32_t        lazyDoneCount   = 0;
-    uint32_t        lazyBindCount   = 0;
-    const uint32_t  ptrSize         = this->pointerSize();
-    bool            stop            = false;
-    const uint32_t  dylibCount      = dependentDylibCount();
-    const uint8_t*  p               = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->lazy_bind_off);
-    const uint8_t*  end             = p + leInfo.dyldInfo->lazy_bind_size;
-    uint8_t         type            = BIND_TYPE_POINTER;
-    uint64_t        segmentOffset   = 0;
-    uint8_t         segmentIndex    = 0;
-    const char*     symbolName      = NULL;
-    int             libraryOrdinal  = 0;
-    bool            segIndexSet     = false;
-    bool            libraryOrdinalSet = false;
-    int64_t         addend          = 0;
-    bool            weakImport = false;
-    while (  !stop && diag.noError() && (p < end) ) {
-        uint8_t immediate = *p & BIND_IMMEDIATE_MASK;
-        uint8_t opcode = *p & BIND_OPCODE_MASK;
-        ++p;
-        switch (opcode) {
-            case BIND_OPCODE_DONE:
-                // this opcode marks the end of each lazy pointer binding
-                ++lazyDoneCount;
-                break;
-            case BIND_OPCODE_SET_DYLIB_ORDINAL_IMM:
-                libraryOrdinal = immediate;
-                libraryOrdinalSet = true;
-                break;
-            case BIND_OPCODE_SET_DYLIB_ORDINAL_ULEB:
-                libraryOrdinal = (int)read_uleb128(diag, p, end);
-                libraryOrdinalSet = true;
-                break;
-            case BIND_OPCODE_SET_DYLIB_SPECIAL_IMM:
-                // the special ordinals are negative numbers
-                if ( immediate == 0 )
-                    libraryOrdinal = 0;
-                else {
-                    int8_t signExtended = BIND_OPCODE_MASK | immediate;
-                    libraryOrdinal = signExtended;
-                }
-                libraryOrdinalSet = true;
-                break;
-            case BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM:
-                weakImport = ( (immediate & BIND_SYMBOL_FLAGS_WEAK_IMPORT) != 0 );
-                symbolName = (char*)p;
-                while (*p != '\0')
-                    ++p;
-                ++p;
-                break;
-            case BIND_OPCODE_SET_ADDEND_SLEB:
-                addend = read_sleb128(diag, p, end);
-                break;
-            case BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                segmentIndex = immediate;
-                segmentOffset = read_uleb128(diag, p, end);
-                segIndexSet = true;
-                break;
-            case BIND_OPCODE_DO_BIND:
-                handler("BIND_OPCODE_DO_BIND", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                        ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, true, addend, true, stop);
-                segmentOffset += ptrSize;
-                ++lazyBindCount;
-                break;
-            case BIND_OPCODE_SET_TYPE_IMM:
-            case BIND_OPCODE_ADD_ADDR_ULEB:
-            case BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB:
-            case BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED:
-            case BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB:
-            default:
-                diag.error("bad lazy bind opcode 0x%02X", opcode);
-                break;
-        }
-    }
-    if ( lazyDoneCount > lazyBindCount+7 ) {
-        // diag.error("lazy bind opcodes missing binds");
-    }
-    return stop;
-}
-
-
-
-bool MachOAnalyzer::forEachBind_OpcodesWeak(Diagnostics& diag, const LinkEditInfo& leInfo, const SegmentInfo segmentsInfo[], BindDetailedHandler handler,  void (^strongHandler)(const char* symbolName)) const
-{
-   if ( (leInfo.dyldInfo == nullptr) || (leInfo.dyldInfo->weak_bind_size == 0) )
-        return false;
-
-    const uint32_t  ptrSize         = this->pointerSize();
-    bool            stop            = false;
-    const uint32_t  dylibCount      = dependentDylibCount();
-    const uint8_t*  p               = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->weak_bind_off);
-    const uint8_t*  end             = p + leInfo.dyldInfo->weak_bind_size;
-    uint8_t         type            = BIND_TYPE_POINTER;
-    uint64_t        segmentOffset   = 0;
-    uint8_t         segmentIndex    = 0;
-    const char*     symbolName      = NULL;
-    int             libraryOrdinal  = BIND_SPECIAL_DYLIB_WEAK_LOOKUP;
-    bool            segIndexSet     = false;
-    bool            libraryOrdinalSet = true;
-    int64_t         addend          = 0;
-    bool            weakImport      = false;
-    bool            targetOrAddendChanged   = true;
-    bool            done            = false;
-    uint64_t        count;
-    uint64_t        skip;
-    while ( !stop && diag.noError() && (p < end) && !done ) {
-        uint8_t immediate = *p & BIND_IMMEDIATE_MASK;
-        uint8_t opcode = *p & BIND_OPCODE_MASK;
-        ++p;
-        switch (opcode) {
-            case BIND_OPCODE_DONE:
-                done = true;
-                break;
-            case BIND_OPCODE_SET_DYLIB_ORDINAL_IMM:
-            case BIND_OPCODE_SET_DYLIB_ORDINAL_ULEB:
-            case BIND_OPCODE_SET_DYLIB_SPECIAL_IMM:
-                diag.error("unexpected dylib ordinal in weak_bind");
-                break;
-            case BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM:
-                weakImport = ( (immediate & BIND_SYMBOL_FLAGS_WEAK_IMPORT) != 0 );
-                symbolName = (char*)p;
-                while (*p != '\0')
-                    ++p;
-                ++p;
-                if ( immediate & BIND_SYMBOL_FLAGS_NON_WEAK_DEFINITION ) {
-                    strongHandler(symbolName);
-                }
-                targetOrAddendChanged = true;
-                break;
-            case BIND_OPCODE_SET_TYPE_IMM:
-                type = immediate;
-                break;
-            case BIND_OPCODE_SET_ADDEND_SLEB:
-                addend = read_sleb128(diag, p, end);
-                targetOrAddendChanged = true;
-                break;
-            case BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                segmentIndex = immediate;
-                segmentOffset = read_uleb128(diag, p, end);
-                segIndexSet = true;
-                break;
-            case BIND_OPCODE_ADD_ADDR_ULEB:
-                segmentOffset += read_uleb128(diag, p, end);
-                break;
-            case BIND_OPCODE_DO_BIND:
-                handler("BIND_OPCODE_DO_BIND", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                        ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                segmentOffset += ptrSize;
-                targetOrAddendChanged = false;
-                break;
-            case BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB:
-                handler("BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                        ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                segmentOffset += read_uleb128(diag, p, end) + ptrSize;
-                 targetOrAddendChanged = false;
-               break;
-            case BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED:
-                handler("BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                        ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                segmentOffset += immediate*ptrSize + ptrSize;
-                targetOrAddendChanged = false;
-                break;
-            case BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB:
-                count = read_uleb128(diag, p, end);
-                skip = read_uleb128(diag, p, end);
-                for (uint32_t i=0; i < count; ++i) {
-                    handler("BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                            ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                    segmentOffset += skip + ptrSize;
-                    targetOrAddendChanged = false;
-                    if ( stop )
-                        break;
-                }
-                break;
-            default:
-                diag.error("bad bind opcode 0x%02X", *p);
-        }
-    }
-    return stop;
-}
-
-bool MachOAnalyzer::forEachBind_OpcodesRegular(Diagnostics& diag, const LinkEditInfo& leInfo, const SegmentInfo segmentsInfo[], BindDetailedHandler handler) const
-{
-    if ( (leInfo.dyldInfo == nullptr) || (leInfo.dyldInfo->bind_size == 0) )
-        return false;
-
-    const uint32_t  ptrSize         = this->pointerSize();
-    bool            stop            = false;
-    const uint32_t  dylibCount      = dependentDylibCount();
-    const uint8_t*  p               = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->bind_off);
-    const uint8_t*  end             = p + leInfo.dyldInfo->bind_size;
-    uint8_t         type            = 0;
-    uint64_t        segmentOffset   = 0;
-    uint8_t         segmentIndex    = 0;
-    const char*     symbolName      = NULL;
-    int             libraryOrdinal  = 0;
-    bool            segIndexSet     = false;
-    bool            libraryOrdinalSet = false;
-    bool            targetOrAddendChanged   = false;
-    bool            done            = false;
-    int64_t         addend          = 0;
-    uint64_t        count;
-    uint64_t        skip;
-    bool            weakImport = false;
-    while ( !stop && diag.noError() && (p < end) && !done ) {
-        uint8_t immediate = *p & BIND_IMMEDIATE_MASK;
-        uint8_t opcode = *p & BIND_OPCODE_MASK;
-        ++p;
-        switch (opcode) {
-            case BIND_OPCODE_DONE:
-                done = true;
-                break;
-            case BIND_OPCODE_SET_DYLIB_ORDINAL_IMM:
-                libraryOrdinal = immediate;
-                libraryOrdinalSet = true;
-                break;
-            case BIND_OPCODE_SET_DYLIB_ORDINAL_ULEB:
-                libraryOrdinal = (int)read_uleb128(diag, p, end);
-                libraryOrdinalSet = true;
-                break;
-            case BIND_OPCODE_SET_DYLIB_SPECIAL_IMM:
-                // the special ordinals are negative numbers
-                if ( immediate == 0 )
-                    libraryOrdinal = 0;
-                else {
-                    int8_t signExtended = BIND_OPCODE_MASK | immediate;
-                    libraryOrdinal = signExtended;
-                }
-                libraryOrdinalSet = true;
-                break;
-            case BIND_OPCODE_SET_SYMBOL_TRAILING_FLAGS_IMM:
-                weakImport = ( (immediate & BIND_SYMBOL_FLAGS_WEAK_IMPORT) != 0 );
-                symbolName = (char*)p;
-                while (*p != '\0')
-                    ++p;
-                ++p;
-                targetOrAddendChanged = true;
-                break;
-            case BIND_OPCODE_SET_TYPE_IMM:
-                type = immediate;
-                break;
-            case BIND_OPCODE_SET_ADDEND_SLEB:
-                addend = read_sleb128(diag, p, end);
-                targetOrAddendChanged = true;
-                break;
-            case BIND_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                segmentIndex = immediate;
-                segmentOffset = read_uleb128(diag, p, end);
-                segIndexSet = true;
-                break;
-            case BIND_OPCODE_ADD_ADDR_ULEB:
-                segmentOffset += read_uleb128(diag, p, end);
-                break;
-            case BIND_OPCODE_DO_BIND:
-                handler("BIND_OPCODE_DO_BIND", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                        ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                segmentOffset += ptrSize;
-                targetOrAddendChanged = false;
-                break;
-            case BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB:
-                handler("BIND_OPCODE_DO_BIND_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                        ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                segmentOffset += read_uleb128(diag, p, end) + ptrSize;
-                targetOrAddendChanged = false;
-                break;
-            case BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED:
-                handler("BIND_OPCODE_DO_BIND_ADD_ADDR_IMM_SCALED", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                        ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                segmentOffset += immediate*ptrSize + ptrSize;
-                targetOrAddendChanged = false;
-                break;
-            case BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB:
-                count = read_uleb128(diag, p, end);
-                skip = read_uleb128(diag, p, end);
-                for (uint32_t i=0; i < count; ++i) {
-                    handler("BIND_OPCODE_DO_BIND_ULEB_TIMES_SKIPPING_ULEB", leInfo, segmentsInfo, segIndexSet, libraryOrdinalSet, dylibCount, libraryOrdinal,
-                            ptrSize, segmentIndex, segmentOffset, type, symbolName, weakImport, false, addend, targetOrAddendChanged, stop);
-                    segmentOffset += skip + ptrSize;
-                    targetOrAddendChanged = false;
-                    if ( stop )
-                        break;
-                }
-                break;
-            default:
-                diag.error("bad bind opcode 0x%02X", *p);
-        }
-    }
-    return stop;
-}
-
-bool MachOAnalyzer::forEachRebaseLocation_Opcodes(Diagnostics& diag, void (^handler)(uint64_t runtimeOffset, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return false;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return false;
-
-    return this->forEachRebase_Opcodes(diag, leInfo, segmentsInfo, ^(const char* opcodeName, const LinkEditInfo& rleInfo, const SegmentInfo segments[],
-                                        bool segIndexSet, uint32_t pointerSize, uint8_t segmentIndex, uint64_t segmentOffset, Rebase kind, bool& stop) {
-        uint64_t rebaseVmOffset = segments[segmentIndex].vmAddr + segmentOffset;
-        uint64_t runtimeOffset  = rebaseVmOffset - leInfo.layout.textUnslidVMAddr;
-        handler(runtimeOffset, stop);
-    });
-}
-
-bool MachOAnalyzer::forEachRebase_Opcodes(Diagnostics& diag, const LinkEditInfo& leInfo, const SegmentInfo segmentsInfo[], RebaseDetailHandler handler) const
-{
-    const Rebase pointerRebaseKind = is64() ? Rebase::pointer64 : Rebase::pointer32;
-    assert(leInfo.dyldInfo != nullptr);
-
-    const uint8_t* const start = getLinkEditContent(leInfo.layout, leInfo.dyldInfo->rebase_off);
-    const uint8_t* const end   = start + leInfo.dyldInfo->rebase_size;
-    const uint8_t* p           = start;
-    const uint32_t ptrSize     = pointerSize();
-    Rebase   kind = Rebase::unknown;
-    int      segIndex = 0;
-    uint64_t segOffset = 0;
-    uint64_t count;
-    uint64_t skip;
-    bool     segIndexSet = false;
-    bool     stop = false;
-    while ( !stop && diag.noError() && (p < end) ) {
-        uint8_t immediate = *p & REBASE_IMMEDIATE_MASK;
-        uint8_t opcode = *p & REBASE_OPCODE_MASK;
-        ++p;
-        switch (opcode) {
-            case REBASE_OPCODE_DONE:
-                // Allow some padding, in case rebases were somehow aligned to 16-bytes in size
-                if ( (end - p) > 15 )
-                    diag.error("rebase opcodes terminated early at offset %d of %d", (int)(p-start), (int)(end-start));
-                stop = true;
-                break;
-            case REBASE_OPCODE_SET_TYPE_IMM:
-                switch ( immediate ) {
-                    case REBASE_TYPE_POINTER:
-                        kind = pointerRebaseKind;
-                        break;
-                    case REBASE_TYPE_TEXT_ABSOLUTE32:
-                        kind = Rebase::textAbsolute32;
-                        break;
-                    case REBASE_TYPE_TEXT_PCREL32:
-                        kind = Rebase::textPCrel32;
-                        break;
-                    default:
-                        kind = Rebase::unknown;
-                        break;
-                }
-                break;
-            case REBASE_OPCODE_SET_SEGMENT_AND_OFFSET_ULEB:
-                segIndex = immediate;
-                segOffset = read_uleb128(diag, p, end);
-                segIndexSet = true;
-                break;
-            case REBASE_OPCODE_ADD_ADDR_ULEB:
-                segOffset += read_uleb128(diag, p, end);
-                break;
-            case REBASE_OPCODE_ADD_ADDR_IMM_SCALED:
-                segOffset += immediate*ptrSize;
-                break;
-            case REBASE_OPCODE_DO_REBASE_IMM_TIMES:
-                for (int i=0; i < immediate; ++i) {
-                    handler("REBASE_OPCODE_DO_REBASE_IMM_TIMES", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                    segOffset += ptrSize;
-                    if ( stop )
-                        break;
-                }
-                break;
-            case REBASE_OPCODE_DO_REBASE_ULEB_TIMES:
-                count = read_uleb128(diag, p, end);
-                for (uint32_t i=0; i < count; ++i) {
-                    handler("REBASE_OPCODE_DO_REBASE_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                    segOffset += ptrSize;
-                    if ( stop )
-                        break;
-                }
-                break;
-            case REBASE_OPCODE_DO_REBASE_ADD_ADDR_ULEB:
-                handler("REBASE_OPCODE_DO_REBASE_ADD_ADDR_ULEB", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                segOffset += read_uleb128(diag, p, end) + ptrSize;
-                break;
-            case REBASE_OPCODE_DO_REBASE_ULEB_TIMES_SKIPPING_ULEB:
-                count = read_uleb128(diag, p, end);
-                if ( diag.hasError() )
-                    break;
-                skip = read_uleb128(diag, p, end);
-                for (uint32_t i=0; i < count; ++i) {
-                    handler("REBASE_OPCODE_DO_REBASE_ULEB_TIMES_SKIPPING_ULEB", leInfo, segmentsInfo, segIndexSet, ptrSize, segIndex, segOffset, kind, stop);
-                    segOffset += skip + ptrSize;
-                    if ( stop )
-                        break;
-                }
-                break;
-            default:
-                diag.error("unknown rebase opcode 0x%02X", opcode);
-        }
-    }
-    return stop;
-}
-
-bool MachOAnalyzer::forEachRebaseLocation_Relocations(Diagnostics& diag, void (^handler)(uint64_t runtimeOffset, bool& stop)) const
-{
-    LinkEditInfo leInfo;
-    getLinkEditPointers(diag, leInfo);
-    if ( diag.hasError() )
-        return false;
-
-    BLOCK_ACCCESSIBLE_ARRAY(SegmentInfo, segmentsInfo, leInfo.layout.lastSegIndex+1);
-    getAllSegmentsInfos(diag, segmentsInfo);
-    if ( diag.hasError() )
-        return false;
-
-    return this->forEachRebase_Relocations(diag, leInfo, segmentsInfo, ^(const char* opcodeName, const LinkEditInfo& rleInfo, const SegmentInfo segments[],
-                                           bool segIndexSet, uint32_t pointerSize, uint8_t segmentIndex, uint64_t segmentOffset, Rebase kind, bool& stop) {
-        uint64_t rebaseVmOffset = segments[segmentIndex].vmAddr + segmentOffset;
-        uint64_t runtimeOffset  = rebaseVmOffset - leInfo.layout.textUnslidVMAddr;
-        handler(runtimeOffset, stop);
-    });
-}
-
-// relocs are normally sorted, we don't want to use qsort because it may switch to mergesort which uses malloc
-void MachOAnalyzer::sortRelocations(Array<relocation_info>& relocs) const
-{
-    // The kernel linker has malloc, and old-style relocations are extremely common.  So use qsort
-#if BUILDING_APP_CACHE_UTIL
-    ::qsort(&relocs[0], relocs.count(), sizeof(relocation_info),
-            [](const void* l, const void* r) -> int {
-                if ( ((relocation_info*)l)->r_address < ((relocation_info*)r)->r_address )
-                    return -1;
-                else
-                    return 1;
-    });
-#else
-    uintptr_t count = relocs.count();
-    for (uintptr_t i=0; i < count-1; ++i) {
-        bool done = true;
-        for (uintptr_t j=0; j < count-i-1; ++j) {
-            if ( relocs[j].r_address > relocs[j+1].r_address ) {
-                relocation_info temp = relocs[j];
-                relocs[j]   = relocs[j+1];
-                relocs[j+1] = temp;
-                done = false;
-            }
-        }
-        if ( done )
-            break;
-    }
-#endif
-}
-
-
-bool MachOAnalyzer::forEachRebase_Relocations(Diagnostics& diag, const LinkEditInfo& leInfo, const SegmentInfo segmentsInfo[], RebaseDetailHandler handler) const
-{
-    // old binary, walk relocations
-    const uint64_t                  relocsStartAddress = localRelocBaseAddress(segmentsInfo, leInfo.layout.linkeditSegIndex);
-    const relocation_info* const    relocsStart = (relocation_info*)getLinkEditContent(leInfo.layout, leInfo.dynSymTab->locreloff);
-    const relocation_info* const    relocsEnd   = &relocsStart[leInfo.dynSymTab->nlocrel];
-    const uint8_t                   relocSize   = (is64() ? 3 : 2);
-    const uint8_t                   ptrSize     = pointerSize();
-    bool                            stop        = false;
-    STACK_ALLOC_OVERFLOW_SAFE_ARRAY(relocation_info, relocs, 2048);
-    for (const relocation_info* reloc=relocsStart; (reloc < relocsEnd) && !stop; ++reloc) {
-        if ( reloc->r_length != relocSize ) {
-            bool shouldEmitError = true;
-#if BUILDING_APP_CACHE_UTIL
-            if ( usesClassicRelocationsInKernelCollection() && (reloc->r_length == 2) && (relocSize == 3) )
-                shouldEmitError = false;
-#endif
-            if ( shouldEmitError ) {
-                diag.error("local relocation has wrong r_length");
-                break;
-            }
-        }
-        if ( reloc->r_type != 0 ) { // 0 == X86_64_RELOC_UNSIGNED == GENERIC_RELOC_VANILLA ==  ARM64_RELOC_UNSIGNED
-            diag.error("local relocation has wrong r_type");
-            break;
-        }
-        relocs.push_back(*reloc);
-    }
-    if ( !relocs.empty() ) {
-        sortRelocations(relocs);
-        for (relocation_info reloc : relocs) {
-            uint32_t addrOff = reloc.r_address;
-            uint32_t segIndex  = 0;
-            uint64_t segOffset = 0;
-            uint64_t addr = 0;
-#if BUILDING_APP_CACHE_UTIL
-            // xnu for x86_64 has __HIB mapped before __DATA, so offsets appear to be
-            // negative
-            if ( isStaticExecutable() || isFileSet() ) {
-                addr = relocsStartAddress + (int32_t)addrOff;
-            } else {
-                addr = relocsStartAddress + addrOff;
-            }
-#else
-            addr = relocsStartAddress + addrOff;
-#endif
-            if ( segIndexAndOffsetForAddress(addr, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-                Rebase kind = (reloc.r_length == 2) ? Rebase::pointer32 : Rebase::pointer64;
-                if ( this->cputype == CPU_TYPE_I386 ) {
-                    if ( segmentsInfo[segIndex].executable() )
-                        kind = Rebase::textAbsolute32;
-                }
-                handler("local relocation", leInfo, segmentsInfo, true, ptrSize, segIndex, segOffset, kind, stop);
-            }
-            else {
-                diag.error("local relocation has out of range r_address");
-                break;
-            }
-        }
-    }
-    // then process indirect symbols
-    const Rebase pointerRebaseKind = is64() ? Rebase::pointer64 : Rebase::pointer32;
-    forEachIndirectPointer(diag, false,
-                           ^(uint64_t address, bool bind, int bindLibOrdinal,
-                             const char* bindSymbolName, bool bindWeakImport, bool bindLazy, bool selfModifyingStub, bool& indStop) {
-        if ( bind )
-           return;
-        uint32_t segIndex  = 0;
-        uint64_t segOffset = 0;
-        if ( segIndexAndOffsetForAddress(address, segmentsInfo, leInfo.layout.linkeditSegIndex, segIndex, segOffset) ) {
-            handler("local relocation", leInfo, segmentsInfo, true, ptrSize, segIndex, segOffset, pointerRebaseKind, indStop);
-        }
-        else {
-            diag.error("local relocation has out of range r_address");
-            indStop = true;
-        }
-    });
-
-    return stop;
-}
-
-
-
-} // dyld3
-
-