Handle munmap() and add support for tracing JNI (native) calls.
The munmap() kernel calls are traced but the tracing code wasn't doing
anything with them. This caused the number of mapped regions in a process
to grow large in some cases and also caused symbol lookup errors in some
rare cases. This change also adds support for new trace record types
for supporting JNI (native) calls from Java into native code. This helps
with constructing a more accurate call stack.
diff --git a/emulator/qtools/hash_table.h b/emulator/qtools/hash_table.h
index 45786ec..4ea9ed5 100644
--- a/emulator/qtools/hash_table.h
+++ b/emulator/qtools/hash_table.h
@@ -21,6 +21,7 @@
typedef T value_type;
void Update(const char *key, T value);
+ bool Remove(const char *key);
T Find(const char *key);
entry_type* GetFirst();
entry_type* GetNext();
@@ -121,6 +122,31 @@
}
template<class T>
+bool HashTable<T>::Remove(const char *key)
+{
+ // Hash the key to get the table position
+ int len = strlen(key);
+ int pos = HashFunction(key) & mask_;
+
+ // Search the chain for a matching key and keep track of the previous
+ // element in the chain.
+ entry_type *prev = NULL;
+ for (entry_type *ptr = table_[pos]; ptr; prev = ptr, ptr = ptr->next) {
+ if (strcmp(ptr->key, key) == 0) {
+ if (prev == NULL) {
+ table_[pos] = ptr->next;
+ } else {
+ prev->next = ptr->next;
+ }
+ delete ptr->key;
+ delete ptr;
+ return true;
+ }
+ }
+ return false;
+}
+
+template<class T>
typename HashTable<T>::value_type HashTable<T>::Find(const char *key)
{
// Hash the key to get the table position