Remove useless if() check
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Tue, 18 Jun 2013 19:33:05 +0000 (15:33 -0400)
committerDavid Goulet <dgoulet@efficios.com>
Wed, 19 Jun 2013 18:47:21 +0000 (14:47 -0400)
1022350 Dereference before null check
In ust_registry_session_destroy: All paths that lead to this null
pointer comparison already dereference the pointer earlier (CWE-476)

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
src/bin/lttng-sessiond/ust-registry.c

index 298ddbb6a9326c8355a98045d254de439e81785f..e22fc6fcfbfc93246172db98e398be6783b99285 100644 (file)
@@ -556,8 +556,6 @@ void ust_registry_session_destroy(struct ust_registry_session *reg)
        }
        rcu_read_unlock();
 
-       if (reg->channels) {
-               ht_cleanup_push(reg->channels);
-       }
+       ht_cleanup_push(reg->channels);
        free(reg->metadata);
 }
This page took 0.02606 seconds and 4 git commands to generate.