Commit f4b5f8cd authored by Daniel Stenberg's avatar Daniel Stenberg
Browse files

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
parent 8ff3bb50

lib/curl_ntlm_wb.c

+2 −3
Original line number Diff line number Diff line
@@ -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 &&