pespin has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/osmo-mgw/+/14598 )
Change subject: mgw: Allow receiving lowercase MGCP header keyword ...................................................................... mgw: Allow receiving lowercase MGCP header keyword MGCP RFC3435 (https://tools.ietf.org/html/rfc3435) states almost all text has to be handled in a case-insensitive way, except SDP parts. Related: OS#4001 Change-Id: I7d1e55faddafa3c3093d38513d4a434ecf5ea5bd --- M src/libosmo-mgcp/mgcp_msg.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved osmith: Looks good to me, approved diff --git a/src/libosmo-mgcp/mgcp_msg.c b/src/libosmo-mgcp/mgcp_msg.c index 44589d1..3e95ed1 100644 --- a/src/libosmo-mgcp/mgcp_msg.c +++ b/src/libosmo-mgcp/mgcp_msg.c @@ -325,7 +325,7 @@ } break; case 2: - if (strcmp("MGCP", elem)) { + if (strcasecmp("MGCP", elem)) { LOGP(DLMGCP, LOGL_ERROR, "MGCP header parsing error\n"); return -510; -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/14598 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I7d1e55faddafa3c3093d38513d4a434ecf5ea5bd Gerrit-Change-Number: 14598 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: osmith <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-MessageType: merged
