Skip to content

Demangling/symbol handling robustness - #45

Open
illwieckz wants to merge 2 commits into
masterfrom
illwieckz/demangling
Open

illwieckz wants to merge 2 commits into
masterfrom
illwieckz/demangling

Conversation

@illwieckz

Copy link
Copy Markdown
Member

Basically a continuation of:

But on the make check part

  • src/common/stabs_to_module_unittest: use POSIX regex on POSIX systems in STABS unit test
    The failure was observed on FreeBSD. Use POSIX-compatible regex syntax on
    POSIX systems while preserving compatibility with Windows' regex implementation.

  • src/common/stabs_to_module: check mangled names before demangling STABS symbols
    The failure was observed on FreeBSD, but checking whether a name is mangled
    before demangling it is a platform-independent robustness improvement.
    Actually a continuation and a reuse of: f591ce6

Picked from:

… in STABS unit test

The failure was observed on FreeBSD. Use POSIX-compatible regex syntax on
POSIX systems while preserving compatibility with Windows' regex implementation.
…BS symbols

The failure was observed on FreeBSD, but checking whether a name is mangled
before demangling it is a platform-independent robustness improvement.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant