diff options
| author | Nick Mathewson <nickm@torproject.org> | 2017-11-20 11:02:40 -0500 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2017-11-20 11:04:37 -0500 |
| commit | 3bb29dd707fbc825501c30ed9a6fe4aecdf4fd22 (patch) | |
| tree | 267dd9972514e75dc17c34839447036e93235c2a /changes | |
| parent | 73585595245ac9551a1982575941e954bb870ad0 (diff) | |
| download | tor-3bb29dd707fbc825501c30ed9a6fe4aecdf4fd22.tar.gz tor-3bb29dd707fbc825501c30ed9a6fe4aecdf4fd22.zip | |
Correctly handle partial success in consensus diff calculation.
Previously, if store_multiple() reported a partial success, we would
store all the handles it gave us as if they had succeeded. But it's
possible for the diff to be only partially successful -- for
example, if LZMA failed but the other compressors succeeded.
Fixes bug 24086; bugfix on 0.3.1.1-alpha.
Diffstat (limited to 'changes')
| -rw-r--r-- | changes/bug24086 | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/changes/bug24086 b/changes/bug24086 new file mode 100644 index 0000000000..2ae0b37e65 --- /dev/null +++ b/changes/bug24086 @@ -0,0 +1,7 @@ + o Minor bugfixes (directory cache): + - When a consensus diff calculation is only partially successful, only + record the successful parts as having succeeded. Partial success + can happen if (for example) one compression method fails but + the others succeed. Previously we misrecorded all the calculations as + having succeeded, which would later cause a nonfatal assertion failure. + Fixes bug 24086; bugfix on 0.3.1.1-alpha. |