Re: [PATCH] wininet: moved cookie loading from HttpOpenRequest() to HttpSendRequest()

[Hans Leidekker]

On Wednesday 02 July 2008 19:27:43 Yann Droneaud wrote:
> This patch does the following:
> - Move cookie retrieval code to HTTP_HttpSendRequest()
> - Don't allocate memory for url if INTERNET_FLAG_NO_COOKIES is enabled
> - Be sure to not overflow lpszCookies:
> buffer size was nCookieSize + 1 + 8,
> written size is 8 + (nCookieSize - 1) + 2 + 1
> - Optimized request header generation:
>  sprintfW() replaced by strcpyW()
>  strcatW() replaced by strcpyW()
>  removed call to strlenW() for HTTP_HttpAddRequestHeadersW()
> - Use HTTP_ADDREQ_FLAG_ADD instead of HTTP_ADDHDR_FLAG_ADD
> - Use HTTP_ADDREQ_FLAG_REPLACE to replace the cookie instead of adding

Thanks Yann, good to see you move on to fix more bugs. To be accepted
in git each of those items should be a separate patch though, and the
impact of moving the cookie code is such that it needs a test case.

-Hans