Fix: relayd: Explicit null dereferenced

Coverity warns about an explicit null dereference on an early exit path.

Coverity report:
  CID 1405577 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)7.
  var_deref_model: Passing null pointer previous_stream_fd to
  stream_fd_put, which dereferences it.

Reported-by: Coverity (1405577) Explicit null dereferenced
Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/src/bin/lttng-relayd/stream-fd.c b/src/bin/lttng-relayd/stream-fd.c
index e376fa1684062ef681939a0a7827d61b3f88825c..f8c0c5343de9c93a22125cfeebc17d1a6e3d177d 100644 (file)
--- a/src/bin/lttng-relayd/stream-fd.c
+++ b/src/bin/lttng-relayd/stream-fd.c
@@ -53,5 +53,8 @@ static void stream_fd_release(struct urcu_ref *ref)
 
 void stream_fd_put(struct stream_fd *sf)
 {
+       if (!sf) {
+               return;
+       }
        urcu_ref_put(&sf->ref, stream_fd_release);
 }