3 ms·
Did some spot checking. Found a race condition in the dictionary code in less than five minutes of poking around. Ugh. Edit: code to add an entry to the dictio
by kabdib 11y ago
Did some spot checking. Found a race condition in the dictionary code in less than five minutes of poking around. Ugh.
Edit: code to add an entry to the dictionary releases its lock, whereupon you can wind up with duplicate NV pairs.
- djm_ 11y agoIt's open source on Github: make a pull request.
- haswell 11y agoThis would be a more constructive comment if you provided more details or perhaps submitted an issue in GitHub and shared it here. Why does "found a bug" equate to an immediate "ugh"? Is the expectation that projects submitted here are perfect?
- kabdib 11y agoI think it was more like the OP lead with his or her chin. "Done right? Okay..." It's also true that pretty much every piece of monitoring software I encounter makes me sad when I look under the hood. And the proprietary ones that I've seen under the hood of . . . hoo boy.
- ktsaou 11y agoHi, if you looked at dictionary.c, this is not used yet. But anyway, can you help me track it down?