Changeset 92157 in webkit

Timestamp:

Aug 1, 2011 5:49:54 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

Use threadsafeCopy() API when using Database Directory path variable
​https://bugs.webkit.org/show_bug.cgi?id=65436

Patch by srikumar bonda <​srikumar.b@gmail.com> on 2011-08-01
Reviewed by David Levin.

Use threadsafeCopy() API while using m_databaseDirectoryPath string variable
when the databaseGuard lock hasn't been taken in DatabaseTracker class.
This is just a safety fix.

No changes to functionality so no new tests.

• storage/DatabaseTracker.cpp:

(WebCore::DatabaseTracker::trackerDatabasePath):
(WebCore::DatabaseTracker::deleteOrigin):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• storage/DatabaseTracker.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-08-01  srikumar bonda  <srikumar.b@gmail.com>
+
+        Use threadsafeCopy() API when using Database Directory path variable 
+        https://bugs.webkit.org/show_bug.cgi?id=65436
+
+        Reviewed by David Levin.
+
+        Use threadsafeCopy() API while using m_databaseDirectoryPath string variable
+        when the databaseGuard lock hasn't been taken in DatabaseTracker class. 
+        This is just a safety fix.
+
+        No changes to functionality so no new tests.
+
+        * storage/DatabaseTracker.cpp:
+        (WebCore::DatabaseTracker::trackerDatabasePath):
+        (WebCore::DatabaseTracker::deleteOrigin):
+
 2011-08-01  Michael Nordman  <michaeln@google.com>
 

trunk/Source/WebCore/storage/DatabaseTracker.cpp

@@ -103 +103 @@
 String DatabaseTracker::trackerDatabasePath() const
 {
-    return SQLiteFileSystem::appendDatabaseFileNameToPath(m_databaseDirectoryPath, "Databases.db");
+    return SQLiteFileSystem::appendDatabaseFileNameToPath(m_databaseDirectoryPath.threadsafeCopy(), "Databases.db");
 }