Re: [PATCH 0/1] MR7879: dbghelp: Bail out when encountering managed symbols.

[eric pouech (＠epo)]

eric pouech (@epo) commented about dlls/dbghelp/msc.c:
> FIXME("Unexpected S_FRAMEPROC %d (%p %p) %x\n", top_frame_size, top_func, curr_func, i);
>              break;
>
> +        /* bail out when encountering managed debug symbols, in order to avoid asserts later */
> +        case S_TOKENREF:
> +        case S_GMANPROC:
> +        case S_LMANPROC:
> +            ERR("Unsupported mamaged symbol id %x, stopping pdb parsing\n", sym->generic.id);
> +            return FALSE;

the issue of returning FALSE is that it will also drop all codeview symbols following that entry.

isn't moving the 3 codeview symbols in the block of 'the symbols we can safely ignore for now' sufficient ?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7879#note_101631