2009-07-17 Brent Fulgham <bfulgham@webkit.org>
authorbfulgham@webkit.org <bfulgham@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 Jul 2009 19:37:21 +0000 (19:37 +0000)
committerbfulgham@webkit.org <bfulgham@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 Jul 2009 19:37:21 +0000 (19:37 +0000)
        Reviewed by Adam Roben.

        Correct crash in WinLauncher due to improper mixing of BSTR
        and TCHAR types.
        https://bugs.webkit.org/show_bug.cgi?id=27381

        * WinLauncher/WinLauncher.cpp:
        (loadURL): Perform SysReAllocString to update the BSTR with
          the contents of the TCHAR string.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@46050 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitTools/ChangeLog
WebKitTools/WinLauncher/WinLauncher.cpp

index 3f658e7..10aaaa7 100644 (file)
@@ -1,3 +1,15 @@
+2009-07-17  Brent Fulgham  <bfulgham@webkit.org>
+
+        Reviewed by Adam Roben.
+
+        Correct crash in WinLauncher due to improper mixing of BSTR
+        and TCHAR types.
+        https://bugs.webkit.org/show_bug.cgi?id=27381
+
+        * WinLauncher/WinLauncher.cpp:
+        (loadURL): Perform SysReAllocString to update the BSTR with
+          the contents of the TCHAR string.
+
 2009-07-17  David Levin  <levin@chromium.org>
 
         Reviewed by Adam Treat.
index d07a66d..2dda619 100644 (file)
@@ -368,14 +368,14 @@ static void loadURL(BSTR urlBStr)
     IWebFrame* frame = 0;
     IWebMutableURLRequest* request = 0;
 
-
     static BSTR methodBStr = SysAllocString(TEXT("GET"));
 
     if (urlBStr && urlBStr[0] && (PathFileExists(urlBStr) || PathIsUNC(urlBStr))) {
         TCHAR fileURL[INTERNET_MAX_URL_LENGTH];
         DWORD fileURLLength = sizeof(fileURL)/sizeof(fileURL[0]);
+
         if (SUCCEEDED(UrlCreateFromPath(urlBStr, fileURL, &fileURLLength, 0)))
-            urlBStr = fileURL;
+            SysReAllocString(&urlBStr, fileURL);
     }
 
     HRESULT hr = gWebView->mainFrame(&frame);