summaryrefslogtreecommitdiff
path: root/src/core
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2018-09-18 15:13:10 -0400
committerNick Mathewson <nickm@torproject.org>2018-10-14 15:25:16 -0400
commitdddecee291cadf391d93b569023f1f1e008880e8 (patch)
treef74fbd03bd47533cbfcd5197fc1c402388f7e390 /src/core
parent370d9922a4e3cb24a5ff093cadc3098771cab957 (diff)
downloadtor-dddecee291cadf391d93b569023f1f1e008880e8.tar.gz
tor-dddecee291cadf391d93b569023f1f1e008880e8.zip
Initialize some locals in socks5 parsing code.
These confused GCC LTO, which thought they might be used uninitialized. I'm pretty sure that as long as 'res' indicates success, they will always be set to something, but let's unconfuse the compiler in any case.
Diffstat (limited to 'src/core')
-rw-r--r--src/core/proto/proto_socks.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/proto/proto_socks.c b/src/core/proto/proto_socks.c
index e2f233ad05..e23da7730b 100644
--- a/src/core/proto/proto_socks.c
+++ b/src/core/proto/proto_socks.c
@@ -744,7 +744,7 @@ handle_socks_message(const uint8_t *raw_data, size_t datalen,
res = SOCKS_RESULT_MORE_EXPECTED;
goto end;
} else if (req->socks_version != SOCKS_VER_5) {
- int have_user_pass, have_no_auth;
+ int have_user_pass=0, have_no_auth=0;
res = parse_socks5_methods_request(raw_data, req, datalen,
&have_user_pass,
&have_no_auth,