Fix: msg-iter.c: accept no packet total and content sizes

This changes an assertion in set_current_packet_content_sizes() to check
that either both the packet and content sizes are set (one can be set
from the other) or both are unset.

It's okay that both total and content sizes are unset: the data decoder
is already implemented to end the packet when the medium returns
`BT_MSG_ITER_MEDIUM_STATUS_EOF`.

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

diff --git a/plugins/ctf/common/msg-iter/msg-iter.c b/plugins/ctf/common/msg-iter/msg-iter.c
index b7a0b9fd47b8f2003cb3dd6cb9363e76195acef2..0d41638e988cfa88a02a6eee3434932214567e78 100644 (file)
--- a/plugins/ctf/common/msg-iter/msg-iter.c
+++ b/plugins/ctf/common/msg-iter/msg-iter.c
@@ -979,8 +979,10 @@ enum bt_msg_iter_status set_current_packet_content_sizes(
                }
        }
 
-       BT_ASSERT(notit->cur_exp_packet_total_size != -1);
-       BT_ASSERT(notit->cur_exp_packet_content_size != -1);
+       BT_ASSERT((notit->cur_exp_packet_total_size >= 0 &&
+               notit->cur_exp_packet_content_size >= 0) ||
+               (notit->cur_exp_packet_total_size < 0 &&
+               notit->cur_exp_packet_content_size < 0));
 
        if (notit->cur_exp_packet_content_size >
                        notit->cur_exp_packet_total_size) {