diff options
| author | Mark Wielaard <mjw@redhat.com> | 2009-03-18 11:42:53 +0100 |
|---|---|---|
| committer | Mark Wielaard <mjw@redhat.com> | 2009-03-18 11:42:53 +0100 |
| commit | bc1a99dfcf9cd081ddf37f362dc1af2158c016b8 (patch) | |
| tree | 8dd076723876e507557a7416faaa6f1ae60e610a | |
| parent | 715bb1ea4a7823580a4f6641c5a03d66735b57a1 (diff) | |
| download | systemtap-steved-bc1a99dfcf9cd081ddf37f362dc1af2158c016b8.tar.gz systemtap-steved-bc1a99dfcf9cd081ddf37f362dc1af2158c016b8.tar.xz systemtap-steved-bc1a99dfcf9cd081ddf37f362dc1af2158c016b8.zip | |
Don't mutex_unlock in __stp_tf_get_vma_map_entry_internal.
All callers of __stp_tf_get_vma_map_entry_internal lock and unlock
the mutex correctly themselves in all cases. This unlock would trigger
a double unlock.
* runtime/task_finder_vma.c (__stp_tf_get_vma_map_entry_internal):
Don't mutex_unlock(&__stp_tf_vma_mutex).
| -rw-r--r-- | runtime/task_finder_vma.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/runtime/task_finder_vma.c b/runtime/task_finder_vma.c index 87a32fe5..83b206e5 100644 --- a/runtime/task_finder_vma.c +++ b/runtime/task_finder_vma.c @@ -203,7 +203,6 @@ __stp_tf_get_vma_map_entry_internal(struct task_struct *tsk, hlist_for_each_entry(entry, node, head, hlist) { if (tsk->pid == entry->pid && vm_start == entry->addr) { - mutex_unlock(&__stp_tf_vma_mutex); return entry; } } |