This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Neels Hofmeyr gerrit-no-reply at lists.osmocom.orgNeels Hofmeyr has submitted this change and it was merged. ( https://gerrit.osmocom.org/12265 ) Change subject: check_rtp: on IP:port errors, log the IP and port ...................................................................... check_rtp: on IP:port errors, log the IP and port Half of those are obviously zero, but I'd rather print the raw data instead of adding string constants, even if the condition must always lead to 0.0.0.0:0. Rationale: I had osmo-mgw listen on 0.0.0.0 and got the error message DRTP ERROR endpoint:0x1 destination IP-address is invalid which didn't convey that 0.0.0.0 is regarded as invalid. Change-Id: I9e98d464a27038904797c5c10735a98ef5b7b9c3 --- M src/libosmo-mgcp/mgcp_network.c 1 file changed, 6 insertions(+), 6 deletions(-) Approvals: Pau Espin Pedrol: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/libosmo-mgcp/mgcp_network.c b/src/libosmo-mgcp/mgcp_network.c index de34cc6..7af8e71 100644 --- a/src/libosmo-mgcp/mgcp_network.c +++ b/src/libosmo-mgcp/mgcp_network.c @@ -945,22 +945,22 @@ * and IP-address for outgoing data. */ if (strcmp(inet_ntoa(conn->end.addr), "0.0.0.0") == 0 && conn->end.rtp_port == 0) { LOGP(DRTP, LOGL_DEBUG, - "endpoint:0x%x destination IP-address and rtp port is (not yet) known\n", - ENDPOINT_NUMBER(endp)); + "endpoint:0x%x destination IP-address and rtp port is (not yet) known (%s:%u)\n", + ENDPOINT_NUMBER(endp), inet_ntoa(conn->end.addr), conn->end.rtp_port); return -1; } if (strcmp(inet_ntoa(conn->end.addr), "0.0.0.0") == 0) { LOGP(DRTP, LOGL_ERROR, - "endpoint:0x%x destination IP-address is invalid\n", - ENDPOINT_NUMBER(endp)); + "endpoint:0x%x destination IP-address is invalid (%s:%u)\n", + ENDPOINT_NUMBER(endp), inet_ntoa(conn->end.addr), conn->end.rtp_port); return -1; } if (conn->end.rtp_port == 0) { LOGP(DRTP, LOGL_ERROR, - "endpoint:0x%x destination rtp port is invalid\n", - ENDPOINT_NUMBER(endp)); + "endpoint:0x%x destination rtp port is invalid (%s:%u)\n", + ENDPOINT_NUMBER(endp), inet_ntoa(conn->end.addr), conn->end.rtp_port); return -1; } -- To view, visit https://gerrit.osmocom.org/12265 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9e98d464a27038904797c5c10735a98ef5b7b9c3 Gerrit-Change-Number: 12265 Gerrit-PatchSet: 3 Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de> Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181217/2454e10f/attachment.htm>