From 7ee636dfd868803daeaee4edfb383057b7d51ec4 Mon Sep 17 00:00:00 2001 From: Julien Desfossez Date: Wed, 14 Jun 2017 15:05:20 -0400 Subject: [PATCH] Fix: wrong return code check on try_set MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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 Signed-off-by: Jérémie Galarneau --- lib/ctf-ir/stream.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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); -- 2.34.1