[PATCH] wininet: Increase user size to 'UNLEN + 1' in create_cookie_url().

[Serge Gautherie]

Signed-off-by: Serge Gautherie <winehq-git_serge_180711(a)gautherie.fr>
---
 dlls/wininet/cookie.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/wininet/cookie.c b/dlls/wininet/cookie.c
index b5f779b..4d0967b 100644
--- a/dlls/wininet/cookie.c
+++ b/dlls/wininet/cookie.c
@@ -147,7 +147,7 @@ static cookie_domain_t *get_cookie_domain(substr_t domain, BOOL create)
 
 static WCHAR *create_cookie_url(substr_t domain, substr_t path, substr_t *ret_path)
 {
-    WCHAR user[UNLEN], *p, *url;
+    WCHAR user[UNLEN + 1], *p, *url;
     DWORD len, user_len, i;
 
     static const WCHAR cookie_prefix[] = {'C','o','o','k','i','e',':'};
-- 
2.10.0.windows.1

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=74599

Your paranoid android.


=== debiant (32 bit Chinese:China report) ===

wininet:
http.c:4947: Test failed: expected 1 pending read, got 2
http.c:5035: Test failed: expected 1 pending read, got 2

[Serge Gautherie]

https://testbot.winehq.org/JobDetails.pl?Key=74602
Not reproduced: flaky/unrelated?


On 01/07/2020 16:55, Marvin wrote:
> Full results can be found at:
> https://testbot.winehq.org/JobDetails.pl?Key=74599
>
> Your paranoid android.
>
> === debiant (32 bit Chinese:China report) ===
>
> wininet:
> http.c:4947: Test failed: expected 1 pending read, got 2
> http.c:5035: Test failed: expected 1 pending read, got 2

--