Check if an entry in the TLS callback list is the terminator before trying to load it

[gabrielesvelto]

This fixes issue #412.

[m4b]

is it possible to add a test or update a test to make sure this doesn't regress?

[gabrielesvelto]

Sure, I'll try to add it.

[m4b]

let me know if you have a test case ready, or if you can modify an existing one that shows this change won't add any regressions, etc. thanks!

[gabrielesvelto]

let me know if you have a test case ready, or if you can modify an existing one that shows this change won't add any regressions, etc. thanks!

Yeah, I'll add a test here and provide a fix for issue #412 too later this year. I've been caught up in other stuff but I'll try to come back to this ASAP.

[m4b]

@gabrielesvelto no worries, just doing some triage this weekend/poking people :)

[m4b]

@gabrielesvelto thank you for this patch; it turns out this was re-discovered and fixed in another patch #425 fixing some recent tls stuff breaking on rust produced binaries, but thank you for your contribution and sorry we didn't merge it earlier :)

[gabrielesvelto]

np, glad it's fixed!