From 292cd6d2b715668c0ea5c9fc32e875824677935f Mon Sep 17 00:00:00 2001
From: Rafal Harabien <rafalh@reactos.org>
Date: Tue, 17 Jan 2012 17:59:26 +0000
Subject: [PATCH] [WININET] - Don't use shell32 functions when processing
 DLL_PROCESS_ATTACH because shell32.dll can be initialized later. Fixes
 shell32+wininet debug spam at boot.

svn path=/trunk/; revision=54992
---
 reactos/dll/win32/wininet/internet.c       |  2 -
 reactos/dll/win32/wininet/urlcache.c       | 10 ++++
 reactos/dll/win32/wininet/wininet_ros.diff | 53 ++++++++++++++++++++--
 3 files changed, 60 insertions(+), 5 deletions(-)

diff --git a/reactos/dll/win32/wininet/internet.c b/reactos/dll/win32/wininet/internet.c
index f473732c8f9..f461c260a1f 100644
--- a/reactos/dll/win32/wininet/internet.c
+++ b/reactos/dll/win32/wininet/internet.c
@@ -292,8 +292,6 @@ BOOL WINAPI DllMain (HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved)
 	    if (g_dwTlsErrIndex == TLS_OUT_OF_INDEXES)
 		return FALSE;
 
-            URLCacheContainers_CreateDefaults();
-
             WININET_hModule = hinstDLL;
 
         case DLL_THREAD_ATTACH:
diff --git a/reactos/dll/win32/wininet/urlcache.c b/reactos/dll/win32/wininet/urlcache.c
index 5d9034804c2..97e2a398639 100644
--- a/reactos/dll/win32/wininet/urlcache.c
+++ b/reactos/dll/win32/wininet/urlcache.c
@@ -182,6 +182,7 @@ typedef struct _URLCACHECONTAINER
 
 /* List of all containers available */
 static struct list UrlContainers = LIST_INIT(UrlContainers);
+BOOL bDefaultContainersAdded = FALSE;
 
 static DWORD URLCache_CreateHashTable(LPURLCACHE_HEADER pHeader, HASH_CACHEFILE_ENTRY *pPrevHash, HASH_CACHEFILE_ENTRY **ppHash);
 
@@ -591,6 +592,9 @@ static DWORD URLCacheContainers_FindContainerW(LPCWSTR lpwszUrl, URLCACHECONTAIN
     if(!lpwszUrl)
         return ERROR_INVALID_PARAMETER;
 
+    if (!bDefaultContainersAdded)
+        URLCacheContainers_CreateDefaults();
+
     LIST_FOR_EACH_ENTRY(pContainer, &UrlContainers, URLCACHECONTAINER, entry)
     {
         int prefix_len = strlenW(pContainer->cache_prefix);
@@ -629,6 +633,9 @@ static BOOL URLCacheContainers_Enum(LPCWSTR lpwszSearchPattern, DWORD dwIndex, U
     if (lpwszSearchPattern && dwIndex > 0)
         return FALSE;
 
+    if (!bDefaultContainersAdded)
+        URLCacheContainers_CreateDefaults();
+
     LIST_FOR_EACH_ENTRY(pContainer, &UrlContainers, URLCACHECONTAINER, entry)
     {
         if (lpwszSearchPattern)
@@ -1532,6 +1539,9 @@ BOOL WINAPI FreeUrlCacheSpaceW(LPCWSTR lpszCachePath, DWORD dwSize, DWORD dwSize
         return FALSE;
     }
 
+    if (!bDefaultContainersAdded)
+        URLCacheContainers_CreateDefaults();
+
     LIST_FOR_EACH_ENTRY(pContainer, &UrlContainers, URLCACHECONTAINER, entry)
     {
         /* The URL cache has prefix L"" (unlike Cookies and History) */
diff --git a/reactos/dll/win32/wininet/wininet_ros.diff b/reactos/dll/win32/wininet/wininet_ros.diff
index 7480e843f32..155029643df 100644
--- a/reactos/dll/win32/wininet/wininet_ros.diff
+++ b/reactos/dll/win32/wininet/wininet_ros.diff
@@ -40,7 +40,16 @@
   * NETCON_create
 --- wine-1.3.4/dlls/wininet/internet.c	2010-10-01 14:46:44.000000000 -0400
 +++ dll/win32/wininet/internet.c	2010-10-09 15:33:04.000000000 -0400
-@@ -3569,19 +3569,22 @@
+@@ -292,8 +292,6 @@
+ 	    if (g_dwTlsErrIndex == TLS_OUT_OF_INDEXES)
+ 		return FALSE;
+ 
+-            URLCacheContainers_CreateDefaults();
+-
+             WININET_hModule = hinstDLL;
+ 
+         case DLL_THREAD_ATTACH:
+@@ -3569,19 +3567,22 @@
  
  LPSTR INTERNET_GetNextLine(INT nSocket, LPDWORD dwLen)
  {
@@ -71,7 +80,15 @@ Index: dll/win32/wininet/urlcache.c
 ===================================================================
 --- dll/win32/wininet/urlcache.c	(revision 50814)
 +++ dll/win32/wininet/urlcache.c	(working copy)
-@@ -527,6 +527,7 @@
+@@ -182,6 +182,7 @@
+ 
+ /* List of all containers available */
+ static struct list UrlContainers = LIST_INIT(UrlContainers);
++BOOL bDefaultContainersAdded = FALSE;
+ 
+ static DWORD URLCache_CreateHashTable(LPURLCACHE_HEADER pHeader, HASH_CACHEFILE_ENTRY *pPrevHash, HASH_CACHEFILE_ENTRY **ppHash);
+ 
+@@ -527,6 +528,7 @@
      static const WCHAR HistoryPrefix[] = {'V','i','s','i','t','e','d',':',0};
      static const WCHAR CookieSuffix[] = {0};
      static const WCHAR CookiePrefix[] = {'C','o','o','k','i','e',':',0};
@@ -79,7 +96,7 @@ Index: dll/win32/wininet/urlcache.c
      static const struct
      {
          int nFolder; /* CSIDL_* constant */
-@@ -540,6 +541,12 @@
+@@ -540,6 +542,12 @@
      };
      DWORD i;
  
@@ -92,6 +109,36 @@ Index: dll/win32/wininet/urlcache.c
      for (i = 0; i < sizeof(DefaultContainerData) / sizeof(DefaultContainerData[0]); i++)
      {
          WCHAR wszCachePath[MAX_PATH];
+@@ -584,6 +592,9 @@
+     if(!lpwszUrl)
+         return ERROR_INVALID_PARAMETER;
+ 
++    if (!bDefaultContainersAdded)
++        URLCacheContainers_CreateDefaults();
++
+     LIST_FOR_EACH_ENTRY(pContainer, &UrlContainers, URLCACHECONTAINER, entry)
+     {
+         int prefix_len = strlenW(pContainer->cache_prefix);
+@@ -622,6 +633,9 @@
+     if (lpwszSearchPattern && dwIndex > 0)
+         return FALSE;
+ 
++    if (!bDefaultContainersAdded)
++        URLCacheContainers_CreateDefaults();
++
+     LIST_FOR_EACH_ENTRY(pContainer, &UrlContainers, URLCACHECONTAINER, entry)
+     {
+         if (lpwszSearchPattern)
+@@ -1525,6 +1539,9 @@
+         return FALSE;
+     }
+ 
++    if (!bDefaultContainersAdded)
++        URLCacheContainers_CreateDefaults();
++
+     LIST_FOR_EACH_ENTRY(pContainer, &UrlContainers, URLCACHECONTAINER, entry)
+     {
+         /* The URL cache has prefix L"" (unlike Cookies and History) */
 Index: dll/win32/wininet/http.c
 ===================================================================
 --- dll/win32/wininet/http.c	(revision 54234)