mirror of
https://gitlab.torproject.org/tpo/core/tor.git
synced 2024-11-30 23:53:32 +01:00
Merge branch 'maint-0.3.5'
This commit is contained in:
commit
8fb318860e
7
changes/bug27841
Normal file
7
changes/bug27841
Normal file
@ -0,0 +1,7 @@
|
|||||||
|
o Minor bugfixes (onion services):
|
||||||
|
- On an intro point for a version 3 onion service, we do not close
|
||||||
|
an introduction circuit on an NACK. This lets the client decide
|
||||||
|
whether to reuse the circuit or discard it. Previously, we closed
|
||||||
|
intro circuits on NACKs. Fixes bug 27841; bugfix on 0.3.2.1-alpha.
|
||||||
|
Patch by Neel Chaunan
|
||||||
|
|
@ -503,12 +503,6 @@ handle_introduce1(or_circuit_t *client_circ, const uint8_t *request,
|
|||||||
/* Circuit has been closed on failure of transmission. */
|
/* Circuit has been closed on failure of transmission. */
|
||||||
goto done;
|
goto done;
|
||||||
}
|
}
|
||||||
if (status != HS_INTRO_ACK_STATUS_SUCCESS) {
|
|
||||||
/* We just sent a NACK that is a non success status code so close the
|
|
||||||
* circuit because it's not useful to keep it open. Remember, a client can
|
|
||||||
* only send one INTRODUCE1 cell on a circuit. */
|
|
||||||
circuit_mark_for_close(TO_CIRCUIT(client_circ), END_CIRC_REASON_INTERNAL);
|
|
||||||
}
|
|
||||||
done:
|
done:
|
||||||
trn_cell_introduce1_free(parsed_cell);
|
trn_cell_introduce1_free(parsed_cell);
|
||||||
return ret;
|
return ret;
|
||||||
|
Loading…
Reference in New Issue
Block a user