Fix: volatile in assert()

From Coverity:
CID 1021642 (#1 of 3): Side effect in assertion
(ASSERT_SIDE_EFFECT)assert_side_effect: Argument test_array of assert()
has a side effect because the variable is volatile. The containing
function might work differently in a non-debug build.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/tests/benchmark/test_rwlock.c b/tests/benchmark/test_rwlock.c
index 6f52e5fc7ef65e2d11376230bbd78604ccb35af8..3fad9cf273bb6300143d2b6e147667aa1752accc 100644 (file)
--- a/tests/benchmark/test_rwlock.c
+++ b/tests/benchmark/test_rwlock.c
@@ -175,8 +175,11 @@ void *thr_reader(void *_count)
        }
 
        for (;;) {
        }
 
        for (;;) {

+               int a;
+

                pthread_rwlock_rdlock(&lock);
                pthread_rwlock_rdlock(&lock);

-               assert(test_array.a == 8);
+               a = test_array.a;
+               assert(a == 8);

                if (caa_unlikely(rduration))
                        loop_sleep(rduration);
                pthread_rwlock_unlock(&lock);
                if (caa_unlikely(rduration))
                        loop_sleep(rduration);
                pthread_rwlock_unlock(&lock);