pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29860 )
Change subject: mgw: Log unexpected RTP AMR OA-vs-BE payload ...................................................................... mgw: Log unexpected RTP AMR OA-vs-BE payload Change-Id: Ib5ae82c01153398491b21191a8cec9969337bbbc --- M src/libosmo-mgcp/mgcp_network.c 1 file changed, 6 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/60/29860/1 diff --git a/src/libosmo-mgcp/mgcp_network.c b/src/libosmo-mgcp/mgcp_network.c index 528764a..4ca4914 100644 --- a/src/libosmo-mgcp/mgcp_network.c +++ b/src/libosmo-mgcp/mgcp_network.c @@ -1535,8 +1535,13 @@ int oa = amr_oa_check((char*)msgb_data(msg), msgb_length(msg)); if (oa < 0) return -1; - if (((bool)oa) != conn_src->end.codec->param.amr_octet_aligned) + if (((bool)oa) != conn_src->end.codec->param.amr_octet_aligned) { + LOG_CONN_RTP(conn_src, LOGL_NOTICE, + "rx_rtp(%u bytes): Expected RTP AMR octet-aligned=%u but got octet-aligned=%u." + " Check your config!\n", + msgb_length(msg), conn_src->end.codec->param.amr_octet_aligned, oa); return -1; + } } /* Check if the origin of the RTP packet seems plausible */ -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/29860 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Ib5ae82c01153398491b21191a8cec9969337bbbc Gerrit-Change-Number: 29860 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pes...@sysmocom.de> Gerrit-MessageType: newchange