neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/31713 )
Change subject: [NOT FOR MERGE] push to debug test error ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: there was a bug in bearer_cap_set_radio(), which failed to stop at the speech_ver=-1 end marker, and picked a different bcap->radio value depending on uninitialized/random memory. It was giving different results based on with or without -fsanitize: succeeded with -fsanitize and failed without. hence the initial build with sanitize succeeded, and 'make distcheck' without sanitize failed. the fix is https://gerrit.osmocom.org/c/osmo-msc/+/30114/3..4 (patch set 4 of 'add codec_mapping.h,.c') -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/31713 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I20a1f2d8ad4721eaf3dbc9c82ed9cbb5e5323bb0 Gerrit-Change-Number: 31713 Gerrit-PatchSet: 1 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Comment-Date: Tue, 07 Mar 2023 04:31:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
