diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2014-12-10 22:43:44 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2014-12-10 22:45:19 +0100 |
| commit | f4b5f8cdf0cd68be427c7093fde72f07012c36ca (patch) | |
| tree | 719d927318fd2fd5b66d3128c18220e3bde4fad5 | |
| parent | 8ff3bb50005a20ced2cffad26b5516820abad4c5 (diff) | |
| download | curl-f4b5f8cdf0cd68be427c7093fde72f07012c36ca.tar.gz | |
ntlm_wb_response: fix "statement not reached"
... and I could use a break instead of a goto to end the loop.
Bug: http://curl.haxx.se/mail/lib-2014-12/0089.html
Reported-by: Tor Arntsen
| -rw-r--r-- | lib/curl_ntlm_wb.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/curl_ntlm_wb.c b/lib/curl_ntlm_wb.c index b83326a9d..5e2d79cb4 100644 --- a/lib/curl_ntlm_wb.c +++ b/lib/curl_ntlm_wb.c @@ -294,7 +294,7 @@ static CURLcode ntlm_wb_response(struct connectdata *conn, len_out += size; if(buf[len_out - 1] == '\n') { buf[len_out - 1] = '\0'; - goto wrfinish; + break; } newbuf = realloc(buf, len_out + NTLM_BUFSIZE); if(!newbuf) { @@ -303,8 +303,7 @@ static CURLcode ntlm_wb_response(struct connectdata *conn, } buf = newbuf; } - goto done; -wrfinish: + /* Samba/winbind installed but not configured */ if(state == NTLMSTATE_TYPE1 && len_out == 3 && |