summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--changes/bug249765
-rw-r--r--src/or/hs_cache.c4
2 files changed, 7 insertions, 2 deletions
diff --git a/changes/bug24976 b/changes/bug24976
new file mode 100644
index 0000000000..9c3be86eab
--- /dev/null
+++ b/changes/bug24976
@@ -0,0 +1,5 @@
+ o Minor bugfixes (hidden service v3 client):
+ - Remove a BUG() statement which can be triggered in normal circumstances
+ where a client fetches a descriptor that has a lower revision counter
+ than the one in its cache. This can happen due to HSDir desync. Fixes
+ bug 24976; bugfix on 0.3.2.1-alpha.
diff --git a/src/or/hs_cache.c b/src/or/hs_cache.c
index 3ebe13fb4d..6a5a3895b0 100644
--- a/src/or/hs_cache.c
+++ b/src/or/hs_cache.c
@@ -624,8 +624,8 @@ cache_store_as_client(hs_cache_client_descriptor_t *client_desc)
if (cache_entry != NULL) {
/* If we have an entry in our cache that has a revision counter greater
* than the one we just fetched, discard the one we fetched. */
- if (BUG(cache_entry->desc->plaintext_data.revision_counter >
- client_desc->desc->plaintext_data.revision_counter)) {
+ if (cache_entry->desc->plaintext_data.revision_counter >
+ client_desc->desc->plaintext_data.revision_counter) {
cache_client_desc_free(client_desc);
goto done;
}