diff options
| author | David Goulet <dgoulet@torproject.org> | 2023-01-12 10:52:35 -0500 |
|---|---|---|
| committer | David Goulet <dgoulet@torproject.org> | 2023-01-12 10:52:35 -0500 |
| commit | 0df4083299970674b4f399d3d85c6eab22cb8c7d (patch) | |
| tree | 08a55c71fc60445018a2ad2e42f3547ac8cd32bf | |
| parent | f2e9ce72d62c4057b32a524ad032b86dffa9a995 (diff) | |
| parent | 7b83e336ec4a1d137d6400f4d501c9a73835e34d (diff) | |
| download | tor-0df4083299970674b4f399d3d85c6eab22cb8c7d.tar.gz tor-0df4083299970674b4f399d3d85c6eab22cb8c7d.zip | |
Merge branch 'maint-0.4.5' into maint-0.4.7
| -rw-r--r-- | changes/ticket40730 | 5 | ||||
| -rw-r--r-- | src/core/proto/proto_socks.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/changes/ticket40730 b/changes/ticket40730 new file mode 100644 index 0000000000..f6d4c9de3b --- /dev/null +++ b/changes/ticket40730 @@ -0,0 +1,5 @@ + o Major bugfixes (TROVE-2022-002, client): + - The SafeSocks option had its logic inverted for SOCKS4 and SOCKS4a. It + would let the unsafe SOCKS4 pass but not the safe SOCKS4a one. This is + TROVE-2022-002 which was reported on Hackerone by "cojabo". Fixes bug + 40730; bugfix on 0.3.5.1-alpha. diff --git a/src/core/proto/proto_socks.c b/src/core/proto/proto_socks.c index 89745bd82c..07177c418a 100644 --- a/src/core/proto/proto_socks.c +++ b/src/core/proto/proto_socks.c @@ -233,7 +233,7 @@ static socks_result_t process_socks4_request(const socks_request_t *req, int is_socks4a, int log_sockstype, int safe_socks) { - if (is_socks4a && !addressmap_have_mapping(req->address, 0)) { + if (!is_socks4a && !addressmap_have_mapping(req->address, 0)) { log_unsafe_socks_warning(4, req->address, req->port, safe_socks); if (safe_socks) |