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

Reply via email to