From 5a99579b1a903d3738bb82450ec40c52f1ea5594 Mon Sep 17 00:00:00 2001 From: Brian Potchik Date: Mon, 30 Mar 2026 08:50:13 -0400 Subject: This reverts commit 7228ba0b889765bc3474fbd5475870e24efc79ca. --- view/macho/universaltransform.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'view/macho/universaltransform.cpp') diff --git a/view/macho/universaltransform.cpp b/view/macho/universaltransform.cpp index 3309d853..33164215 100644 --- a/view/macho/universaltransform.cpp +++ b/view/macho/universaltransform.cpp @@ -266,19 +266,20 @@ bool UniversalTransform::DecodeWithContext(Ref context, const architectures.push_back(archName); } - if (!context->IsInteractive()) + // TODO: It is surprising that this is UniversalTransform's responsibility. + if (!BinaryNinja::IsUIEnabled()) { + // When headless, filter to the preferred architecture if one is configured. vector archPref = context->GetSettings()->Get>("files.universal.architecturePreference"); if (auto result = find_first_of(archPref.begin(), archPref.end(), architectures.begin(), architectures.end()); result != archPref.end()) { - // Filter to preferred architecture to support container auto-open policy size_t archIndex = find(architectures.begin(), architectures.end(), *result) - architectures.begin(); context->SetAvailableFiles({architectures[archIndex]}); return false; } - // Preserve original headless load behavior when no architecturePreference is specified - if (!BinaryNinja::IsUIEnabled() && archPref.empty() && architectures.size()) + // Load the first architecture if no preference is found. + if (archPref.empty() && architectures.size()) { context->SetAvailableFiles({architectures[0]}); return false; -- cgit v1.3.1