Exported symbols registered twice for mach-o binaries

[op2786]

Version and Platform (required):

• Binary Ninja Version: 3.6.4790-dev, 10b89271
• OS: macos
• OS Version: 14.3
• CPU Architecture: arm64

Bug Description:
Looks like exported symbols registered twice at the same address.

Steps To Reproduce:
Build the file given in the additional information section and check exports.

Screenshots:

Example 2 from binaryninja binary from version 4890-dev:

Additional Information:

#include <stdio.h>

int main() {
    printf("test\n");
    return 0;
}

Build with clang++ test.cpp on arm64 architecture.

[negasora]

This seems like a regression after #4109 was fixed

[bpotchik]

The regression is caused by the commit Parse Export Tries in Mach-Os. This commit adds the entire export trie as DataSymbols, even though other symbols for the export may have already been added.

There is a separate issue and commit impacting certain ELF files as well.

[bpotchik]

Fixed in 3.6.4791-dev

Note that this will be fixed when opening new files, however existing BNDBs will preserve the duplicate symbols.