control: Fix wrong Action on failure to upload an HS descriptor

On failure to upload, the HS_DESC event would report "UPLOAD_FAILED" as the
Action but it should have reported "FAILED" according to the spec.

Fixes #24230

Signed-off-by: David Goulet <dgoulet@torproject.org>

diff --git a/changes/bug24230 b/changes/bug24230
new file mode 100644 (file)
index 0000000..b08c4cd
--- /dev/null
+++ b/changes/bug24230
@@ -0,0 +1,4 @@
+  o Minor bugfixes (control port, hidden service):
+    - Control port was reporting the action "UPLOAD_FAILED" instead of
+      "FAILED" for the HS_DESC event when a service was not able to upload a
+      descriptor. Fixes bug 24230; bugfix on 0.2.7.1-alpha.

diff --git a/src/or/control.c b/src/or/control.c
index 8173cb1e56e19feed7887dc5afd80658a149532c..202366aaecdc065e0272c137841144445e9d1cc6 100644 (file)
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -7284,7 +7284,7 @@ control_event_hs_descriptor_upload_failed(const char *id_digest,
              id_digest);
     return;
   }
-  control_event_hs_descriptor_upload_end("UPLOAD_FAILED", onion_address,
+  control_event_hs_descriptor_upload_end("FAILED", onion_address,
                                          id_digest, reason);
 }