Changeset 27132 in webkit for trunk/JavaScriptCore

Timestamp:

Oct 26, 2007, 4:31:14 PM (18 years ago)

Author:

ggaren

Message:

Suggested by Darin Adler.

Use computedHash(), which is safer than just directly accessing _hash.

• kjs/lookup.cpp: (KJS::Lookup::findEntry): (KJS::Lookup::find):

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• kjs/lookup.cpp (modified) (2 diffs)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2007-10-26  Geoffrey Garen  <[email protected]>
+
+        Suggested by Darin Adler.
+        
+        Use computedHash(), which is safer than just directly accessing _hash.
+
+        * kjs/lookup.cpp:
+        (KJS::Lookup::findEntry):
+        (KJS::Lookup::find):
+
 2007-10-26  Geoffrey Garen  <[email protected]>
 

trunk/JavaScriptCore/kjs/lookup.cpp

@@ -60 +60 @@
 const HashEntry* Lookup::findEntry(const struct HashTable* table, const Identifier& s)
 {
-  return KJS::findEntry(table, s.ustring().rep()->_hash, s.data(), s.size());
+  return KJS::findEntry(table, s.ustring().rep()->computedHash(), s.data(), s.size());
 }
 
@@ -73 +73 @@
 int Lookup::find(const struct HashTable* table, const Identifier& s)
 {
-  const HashEntry* entry = KJS::findEntry(table, s.ustring().rep()->_hash, s.data(), s.size());
+  const HashEntry* entry = KJS::findEntry(table, s.ustring().rep()->computedHash(), s.data(), s.size());
   if (entry)
     return entry->value;