]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
If a _guessed_ compression method fails, it is never PROTOCOL_WARN.
authorNick Mathewson <nickm@torproject.org>
Tue, 20 Jun 2017 15:49:54 +0000 (11:49 -0400)
committerNick Mathewson <nickm@torproject.org>
Tue, 20 Jun 2017 16:08:11 +0000 (12:08 -0400)
Rationale: When use a guessed compression method, we already gave a
PROTOCOL_WARN when our guess differed from the declared method,
AND we gave a PROTOCOL_WARN when the declared method failed.  It is
not a protocol problem that the guessed method failed too; it's just
a recovery attempt that failed.

changes/bug22670 [new file with mode: 0644]
src/or/directory.c

diff --git a/changes/bug22670 b/changes/bug22670
new file mode 100644 (file)
index 0000000..4740327
--- /dev/null
@@ -0,0 +1,4 @@
+  o Minor bugfixes (logging, compression):
+    - When decompressing, do not warn if we fail to decompress using a
+      compression method that we merely guessed. Fixes part of
+      bug 22670; bugfix on 0.1.1.14-alpha.
index 1d2f8986aa7a8dc4037919ebcf73a04ed4cb3dc0..b6121d80d8b0ad48f74f6de1b2d213a6510d961f 100644 (file)
@@ -2271,7 +2271,7 @@ dir_client_decompress_response_body(char **bodyp, size_t *bodylenp,
   if (!new_body && tor_compress_supports_method(guessed) &&
       compression != guessed) {
     tor_uncompress(&new_body, &new_len, body, body_len, guessed,
-                   !allow_partial, LOG_PROTOCOL_WARN);
+                   !allow_partial, LOG_INFO);
     tried_both = 1;
   }
   /* If we're pretty sure that we have a compressed directory, and