Silence warnings on GCC 4.8 with -Wmaybe-uninitialized
authorMichael Jeanson <mjeanson@efficios.com>
Mon, 14 Jun 2021 15:18:19 +0000 (11:18 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 16 Jun 2021 18:37:59 +0000 (14:37 -0400)
We still build on SLES12 with GCC 4.8 in which '-Wmaybe-uninitialized'
doesn't seem to be the sharpest tool in the shed. Add explicit
initialization of 'ret' to silence the warnings.

Change-Id: I1f9de535b6be48357735af106ff555ab9eceb730
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
src/bin/lttng-sessiond/action-executor.c

index 8b8aba7618dc1836229bb2c597c0cb4658edab2e..14ac8103b1fe0c7738314c7ac962ca538234cb29 100644 (file)
@@ -748,7 +748,7 @@ static void *action_executor_thread(void *_data)
        DBG("Entering work execution loop");
        pthread_mutex_lock(&executor->work.lock);
        while (!executor->should_quit) {
        DBG("Entering work execution loop");
        pthread_mutex_lock(&executor->work.lock);
        while (!executor->should_quit) {
-               int ret;
+               int ret = 0;
                struct action_work_item *work_item;
 
                health_code_update();
                struct action_work_item *work_item;
 
                health_code_update();
@@ -983,7 +983,7 @@ error_unlock:
 static int add_action_to_subitem_array(struct lttng_action *action,
                struct lttng_dynamic_array *subitems)
 {
 static int add_action_to_subitem_array(struct lttng_action *action,
                struct lttng_dynamic_array *subitems)
 {
-       int ret;
+       int ret = 0;
        enum lttng_action_type type = lttng_action_get_type(action);
        const char *session_name = NULL;
        enum lttng_action_status status;
        enum lttng_action_type type = lttng_action_get_type(action);
        const char *session_name = NULL;
        enum lttng_action_status status;
This page took 0.034207 seconds and 4 git commands to generate.