Attention is currently required from: laforge, neels.

View Change

2 comments:

• File src/libosmo-mgcp/mgcp_network.c:

  • Patch Set #1, Line 838: if (mgcp_conn_rtp_is_iuup(conn) && !conn->iuup.configured) {

    Re the linked patch, I see only address and port related code in that patch, nothing about codecs no […]

    1) Yes, we used to always add the codec in CRCX in that scenario, but with my libosmo-mgcp-client in gerrit we didn't. I already changed again that patch to include the codec.

    2) Yes we can, but since we want to send a tentative remote address in the first CRCX, then we need to add the SDP instead.

    3) This is precisely how all this patchset started: we want to send a remote IP address on the RAN-side conn during first CRCX so that osmo-mgw's "ip probing" can potentially bind to the correct local IP address and provide it in CRCX ACK. osmo-hnbgw will use the remote Iuh IP address (HNB's Iuh IP addr) as a remote IuUP address in the first CRCX, which is usually correct. In the event it wasn't, during rx RAB-Ass-Resp it will send an MDCX to the MGW, and the MGW will update the address.

    We can ofc only guess the IP address, not the port. And even the IP address may be wrong, hence why we neeed to accept rx of IuUP Init from any source.

• File src/libosmo-mgcp/mgcp_network.c:

  • Patch Set #2, Line 837: != 0)

    (... […]

    in the resulting code execution yes, but this way also provides more info that errors may occur and how are they handled, eg AF_UNSPEC.

To view, visit change 35205. To unsubscribe, or for help writing mail filters, visit settings.