mirror of
https://github.com/morluto/rea.git
synced 2026-11-05 07:47:29 +00:00
[GH-ISSUE #3072] [Bug] Hopper provider on macOS: FAT Mach-O slice selection loads arm64e.x1, collapsing analysis of system binaries #1077
Closed
opened 2026-10-15 18:30:52 +00:00 by bsakaguchi
·
3 comments
No Branch/Tag specified
main
release-please--branches--main--components--rea-agents
release-please--branches--main--components--rea-agents--release-notes
codex/procedure-reference-stacked
codex/kuna-independent-retry
codex/byte-search-overlay-coverage
codex/open-failure-target-receipt
codex/kuna-xref-evidence
codex/ghidra-ambiguous-procedure-recovery
codex/function-source-bundle
codex/trace-function-integration-prototype
codex/snapshot-source-closure
codex/recorded-evidence-array-pages
codex/evidence-recovery-redesign
codex/trace-feature-address-scope
codex/call-discovery-scope
codex/ghidra-literal-candidate-evidence
codex/decompile-directory-independent-retry
codex/retry-provenance-base
codex/value-flow-capability-experiment
codex/ghidra-containing-instruction
cursor/readme-discord-button-ef9f
dependabot/github_actions/actions/setup-node-7.1.0
dependabot/github_actions/actions/upload-artifact-7.0.2
dependabot/npm_and_yarn/npm-minor-and-patch-d8d97ce6df
codex/kuna-1989
fix/8559-native-ui-fixture-ownership
fix/8555-go-verifier-mcp-ownership
codex/provider-capability-recovery
fix/8642-rizin-target-stdin
codex/ghidra-memory-effect-scope
codex/ghidra-annotation-entry-selector
codex/ghidra-readonly-recovery
codex/ghidra-regex-verifier-recovery
codex/evidence-discovery-view-guidance
fix/8545-bulk-append-overflow
fix/8428-navigation-refusal-alternatives
codex/address-context-recovery
codex/matching-member-limitations
codex/unknown-scope-contract
codex/evidence-view-guidance
feat/8373-native-trace-views
codex/ghidra-snapshot-verifier-binding
codex/elf-export-visibility
codex/evidence-view-recovery
codex/native-reference-cli-help
codex/provider-candidate-observation-scope
codex/kuna-session-verification
codex/ghidra-switch-lifetime-guidance
codex/pseudocode-paging-discovery
codex/clarify-evidence-view-support
codex/kuna-xrefs-selector-recovery
codex/context-capacity-recovery
codex/evidence-closure-large-links
codex/same-function-evidence-identity
codex/ghidra-verifier-inspected-jdk
codex/audit-guided-workflows
codex/crash-prompt-retained-dossiers
codex/analysis-view-error-recovery
codex/kuna-verifier-target-selection
codex/ghidra-selected-project-call-coverage
codex/docs-native-analysis-scope
codex/ghidra-java-socket-boundary
codex/ghidra-retained-snapshot-verifier
codex/ghidra-search-inventory-facts
fix/8284-zero-argument-parameter-bindings
fix/8198-ui-action-handler-offset
fix/8257-elf-entry-label-identity
fix/8304-deferred-output-schema-projection
fix/8277-call-path-computed-target-uncertainty
fix/pr-8089-conflicts
fix/ci-static-duplicate-export
dependabot/npm_and_yarn/lint-staged-17.6.0
dependabot/github_actions/actions/download-artifact-8.0.2
dependabot/npm_and_yarn/write-file-atomic-8.0.0
dependabot/npm_and_yarn/canonicalize-5.1.0
dependabot/github_actions/actions/cache/restore-6.1.0
dependabot/npm_and_yarn/toon-format/toon-4.4.0
dependabot/github_actions/actions/deploy-pages-5.0.1
codex/review-fixes-pr-2403
codex/review-fixes-pr-2400
codex/process-terminal-width
codex/test-brittleness-cleanup
verify/7167-browser-connect-macos
fix/7167-browser-connect
feat/native-matching-allowance-setting
feat/browser-scenario-dry-run
test/managed-caller-parameters-20261024
codex/parser-fortythird-boundaries
fix/6926-pinvoke-platform-decorations-20261023
fix/6889-har-body-encoding-20261023
fix/6888-sqlite-schema-budget-20261023
fix/dependency-security-gates-6708
cursor/evidence-overwrite-rebase-b4ac
feat/4057-verify-native-exports
fix/windows-acl-observer-overhead-20261022
cursor/erlang-beam-restack-4476
cursor/pe-certificate-review-bf32
fix/6116-javascript-application-fs-errors
fix/6061-unmapped-runtime-script-unknown
feat/browser-scenario-summary
fix/pe-certificates
codex/verify-3407
cursor/rebase-5699
cursor/rebase-5352
cursor/rebase-5600
cursor/fix-cli-schema-omission-10a8
codex/audit-hopper-shutdown-reasons-20261021
fix/demangle-swift-omitted-record
fix/nib-boolean-value-types
cursor/cancellation-partial-observation-a5ac
fix/ib-object-limit-reasons
fix/electron-fuse-limit-advice
fix/kuna-doctor-dangling-cache
fix/fixed-budget-refusal-advice
fix/js-ipc-listener-dedup
fix/js-return-member-provenance
fix/dmg-integrity-retry-advice
fix/ida-doctor-readiness
fix/kuna-version-probe-output-drain
fix/legacy-swift-runtime-name-matching
fix/dylib-lazy-load-commands
fix/browser-scenario-startup-cancel
fix/swift-nested-punycode-classification
fix/compare-unselected-scopes
fix/objc-method-budget-coverage
fix/dmg-mounted-root-observations
audit/ci-tests-20261019-round2
release/6.7.0
fix/wabt-cleanup-retry-resolution
fix/electron-fuse-elf-pe-architecture
fix/native-plist-descriptor-ownership
fix/kuna-caller-cleanup-ownership
experiment/atomic-codemode-20261016
review/first-contributor-3776
fix/call-resolution-full-read-cycles
fix/member-mutation-selection-repeat
experiment/atomic-re-tools-20261015
perf/member-mutation-lazy-origins
perf/static-network-mutation-index
fix/hopper-startup-lifecycle
audit/3050-windows-proof-20261015
feat/windows-window-capture-2183
audit/3049-windows-proof-20261015
audit/3735-macos-fixtures-20261015
fix/batch-decompile-null-pseudocode
fix/kuna-inventory-honors-jobs
experiment/browser-codemode-tool-design-20261015
fix/hopper-clean-runtime-argv
codex/review-2049-27f1b367
review-fixes/pr-2787-routing
fix/3360-scoped-this-emitters
fix/3367-replaced-global-effects
fix/3838-compact-any-value-items
test/hopper-download-probe-20261015
codex/windows-runtime-creation-red-green-20261015
codex/windows-readonly-root-red-green-20261015
codex/windows-root-share-red-green-20261015
codex/exp-1685-win-mcp-startup
codex/bug-sweep-20261015
investigate/2190-windows-package-phases
investigate/3543-macos-cli-deadline
perf/3281-reverse-tools-feedback-quality
fix/3406-hopper-doctor-fixture
fix/3341-wrapper-require-test-context
fix/3301-owned-uninstall-fixtures
fix/3164-compiler-input-ownership
fix/pcap-cleanup-stdout-newline-20261014
perf/interface-builder-summary
codex/review-fixes-pr-1977
fix/ghidra-effective-jvm-platform
fix/docs-turkish-rust-link-20261013
fix/main-maintenance-ci-20261010
fix/recover-binary-session-process-effects
codex/review-fixes-pr-2457
codex/ci-required-cancellation-gate
fix/main-format-baseline-20261013
fix/setup-preserve-entry-fields
cursor/restore-main-checks-1ffc
fix/ghidra-bridge-hashmap-import
codex/shared-ci-jadx-json-default-20261013
codex/shared-ci-ghidra-test-groups-20261013
codex/shared-ci-java-fixture-format-20261013
codex/fix-intrinsic-invocation-boundaries
test/release-preflight-reusable-jobs
test/js-parameter-provenance-expectation
fix/linux-hopper-verification-current
test/2419-ghidra-elf-probe-size
followup/pr-1544-native-ida
codex/fold-pr-2230-contribution
docs/readme-responsible-use
fix/issue-2064
fix/issue-2050
fix/issue-2053
fix/issue-1968
fix/issue-1945
experiment/codemode-bridge-20261011
fix/tool-effects-negative-wording-1904-20261012
fix/1842-electron-interop-provenance
feat/recover-javascript-multi-input
refactor/website-i18n-prep
fix/nativeaot-pe-virtual-extent
fix/browser-verifier-ci-hang
fix/pe-resources-input-selection
docs/beginner-friendly-guides
docs/beginner-setup-guide
fix/javascript-analysis-worker-1563
fix/1649-ghidra-duplicate-names
fix/javascript-export-keys-integration-1369
test/maintenance-boundary-contracts-20261011
review/javascript-export-keys-1369
review/process-live-progress-1483
test/javascript-1495-growth-ratio
fix/ci-escape-timing
docs/normalized-terminal-geometry-1450
release/6.3.0
archive/release-6.3.0-pr1381
release/6.2.0
fix/electron-preload-dynamic-override
fix/javascript-self-referential-bundler-edges
release/6.1.0
cursor/grok-client-cherry-pick-0b56
feat/signature-posture
feat/dyld-shared-cache
feat/artifact-integrity-policy
feat/pkg-inventory
release/6.0.0
chore/sync-release-5.0.0
release/5.0.0
cursor/cloud-agent-node-787e
fix/javascript-xhr-open-endpoints
review/pr-711-realpath-fallback-wording
cleanup/replay-confinement
codex/test-suite-performance
codex/issue-foundations
codex/address-open-issues-2026-07-20
agent/javascript-vite-rollup-esbuild-adapters
agent/process-cleanup
agent/triage-integration
agent/unknown-commands-registry
feat/mcp-identity-metadata
morlvps/issue_cleanup
codex/installer-redesign
morl/roadmap-investigation-core
morl/roadmap-investigation-docs
release-please--branches--main--components--rea
rea-agents-6.8.0
rea-agents-6.7.0
rea-agents-6.6.0
experiment/atomic-codemode-wait-runtime-20261016
rea-agents-6.5.0
rea-agents-6.4.0
rea-agents-6.3.0
rea-agents-6.2.0
rea-agents-6.1.0
rea-agents-6.0.0
rea-agents-5.0.0
rea-agents-4.1.0
rea-agents-4.0.1
rea-agents-4.0.0
rea-agents-3.2.1
rea-agents-3.2.0
rea-agents-3.1.0
rea-agents-3.0.0
rea-agents-2.7.0
rea-agents-2.6.0
rea-agents-2.5.1
rea-agents-2.5.0
rea-agents-2.4.0
rea-agents-2.3.0
rea-agents-2.2.0
rea-agents-2.1.0
rea-agents-2.0.0
rea-agents-1.7.0
rea-agents-1.6.0
rea-agents-1.5.0
rea-agents-1.4.0
rea-agents-1.3.0
rea-agents-1.2.0
rea-agents-1.1.0
rea-agents-1.0.0
rea-agents-0.5.0
rea-agents-0.4.0
rea-agents-0.3.0
rea-0.2.1
rea-0.2.0
Labels
Clear labels
area: bridge
area: bridge
area: build
area: build
area: build
area: cli
area: domain
area: evidence
area: mcp
area: native
area: providers
area: runtime
area: server
area: setup
area: tests
bug
design
documentation
enhancement
enhancement
enhancement
experiment
help wanted
i18n
invalid
P0: critical
P1: high
P2: medium
P2: medium
P2: medium
P3: low
performance
provider-request
pull-request
Mirrored from GitHub Pull Request
question
roadmap
triage
No labels
area: bridge
area: bridge
area: build
area: build
area: build
area: cli
area: domain
area: evidence
area: mcp
area: native
area: providers
area: runtime
area: server
area: setup
area: tests
bug
design
documentation
enhancement
enhancement
enhancement
experiment
help wanted
i18n
invalid
P0: critical
P1: high
P2: medium
P2: medium
P2: medium
P3: low
performance
provider-request
pull-request
question
roadmap
triage
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set.
Reference
morluto/rea#1077
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @XYishion on GitHub (Oct 14, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/3072
Summary
On Apple-silicon macOS, system binaries (
/bin/*,/usr/bin/*) are FAT Mach-O files with threeslices:
x86_64,arm64e(cpusubtype 2) andarm64e.x1(cpusubtype 12).The Hopper provider is launched as
hopper -l FAT --aarch64 -l Mach-O --analysis ..., and thebridge ends up loading the
arm64e.x1slice. That slice's entry point begins withpacibsppc(
0xdac1a7fe) — an arm64e.x1-only encoding that is an invalid instruction under plain arm64(whereas the same offset in the
arm64eslice holdspacibsp/0xd503237f, a decodableHINT-class NOP). Hopper's recursive descent therefore aborts at the first instruction of every
function, and almost no code procedures are created.
The session then fails closed with
No procedure exists at the requested address, which is correctbehaviour on REA's side — but the underlying slice choice makes the target effectively unanalyzable.
Environment
rea-agents6.4.0 (vianpxand via pinnedrea.mjs)dad6f886261fad0d4cdd01aee6fcf57439c60d7b5c54e62fc97f0180116690e0rea inspect/rea function)Reproduction
Open
/bin/echowith REA (MCPopen_binary+binary_overview/list_procedures, orrea inspect /bin/echo):binary_overview→procedure_count: 5, all of them import stubs; nosub_*proceduresanalyze_function { "procedure": "0x1000004f0" }→Hopper analyze_function failed (-32000, invalid_request): No procedure exists at the requested addresstrace_feature "POSIXLY_CORRECT"→ string found at0x100000913, but no incoming references(same for every other string in
main)The loaded bytes are the
arm64e.x1sliceREA's bytes match the
arm64e.x1slice exactly, and do not appear anywhere in thearm64eslice. Note that REA reports
architecture: "arm64"andavailable_architectures: ["arm64"]forthe FAT file — the
arm64e/arm64e.x1distinction is not surfaced.The two slices differ by only ~380 bytes overall (the cpusubtype byte,
LC_UUID, and scattered PACencodings at function entries and returns) — but the entry encodings decide whether analysis works
at all.
Scope — not specific to
/bin/echoUsing
dyld_info -function_startsas ground truth:/bin/echo0x4F0,0x7C4,0x7F8)/bin/pwd0x588,0x74C)sub_10000058c= entry +4)/bin/ln0x5E8/bin/catThe
+4offset is the signature of the failure: the first instruction is undecodable, so Hopperonly starts a function at the following instruction.
Control — the toolchain itself is fine
/opt/homebrew/bin/bomtoolis a thin arm64 binary with symbol names. REA/Hopper returns all14/14
LC_FUNCTION_STARTSentries at exactly the ground-truth addresses (_mainat0x100000e70…_parse_long_optionsat0x100002408). Its loader arguments are["-l","Mach-O","--aarch64"]withmacho_container: "thin", versus["-l","FAT","--aarch64","-l","Mach-O"]with
macho_container: "fat32"for the failing cases.So the problem is specific to FAT + arm64e.x1 + stripped, i.e. Apple's own system binaries.
Workaround (verified)
Thin the file to the normal
arm64eslice before handing it to REA:Results on
/bin/echo:lipo -thin arm64eprocedure_countread_bytes 0x1000004f0fea7c1da…(pacibsppc)7f2303d5…(pacibsp)analyze_function 0x1000004f0EntryPoint+ real pseudocodetrace_feature "POSIXLY_CORRECT"0x100000520to string0x100000913/bin/pwdlikewise goes 10 → 11 procedures, with the function boundary corrected to0x588(instead of
sub_10000058cat entry+4).Suspected cause / suggested fix
When resolving a FAT Mach-O for
--aarch64, thearm64e.x1slice (cpusubtype 12) is selectedrather than the
arm64eslice (cpusubtype 2). Whether the selection happens inside REA or insideHopper's
-l FATloader, the outcome is a slice whose entry encodings plain arm64 cannot decode.Suggestions, in order of preference:
arm64e) over cpusubtype 12(
arm64e.x1) during FAT slice selection.architecture: "arm64"currently hides a choice that decides whether analysis succeeds.Happy to test a patch — this is reproducible in one command on any Apple-silicon macOS with Hopper
installed.
@akushonkamen commented on GitHub (Oct 14, 2026):
Claiming this for a fix.
Scope I am taking: extend REA's existing FAT64 thin-slice preparation path (
resolveHopperMachOImage/prepareHopperMachOImage) to FAT32 containers, so an arm64 target on a FAT Mach-O resolves a verified thin image of the cpusubtype-2 (arm64e) slice — preferring cpusubtype 2 over 12 (arm64e.x1) when both are present, per suggestion 1 — and opens it through the already-verified-l Mach-O --aarch64thin-image route instead of Hopper's implicit-l FATslice choice (the explicit-selection intent of suggestion 2). The selected slice'scpu_type/cpu_subtypestay surfaced in the prepared-image profile per suggestion 3.Out of scope, so nobody duplicates: changes to Hopper itself, new user-facing slice-selector flags, Ghidra or other-provider behavior, and the FAT64 ambiguity/fail-closed policy beyond reusing it for the shared selector. Unit-level regression tests will use synthesized FAT32 fixtures, so no Hopper install is needed to verify slice selection; real-Hopper end-to-end stays with the reporter's verified workaround as the reference.
PR to follow on top of current main.
@akushonkamen commented on GitHub (Oct 14, 2026):
Opened PR #3092 with the fix: the existing FAT64 thin-slice preparation now covers FAT32 containers, and arm64 targets prefer cpusubtype 0/2 over
arm64e.x1(x1 only as sole candidate; other ambiguity stays fail-closed), loading through the verified-l Mach-O --aarch64thin route. 11 new FAT32 regressions fail on pre-fix code and pass after (plus 143 related tests green); the real-Hopper FAT32 lane (verify:hopper:fat) still needs a run where Hopper is installed.@morluto commented on GitHub (Oct 14, 2026):
Resolved on main by
cf806ea02.FAT slice preparation now selects and verifies a deterministic compatible thin image. Current tests exercise slice/profile production; no fresh Hopper analysis of the system binary is claimed.
Rechecked on main
b8510ee00in the current focused run: 17 test files passed, 485 tests passed and one platform-dependent test skipped. Relevant regression coverage: src/hopper/HopperMachOImage.test.ts and src/hopper/HopperAnalysisProfile.test.ts.