[PATCH v2] gold: ensure file_counts_lock is initialized before using

Nick Gasson nick.gasson@arm.com
Thu Nov 5 09:59:09 GMT 2020


On 11/05/20 16:52 pm, Martin Liška wrote:
>
> Thank you for your patch.
>
> May I ask you, is it going to fix the following issue:
> https://sourceware.org/bugzilla/show_bug.cgi?id=26200
> ?
>

It's not directly related. I think H. J. Lu's patch on Bugzilla is
correct: I checked just now with -fsanitize=thread and can see two
threads concurrently accessing the current_ iterator in
Plugin_manager::all_symbols_read() and Plugin_manager::claim_file(). But
without my patch it would still have segfaulted, just in a different
place.

--
Thanks,
Nick


More information about the Binutils mailing list