pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/30222 )

Change subject: tests/osmux: Add extra asserts to validate osmux header is pulled correctly ......................................................................

tests/osmux: Add extra asserts to validate osmux header is pulled correctly

Change-Id: Ie2ccc22c53c33025c477236166ea97972c21ec3e --- M tests/osmux/osmux_input_test.c 1 file changed, 5 insertions(+), 0 deletions(-)

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve osmith: Looks good to me, approved

diff --git a/tests/osmux/osmux_input_test.c b/tests/osmux/osmux_input_test.c index 5d1b84e..df02423 100644 --- a/tests/osmux/osmux_input_test.c +++ b/tests/osmux/osmux_input_test.c @@ -264,6 +264,7 @@

/* We expect 1 batch: */ osmuxh = osmux_xfrm_output_pull(batch_msg); + OSMO_ASSERT(osmuxh); OSMO_ASSERT(osmuxh->ft == OSMUX_FT_VOICE_AMR); /* Check CMR and Q values are the ones from the last message: */ OSMO_ASSERT(osmuxh->amr_f == 0); @@ -357,6 +358,7 @@

/* We expect 1 batch: */ osmuxh = osmux_xfrm_output_pull(batch_msg); + OSMO_ASSERT(osmuxh); /* Check seqnum is the one configured beforehand: */ OSMO_ASSERT(osmuxh->seq == 123);

@@ -425,6 +427,7 @@

/* We expect 1 batch: */ osmuxh = osmux_xfrm_output_pull(batch_msg); + OSMO_ASSERT(osmuxh); /* Check seqnum is the one configured beforehand: */ OSMO_ASSERT(osmuxh->seq == 123); osmux_pl = (uint8_t *)osmuxh + sizeof(*osmuxh); @@ -503,6 +506,7 @@

/* We expect 1 batch: */ osmuxh = osmux_xfrm_output_pull(batch_msg); + OSMO_ASSERT(osmuxh); /* Check seqnum is the one configured beforehand: */ OSMO_ASSERT(osmuxh->seq == 123); /* Check four AMR payloads appear: */ @@ -587,6 +591,7 @@

/* We expect 1 batch: */ osmuxh = osmux_xfrm_output_pull(batch_msg); + OSMO_ASSERT(osmuxh); /* Check seqnum is the one configured beforehand: */ OSMO_ASSERT(osmuxh->seq == (123 + *osmux_transmitted - 1)); /* Check four AMR payloads appear: */

4 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.