[or-cvs] [tor/master 3/3] Rearrange _thread_test_func to please Coverity Scan

[nickm at torproject.org]

Author: Nick Mathewson <nickm at torproject.org>
Date: Mon, 25 Jan 2010 14:12:37 -0500
Subject: Rearrange _thread_test_func to please Coverity Scan
Commit: df3812ede144d7bc8dd15f84497b09f39c850157

I believe  that since  we were allocating  *cp while holding  a mutex,
coverity deduced that  *cp must be protected by  that mutex, and later
flipped out  when we didn't  use it  that way. If  this is so,  we can
solve our problems by moving the *cp = tor_strdup(buf) part outside of
the mutex-protected code.
---
 src/test/test_util.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/test/test_util.c b/src/test/test_util.c
index ba0f8cd..ad8d82b 100644
--- a/src/test/test_util.c
+++ b/src/test/test_util.c
@@ -420,11 +420,12 @@ _thread_test_func(void* _s)
     cp = &_thread2_name;
     count = &t2_count;
   }
-  tor_mutex_acquire(m);
 
   tor_snprintf(buf, sizeof(buf), "%lu", tor_get_thread_id());
   *cp = tor_strdup(buf);
 
+  tor_mutex_acquire(m);
+
   for (i=0; i<10000; ++i) {
     tor_mutex_acquire(_thread_test_mutex);
     strmap_set(_thread_test_strmap, "last to run", *cp);
-- 
1.6.5