On Fri, 11 Jul 2014, Steve Holme wrote:

> 200 bytes is not enough; I currently see 516 bytes for an NTLMv2
> session auth with target_info included. I can't bring myself just to
> take the easy option and increase the buffer size. Instead, make it
> reallocate as needed instead of having a hard limit.

We currently use NTLM_BUFSIZE (as defined in curl_ntlm_msg.h) for the native
implementation - I'm not necessarily saying this is good practise but at
least it would be consistent and nice and quick (both in terms of a fix and
in performance) ;-)

I have quickly reviewed your patch and there are a few lines that don't
follow the curl coding standards - which I can fix up before I push.

However, I would be interested to hear other user's views on a hardcoded
buffer size versus having to read the input and reallocate the buffer if it
isn't long enough first.

Kind Regards

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