Change in osmo-iuh[master]: tests/hnb-test: check return code of hnbap_encode_ueregisterrequesties

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Thu Jan 14 13:38:28 UTC 2021


laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-iuh/+/22171 )

Change subject: tests/hnb-test: check return code of hnbap_encode_ueregisterrequesties
......................................................................

tests/hnb-test: check return code of hnbap_encode_ueregisterrequesties

gcc warns about set but not used variable otherwise.

Change-Id: Ia49b7e88a66a4a744c2b14db6331cc0de431f60b
---
M src/tests/hnb-test.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/src/tests/hnb-test.c b/src/tests/hnb-test.c
index 6b9985f..4696e80 100644
--- a/src/tests/hnb-test.c
+++ b/src/tests/hnb-test.c
@@ -94,6 +94,7 @@
 
 	memset(&dereg, 0, sizeof(dereg));
 	rc = hnbap_encode_uede_registeries(&dereg, &dereg_ies);
+	OSMO_ASSERT(rc == 0);
 
 	msg = hnbap_generate_initiating_message(ProcedureCode_id_UEDe_Register,
 						Criticality_ignore,
@@ -129,6 +130,7 @@
 
 	memset(&request_out, 0, sizeof(request_out));
 	rc = hnbap_encode_ueregisterrequesties(&request_out, &request);
+	OSMO_ASSERT(rc == 0);
 
 	msg = hnbap_generate_initiating_message(ProcedureCode_id_UERegister,
 						Criticality_reject,

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22171
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: Ia49b7e88a66a4a744c2b14db6331cc0de431f60b
Gerrit-Change-Number: 22171
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210114/c740980e/attachment.htm>


More information about the gerrit-log mailing list