Jacek Caban : wininet: Don' t use INTERNET_MAX_HOST_NAME_LENGTH in InternetGetSecurityInfoByURLW, .

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 72f5c3f2e711f0fe4ff6afa799e1424933d46d6e
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=72f5c3f2e711f0fe4ff6afa799e1424933d46d6e

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Wed Apr 27 19:19:47 2016 +0200

wininet: Don't use INTERNET_MAX_HOST_NAME_LENGTH in InternetGetSecurityInfoByURLW,.

Signed-off-by: Jacek Caban <jacek at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/wininet/internet.c | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c
index e82a6a4..6cf890b 100644
--- a/dlls/wininet/internet.c
+++ b/dlls/wininet/internet.c
@@ -4499,23 +4499,28 @@ BOOL WINAPI InternetGetSecurityInfoByURLA(LPSTR lpszURL, PCCERT_CHAIN_CONTEXT *p
  */
 BOOL WINAPI InternetGetSecurityInfoByURLW(LPCWSTR lpszURL, PCCERT_CHAIN_CONTEXT *ppCertChain, DWORD *pdwSecureFlags)
 {
-    WCHAR hostname[INTERNET_MAX_HOST_NAME_LENGTH];
     URL_COMPONENTSW url = {sizeof(url)};
     server_t *server;
-    BOOL res = FALSE;
+    WCHAR *hostname;
+    BOOL res;
 
     TRACE("(%s %p %p)\n", debugstr_w(lpszURL), ppCertChain, pdwSecureFlags);
 
-    url.lpszHostName = hostname;
-    url.dwHostNameLength = sizeof(hostname)/sizeof(WCHAR);
-
+    url.dwHostNameLength = 1;
     res = InternetCrackUrlW(lpszURL, 0, 0, &url);
     if(!res || url.nScheme != INTERNET_SCHEME_HTTPS) {
         SetLastError(ERROR_INTERNET_ITEM_NOT_FOUND);
         return FALSE;
     }
 
+    hostname = heap_strndupW(url.lpszHostName, url.dwHostNameLength);
+    if(!hostname) {
+        SetLastError(ERROR_OUTOFMEMORY);
+        return FALSE;
+    }
+
     server = get_server(hostname, url.nPort, TRUE, FALSE);
+    heap_free(hostname);
     if(!server) {
         SetLastError(ERROR_INTERNET_ITEM_NOT_FOUND);
         return FALSE;