Hi Paul,
Am 24.10.26 um 10:55 AM schrieb Paul Richard Thomas:
Hello Harald,
Yes, I think that this one is 'obvious' and is certainly OK for
mainline. I would suggest that it be backported to 16- as well.
ok, will do.
Pushed to mainline r17-5533-g488509e6d4b243 so far.
Thanks for the review!
Harald
Thanks
Paul
On Fri, 23 Oct 2026 at 20:41, Harald Anlauf <[email protected]> wrote:
Dear all,
the attached, simple and obvious patch fixes the invalid read
detected with valgrind for compiling the testcase in the PR.
The problem was in the logic checking for the prefix "__", which
did not properly handle the case when the name to check was empty,
which caused the invalid read. However, looking twice at the logic
seems to reveal that it would handle the case of only one '_' in
the first two positions of the name different from what the comment
in the code said: only internal symbols with prefix "__" were meant.
So the fix should be obvious. Or do I miss something?
Regtested on x86_64-pc-linux-gnu. OK for mainline?
Thanks,
Harald
