summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2018-01-31 12:51:45 -0500
committerNick Mathewson <nickm@torproject.org>2018-01-31 12:51:45 -0500
commit2294e330bde259f6b575f301258783ce2947bdc0 (patch)
treef526812bc11d2897f759d0630ddfe55124685682
parent1e81aaa62f8c7c4ac7dfd3c8b4182cc2d5368ab5 (diff)
parentdf312b3cf637d2c5487adc60ed541823bef56f06 (diff)
downloadtor-2294e330bde259f6b575f301258783ce2947bdc0.tar.gz
tor-2294e330bde259f6b575f301258783ce2947bdc0.zip
Merge branch 'maint-0.3.2'
-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 1a8fdbd03b..ed38a603b4 100644
--- a/src/or/hs_cache.c
+++ b/src/or/hs_cache.c
@@ -636,8 +636,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;
}