summaryrefslogtreecommitdiff
path: root/lib/curl_ntlm_wb.c
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2014-12-10 22:43:44 +0100
committerDaniel Stenberg <daniel@haxx.se>2014-12-10 22:45:19 +0100
commitf4b5f8cdf0cd68be427c7093fde72f07012c36ca (patch)
tree719d927318fd2fd5b66d3128c18220e3bde4fad5 /lib/curl_ntlm_wb.c
parent8ff3bb50005a20ced2cffad26b5516820abad4c5 (diff)
downloadcurl-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
Diffstat (limited to 'lib/curl_ntlm_wb.c')
-rw-r--r--lib/curl_ntlm_wb.c5
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 &&