neels has submitted this change. (
https://gerrit.osmocom.org/c/osmo-msc/+/31864 )
Change subject: coverity: sdp_msg_test.c: check rc of sdp_msg_from_sdp_str()
......................................................................
coverity: sdp_msg_test.c: check rc of sdp_msg_from_sdp_str()
Related: CID#310975
Change-Id: I69d8972c58fb7cde965ff0b231947dd26247c2c7
---
M tests/sdp_msg/sdp_msg_test.c
1 file changed, 11 insertions(+), 1 deletion(-)
Approvals:
fixeria: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/tests/sdp_msg/sdp_msg_test.c b/tests/sdp_msg/sdp_msg_test.c
index a1ffac0..ebe8328 100644
--- a/tests/sdp_msg/sdp_msg_test.c
+++ b/tests/sdp_msg/sdp_msg_test.c
@@ -136,7 +136,7 @@
printf("\n[%d]\n", i);
dump_sdp(t->sdp_input, "sdp input: ");
- sdp_msg_from_sdp_str(&sdp, t->sdp_input);
+ OSMO_ASSERT(sdp_msg_from_sdp_str(&sdp, t->sdp_input) == 0);
sdp_msg_to_sdp_str_buf(str, sizeof(str), &sdp);
dump_sdp(str, "sdp_msg_to_sdp_str_buf: ");
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/31864
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I69d8972c58fb7cde965ff0b231947dd26247c2c7
Gerrit-Change-Number: 31864
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
