cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: patch http.c

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Fri, 26 Dec 2014 16:57:03 +0100 (CET)

On Wed, 24 Dec 2014, wrote:

> I found bug in http implementation with reuse cached connection. bug
> description: if easy options set using ntlm authentication, but server not
> support ntlm authentication, connection will not reused in next time, and
> will be created new connection. reason: Checking exists connection is set
> wantNTLM flag, but info in conn.ntlm.state is no set.

Thanks for this, but that's not actually what your patch fixes. :-/

First, url.c:ConnectionExists() is the function that picks or doesn't pick an
existing connection to re-use so if there's a problem like you describe, then
I really think that is the function that should be fixed. Don't you?

Secondly, your patch uses only a single variable to somehow deem if NTLM is
chosen or not only for the host, but we have both a host authentication and a
proxy one that both can and cannot use NTLM independently of each other.

Thirdly, please provide a test case or a way for us to see this problem so
that we can verify that it is actually fixed with whatever change we deem to
be necessary!

-- 
  / daniel.haxx.se

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2014-12-26