Remove TLS and make sdb thread-safe - Githubissues

radareorg / sdb

Simple and fast string based key-value database with support for arrays and json

https://www.radare.org/

MIT License

218 stars 62 forks source link

Remove TLS and make sdb thread-safe #244

Closed meme closed 2 years ago

meme commented 2 years ago

Checklist

[ ] Closing issues: #issue
[ ] Mark this if you consider it ready to merge
[ ] I've added tests (optional)
[ ] I wrote some documentation

Description

This makes sdb thread-safe without the need for TLS.

Nuke sdb_fmt
Mark goto tables as const
Mark lookup table as const
Move nextcas into SdbKv
Remove global hook (cannot be used without TLS, but we want to avoid that)
Fix lock path to not use a static buffer