From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Tue, 15 Aug 2017 21:19:39 +0000 (-0400)
Subject: Fix: uninitialized return value on error
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=f01de2aee1992a782fe57109c100d4a36b79ebf2;p=deliverable%2Fbabeltrace.git

Fix: uninitialized return value on error

Found by Coverity:

CID 1379248 (#1 of 1): Uninitialized scalar variable (UNINIT).
uninit_use: Using uninitialized value ret.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/plugins/libctfcopytrace/clock-fields.c b/plugins/libctfcopytrace/clock-fields.c
index 9584eb7bb..5d312a811 100644
--- a/plugins/libctfcopytrace/clock-fields.c
+++ b/plugins/libctfcopytrace/clock-fields.c
@@ -599,6 +599,7 @@ int copy_find_clock_array_field(FILE *err,
 	for (i = 0; i < count; i++) {
 		entry_field = bt_ctf_field_array_get_field(field, i);
 		if (!entry_field) {
+			ret = -1;
 			BT_LOGE_STR("Failed to get array field.");
 			goto error;
 		}