Fix: tests: fix unused-but-set warning in test_fd_tracker.c

[lttng-tools.git] / src / bin / lttng-sessiond / timer.c

diff --git a/src/bin/lttng-sessiond/timer.c b/src/bin/lttng-sessiond/timer.c
index c65df5c2dc132706b1a6335e0d73ecff10eabf00..4e522a7fc70e4770b30904cd0ddd01065f8c2bd9 100644 (file)
--- a/src/bin/lttng-sessiond/timer.c
+++ b/src/bin/lttng-sessiond/timer.c
@@ -1,23 +1,12 @@
 /*
- * Copyright (C) 2017 - Julien Desfossez <jdesfossez@efficios.com>
- * Copyright (C) 2018 - Jérémie Galarneau <jeremie.galarneau@efficios.com>
+ * Copyright (C) 2017 Julien Desfossez <jdesfossez@efficios.com>
+ * Copyright (C) 2018 Jérémie Galarneau <jeremie.galarneau@efficios.com>
  *
- * This program is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License, version 2 only, as
- * published by the Free Software Foundation.
+ * SPDX-License-Identifier: GPL-2.0-only
  *
- * This program is distributed in the hope that it will be useful, but WITHOUT
- * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
- * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
- * more details.
- *
- * You should have received a copy of the GNU General Public License along with
- * this program; if not, write to the Free Software Foundation, Inc., 51
- * Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
 #define _LGPL_SOURCE
-#include <assert.h>
 #include <inttypes.h>
 #include <signal.h>
 
@@ -33,7 +22,7 @@
 
 #define UINT_TO_PTR(value)                             \
        ({                                              \
-               assert(value <= UINTPTR_MAX);           \
+               LTTNG_ASSERT(value <= UINTPTR_MAX);             \
                (void *) (uintptr_t) value;             \
        })
 #define PTR_TO_UINT(ptr) ((uintptr_t) ptr)
@@ -154,7 +143,7 @@ int timer_start(timer_t *timer_id, struct ltt_session *session,
                unsigned int timer_interval_us, int signal, bool one_shot)
 {
        int ret = 0, delete_ret;
-       struct sigevent sev;
+       struct sigevent sev = {};
        struct itimerspec its;
 
        sev.sigev_notify = SIGEV_SIGNAL;
@@ -248,8 +237,8 @@ int timer_session_rotation_pending_check_stop(struct ltt_session *session)
 {
        int ret;
 
-       assert(session);
-       assert(session->rotation_pending_check_timer_enabled);
+       LTTNG_ASSERT(session);
+       LTTNG_ASSERT(session->rotation_pending_check_timer_enabled);
 
        DBG("Disabling session rotation pending check timer on session %" PRIu64,
                        session->id);
@@ -299,7 +288,7 @@ int timer_session_rotation_schedule_timer_stop(struct ltt_session *session)
 {
        int ret = 0;
 
-       assert(session);
+       LTTNG_ASSERT(session);
 
        if (!session->rotation_schedule_timer_enabled) {
                goto end;
@@ -356,7 +345,7 @@ void *thread_timer(void *data)
        rcu_register_thread();
        rcu_thread_online();
 
-       health_register(health_sessiond, HEALTH_SESSIOND_TYPE_TIMER);
+       health_register(the_health_sessiond, HEALTH_SESSIOND_TYPE_TIMER);
        health_code_update();
 
        /* Only self thread will receive signal mask. */
@@ -404,13 +393,13 @@ void *thread_timer(void *data)
                         * still fire.
                         */
                } else {
-                       ERR("Unexpected signal %d\n", info.si_signo);
+                       ERR("Unexpected signal %d", info.si_signo);
                }
        }
 
 end:
-       DBG("[timer-thread] Exit");
-       health_unregister(health_sessiond);
+       DBG("Thread exit");
+       health_unregister(the_health_sessiond);
        rcu_thread_offline();
        rcu_unregister_thread();
        return NULL;