Changeset 129598 in webkit

Timestamp:

Sep 25, 2012 11:55:27 PM (12 years ago)

Author:

Patrick Gansterer

Message:

[WIN] Fix ASSERT in BString after r128988.

• platform/win/BString.cpp:

(WebCore::BString::~BString):
(WebCore::BString::adoptBSTR):
(WebCore::BString::clear): Set pointer to 0 after freeing.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/win/BString.cpp (modified) (3 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-09-25  Patrick Gansterer  <paroga@webkit.org>
+
+        [WIN] Fix ASSERT in BString after r128988.
+
+        * platform/win/BString.cpp:
+        (WebCore::BString::~BString):
+        (WebCore::BString::adoptBSTR):
+        (WebCore::BString::clear): Set pointer to 0 after freeing.
+
 2012-09-25  Yoshifumi Inoue  <yosin@chromium.org>
 

trunk/Source/WebCore/platform/win/BString.cpp

@@ -107 +107 @@
 BString::~BString()
 {
-    clear();
+    SysFreeString(m_bstr);
 }
 
@@ -120 +120 @@
 void BString::adoptBSTR(BSTR bstr)
 {
-    clear();
+    SysFreeString(m_bstr);
     m_bstr = bstr;
 }
@@ -127 +127 @@
 {
     SysFreeString(m_bstr);
+    m_bstr = 0;
 }