eggheads / eggdrop

The Eggdrop IRC Bot
GNU General Public License v2.0
494 stars 88 forks source link

tdns: strerror() -> strerror_r() #1623

Open michaelortmann opened 2 weeks ago

michaelortmann commented 2 weeks ago

Found by: thommey and michaelortmann Patch by: michaelortmann Fixes:

One-line summary: Fix locking / race for tdns error logging

Additional description (if needed): Non main threads like tdns must use strerror_r() instead of strerror().

Test cases demonstrating functionality (if applicable):

valgrind --tool=helgrind ./eggdrop -nt BotA.conf
[...]
.tcl dnslookup google.de foo

Without this patch applied, valgrind will report race