Fix: potential use of uninitialized return value
[lttng-tools.git] / src / common / tracker.c
index ec908081cef84cd75a350cd5394f5672a0935262..0fdbd60f9eaacce380478be6b3dc9b68e77b3944 100644 (file)
@@ -1,18 +1,8 @@
 /*
- * Copyright (C) 2019 Jonathan Rajotte-Julien <jonathan.rajotte-julien@efficios.com>
+ * Copyright (C) 2019 Jonathan Rajotte-Julien <jonathan.rajotte-julien@efficios.com>
  *
- * This library is free software; you can redistribute it and/or modify it
- * under the terms of the GNU Lesser General Public License, version 2.1 only,
- * as published by the Free Software Foundation.
+ * SPDX-License-Identifier: LGPL-2.1-only
  *
- * This library 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 Lesser General Public License
- * for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with this library; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
 #include <assert.h>
@@ -277,13 +267,13 @@ const struct lttng_tracker_id *lttng_tracker_ids_get_at_index(
        return lttng_tracker_ids_get_pointer_of_index(ids, index);
 }
 
-int lttng_tracker_ids_get_count(const struct lttng_tracker_ids *ids, unsigned int *count)
+enum lttng_tracker_id_status lttng_tracker_ids_get_count(const struct lttng_tracker_ids *ids, unsigned int *count)
 {
 
-       enum lttng_tracker_id_status status = LTTNG_ROTATION_STATUS_OK;
+       enum lttng_tracker_id_status status = LTTNG_TRACKER_ID_STATUS_OK;
 
        if (!ids || !count) {
-               status = LTTNG_ROTATION_STATUS_INVALID;
+               status = LTTNG_TRACKER_ID_STATUS_INVALID;
                goto end;
        }
 
@@ -294,11 +284,13 @@ end:
 
 void lttng_tracker_ids_destroy(struct lttng_tracker_ids *ids)
 {
+       int i;
+
        if (!ids) {
                return;
        }
 
-       for (int i = 0; i < ids->count; i++) {
+       for (i = 0; i < ids->count; i++) {
                lttng_tracker_id_reset(&ids->id_array[i]);
        }
        free(ids->id_array);
@@ -308,12 +300,13 @@ void lttng_tracker_ids_destroy(struct lttng_tracker_ids *ids)
 int lttng_tracker_ids_serialize(const struct lttng_tracker_ids *ids,
                struct lttng_dynamic_buffer *buffer)
 {
-       int ret;
+       int ret = 0;
        int value;
        const char *string;
        unsigned int count;
        enum lttng_tracker_id_status status;
        const struct lttng_tracker_id *id;
+       unsigned int i;
 
        status = lttng_tracker_ids_get_count(ids, &count);
        if (status != LTTNG_TRACKER_ID_STATUS_OK) {
@@ -321,7 +314,7 @@ int lttng_tracker_ids_serialize(const struct lttng_tracker_ids *ids,
                goto error;
        }
 
-       for (unsigned int i = 0; i < count; i++) {
+       for (i = 0; i < count; i++) {
                struct lttcomm_tracker_id_header id_hdr;
                size_t var_data_len = 0;
 
This page took 0.0328850000000001 seconds and 4 git commands to generate.