find_symbol fails on Mach-O

Attempting to run find_symbol(string) on a Mach-O binary results in the error:

File "cle/loader.py", line 473, in find_symbol
    if sym.is_import:
AttributeError: 'list' object has no attribute 'is_import'

This is because the Mach-O implementation of get_symbol always returns a list rather than a single symbol as expected by load_symbol.

Pip version cle 8.19.4.5

[fmagin]

Because I am probably the only person around here that feels somewhat responsible for the Mach-O backend, I will be looking into this. The current get_symbol implementation obviously violates assumptions made by other parts of CLE(a strong case for type annotations and checking). The question is of the rest of CLE should be changed to handle binaries with duplicate symbol names (which is allowed in Mach-O) or the backend should be changed so it always returns one symbol, however that is chosen. Namespacing would be one option for the latter, but that might require changes to CLE again to support the concept of namespaces instead of just building them as a string. Other option would be that only an index is valid as an argument for get_symbol and a str requires an additional parameter to specify a namespace or accept a list as a result.

[github-actions[bot]]

This issue has been closed due to inactivity.