diff options
| author | lloyd <[email protected]> | 2011-07-05 20:45:23 +0000 |
|---|---|---|
| committer | lloyd <[email protected]> | 2011-07-05 20:45:23 +0000 |
| commit | 1dbbec7519d75f7bc5b8fa23eb175c8a8f96a9b5 (patch) | |
| tree | 4b2096bb10cd4de87fcd8aef7549e39cc1a0c95d /doc | |
| parent | b16bd4ad156f954dbcd4299e298765eececac19d (diff) | |
The Algorithm_Factory has this logic on looking for an object:
- Check the cache; if found, return value
- Populate cache, if the value is already there, delete the old
object and save the new one.
- Recheck the cache value
Raja <[email protected]> pointed out on the list that this could race
if multiple threads called a lookup function in close succession while
the cache was cold. All of them would fail the lookup, then each of
them would add it, but the values returned would be deleted by other
threads.
Instead, declare that first write wins. Then, the cache stays
consistent even if there is a race, the only issue is an extra search
and delete.
Modify GOST and Skein, as their name() function did not roundtrip
properly which caused failures otherwise.
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/log.txt | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/doc/log.txt b/doc/log.txt index 3deb98e12..ed8a08f21 100644 --- a/doc/log.txt +++ b/doc/log.txt @@ -10,6 +10,11 @@ Series 1.10 Version 1.10.1, Not Yet Released ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ +* A race condition in `Algorithm_Factory` could cause crashes in + multithreaded code. See `this thread on botan-devel + <http://lists.randombit.net/pipermail/botan-devel/2011-July/001455.html>`_ + for details and workarounds. + Version 1.10.0, 2011-06-20 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ |