test_urcu_lfq: test for the proper pointer

We should use "if (qnode)" instead of "if (node)" in case of
the struct cds_lfq_node_rcu is not the first field of struct node.

Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/tests/test_urcu_lfq.c b/tests/test_urcu_lfq.c
index 39e0351abb27672f5d6677e436ebaeebc88567b3..1bbaf3cfd504a2abd0ead6e724f2b514c079dff5 100644 (file)
--- a/tests/test_urcu_lfq.c
+++ b/tests/test_urcu_lfq.c
@@ -243,14 +243,15 @@ void *thr_dequeuer(void *_count)
 
        for (;;) {
                struct cds_lfq_node_rcu *qnode;
-               struct test *node;
 
                rcu_read_lock();
                qnode = cds_lfq_dequeue_rcu(&q);
-               node = caa_container_of(qnode, struct test, list);
                rcu_read_unlock();
 
-               if (node) {
+               if (qnode) {
+                       struct test *node;
+
+                       node = caa_container_of(qnode, struct test, list);
                        call_rcu(&node->rcu, free_node_cb);
                        URCU_TLS(nr_successful_dequeues)++;
                }