From: Julien Desfossez <jdesfossez@efficios.com>
Date: Wed, 14 Jun 2017 19:05:20 +0000 (-0400)
Subject: Fix: wrong return code check on try_set
X-Git-Url: http://git.efficios.com/?p=babeltrace.git;a=commitdiff_plain;h=7ee636dfd868803daeaee4edfb383057b7d51ec4

Fix: wrong return code check on try_set

try_set_structure_field_integer can return 0 if it did not perform any
action, it is not an error.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/lib/ctf-ir/stream.c b/lib/ctf-ir/stream.c
index 16286501..3d21ecac 100644
--- a/lib/ctf-ir/stream.c
+++ b/lib/ctf-ir/stream.c
@@ -852,7 +852,7 @@ struct bt_ctf_stream *bt_ctf_stream_create_with_id_no_check(
 			/* Initialize events_discarded */
 			ret = try_set_structure_field_integer(
 				stream->packet_context, "events_discarded", 0);
-			if (ret != 1) {
+			if (ret < 0) {
 				BT_LOGW("Cannot set `events_discarded` field in packet context: "
 					"ret=%d, packet-context-field-addr=%p",
 					ret, stream->packet_context);