OpenCore bootloader
This report summarizes scanner evidence for acidanthera/OpenCorePkg, a C project with 15,608 GitHub stars. SynthScan examined 415,926 lines of code across 1125 source files, recording 1129 pattern matches distributed across 12 syntactic categories. The legacy adjusted pattern signal is 2.6 (Low pattern signal); it is not an authorship estimate.
Use the findings to ask whether maintainers can explain intent, verify dependencies, simplify unnecessary scaffolding, and focus review effort. Risk Payload v1 adds ownership, supply-chain, over-engineering, and review-effort vectors when available. These are prompts for human verification, not probabilities, security findings, or measured maintenance costs.
Use each vector to guide manual verification of the underlying findings.
Payload schema: 1.0.0 · Engine: 2.0.0 · Scoring: heuristic-density-v1 · Scan date: 2026-10-31.
Can maintainers explain intent, assumptions, and trade-offs? 1115 finding(s).
Are imports declared and package/API names verified? 3 finding(s).
Are wrappers, guards, and repeated structures necessary? 12 finding(s).
Which changes need focused human review? 1129 finding(s).
This chart tracks the legacy adjusted pattern score across scan runs. Changes can reflect source changes, scan coverage, or detector/scoring versions; they do not measure AI adoption or prove that a team refactored code. Compare runs only when their scoring versions are compatible. The dashed line tracks the raw pattern-hit count.
Groups findings by configured severity weight. Severity is not a confidence estimate, security rating, or proof that a package is missing. Verify imports against the project's manifests and inspect each finding in its local design context.
This horizontal bar chart shows where configured pattern scores accumulate across directories. Treat the bars as locations to inspect, not as evidence of authorship or code quality. Use the file-level findings to decide whether a directory needs a focused ownership, dependency, complexity, or review check.
The scanner identified 1129 distinct pattern matches across 12 syntactic categories. Each entry below represents a discrete location in the source code where the engine recorded a detector match to inspect in context. Expand any category row to inspect the individual file paths, line numbers, code snippets, and the lexical context (CODE, COMMENT, or STRING) in which each match was detected.
Reading the findings table: The Severity column indicates the configured finding weight (CRITICAL / HIGH / MEDIUM / LOW). The Context column identifies whether the match occurred inside executable code, an inline comment, or a string literal — legacy comment-context matches receive a ×1.5 score weight. The ⚡ bolt icon marks clustered matches: three or more patterns within a 10-line window, each receiving an additional ×1.5 density multiplier under the legacy scoring model.
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | Staging/EnableGop/EnableGop.c | 121 | gDS->Hdr.CRC32 = 0; | COMMENT |
| LOW | Staging/AudioDxe/AudioDxe.h | 21 | * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE | COMMENT |
| LOW | Staging/AudioDxe/AudioDxe.h | 41 | #include <Library/UefiLib.h> | COMMENT |
| LOW | Staging/AudioDxe/HdaCodec/HdaCodecAudioIo.c | 641 | continue; | COMMENT |
| LOW | Staging/AudioDxe/HdaCodec/HdaCodecAudioIo.c | 661 | // | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.c | 1321 | // Gather open mode. | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 21 | * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 41 | // | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 61 | #define PCI_HDA_TCSEL_OFFSET 0x44 | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 81 | // | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 101 | #define HDA_STREAM_BUF_SIZE_HALF (HDA_STREAM_BUF_SIZE / 2) | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 121 | } HDA_DMA_POS_ENTRY; | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 141 | // TRUE once Index is valid. Required for tearing down streams safely if there is | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 161 | // | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 181 | // | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 221 | // | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 241 | // | COMMENT |
| LOW | Staging/AudioDxe/HdaController/HdaController.h | 281 | // commented out, but worked somewhat) of setting RINTCNT to 0xFF, in order to make | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_efi_edk2_base.h | 41 | */ | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_efi_edk2_base.h | 61 | # include <Protocol/DiskIo.h> | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_base.h | 61 | #if FSW_DEBUG_LEVEL >= 1 | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_efi_lib.c | 41 | // | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_core.h | 101 | #define fsw_u16_le_sip(var) | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_core.h | 121 | #define fsw_u32_be_sip(var) | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_core.h | 481 | * Posix-style modes. | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_core.h | 501 | COMMENT | |
| LOW | Staging/OpenHfsPlus/fsw_core.h | 521 | #define S_ISREG(m) (((m) & 0170000) == 0100000) /* regular file */ | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_efi_base.h | 61 | #define fsw_alloc(size, ptrptr) (((*(ptrptr) = AllocatePool(size)) == NULL) ? FSW_OUT_OF_MEMORY : FSW_SUCCESS) | COMMENT |
| LOW | Staging/OpenHfsPlus/fsw_hfsplus.h | 41 | #define kHFSPlusResourceFork 0xFF // resource fork type | COMMENT |
| LOW | Include/Apple/AppleMacEfi/AppleMacEfiSpec.h | 21 | // | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleProvisioning.h | 21 | // D1A26C1F-ABF5-4806-BB24-68D317E071D5 | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleIcon.h | 21 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 21 | // | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 41 | COMMENT | |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 301 | /// union. The strings are stored just after the load command structure and | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 321 | UINT32 HeaderAddress; ///< library's header address | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 341 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 361 | /// command constant. If the dynamic linker sees such a load command it it | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 381 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 401 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 421 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 441 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 461 | /// (all symbols are weak imported). | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 481 | /// local of code signature | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 501 | /// compressed dyld information | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 521 | /// compressed table of function start addresses | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 541 | /// Code signing DRs copied from linked dylibs | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 561 | /// build for Watch min OS version | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 601 | // | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 621 | /// segment are protected. | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 641 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 661 | COMMENT | |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 681 | /// section with only lazy symbol pointers | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 701 | /// zero fill on demand section (that can be larger than 4 gigabytes) | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 721 | // | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 741 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 761 | /// User setable attributes | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 781 | // than one attribute). | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 801 | /// | COMMENT |
| LOW | Include/Apple/IndustryStandard/AppleMachoImage.h | 821 | /// | COMMENT |
| 1030 more matches not shown… | ||||
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | AppleModels/update_generated.py | 24 | CODE | |
| LOW | AppleModels/update_generated.py | 57 | CODE | |
| LOW | AppleModels/update_generated.py | 86 | CODE | |
| LOW | AppleModels/update_products.py | 98 | CODE | |
| LOW | AppleModels/update_products.py | 203 | CODE | |
| LOW | AppleModels/update_products.py | 232 | CODE | |
| LOW | Utilities/macrecovery/macrecovery.py | 194 | CODE | |
| LOW | Utilities/macrecovery/macrecovery.py | 267 | CODE | |
| LOW | Utilities/macrecovery/macrecovery.py | 418 | CODE | |
| LOW | Debug/Scripts/common_uefi.py | 15 | CODE | |
| LOW | Debug/Scripts/common_uefi.py | 62 | CODE | |
| LOW | Debug/Scripts/lldb_uefi.py | 363 | CODE | |
| LOW | Debug/Scripts/lldb_uefi.py | 473 | CODE | |
| LOW | Debug/Scripts/gdb_uefi.py | 127 | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| HIGH | Debug/Scripts/common_uefi.py | 41 | except IndexError: | CODE |
| HIGH | Debug/Scripts/gdb_uefi.py | 141 | except gdb.MemoryError: | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | AppleModels/update_products.py | 122 | except Exception as e: | CODE |
| LOW | AppleModels/update_products.py | 184 | except Exception as e: | CODE |
| MEDIUM | AppleModels/update_products.py | 98 | def load_products(path='Products.zjson'): | CODE |
| LOW | Utilities/MacEfiUnpack/MacEfiUnpack.py | 104 | except Exception as e: | CODE |
| LOW | Utilities/macrecovery/macrecovery.py | 307 | except Exception as err: | CODE |
| LOW | Utilities/macrecovery/macrecovery.py | 314 | except Exception: | CODE |
| LOW | Utilities/macrecovery/macrecovery.py | 459 | except Exception as e: | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | Platform/OpenVariableRuntimeDxe/Variable.c | 2180 | // Step 1: | COMMENT |
| LOW | Platform/OpenVariableRuntimeDxe/Variable.c | 2197 | // Step 2: | COMMENT |
| LOW | Platform/OpenVariableRuntimeDxe/Variable.c | 2215 | // Step 3: | COMMENT |
| LOW | Platform/OpenVariableRuntimeDxe/Variable.c | 2232 | // Step 4: | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | Library/OcConsoleLib/GopUtils.c | 241 | // Definitely no need to set again if set in MTRR. | COMMENT |
| MEDIUM | Application/BootKicker/BootKicker.c | 58 | // firmware such as APTIO IV with less robust Stall implementations. | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | Debug/Scripts/x86_64_target_definition.py | 11 | # ---------------------------------------------------------------------- | COMMENT |
| MEDIUM | Debug/Scripts/x86_64_target_definition.py | 39 | # ---------------------------------------------------------------------- | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | Library/OcAppleKernelLib/KernelReader.c | 258 | // TODO: implement adler32 hash verification. | COMMENT |
| LOW | Debug/Scripts/gdb_uefi.py | 267 | # TODO: implement .elf handling. | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | Debug/Scripts/lldb_uefi.py | 16 | CODE | |
| LOW | Debug/Scripts/x86_64_target_definition.py | 44 | CODE | |
| LOW | Debug/Scripts/gdb_uefi.py | 33 | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | Debug/Scripts/x86_64_target_definition.py | 28 | # This file implements the x86_64 registers for the darwin version of | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | Library/OcPngLib/lodepng.c | 70 | lodepng source code. Don't forget to remove "static" if you copypaste them | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | Debug/Scripts/lldb_uefi.py | 91 | def get_child_member_with_name(self, value, field_name): | CODE |