Attention is currently required from: neels, laforge, pespin, fixeria.

View Change

1 comment:

• File src/osmo-bsc-nat/bssap_conn.c:

  • Patch Set #4, Line 47: uint8_t tag = tag_order[i];

    See osmo-hnbgw, dexter did it there for RTP afair.

    I've looked at the osmo-hnbgw implementation, it's for ASN.1.

    I've updated the patch to decode each IE at a time, if it's the aoip transp layer addr, encode the new address, otherwise re-encode the original IE.

    Replacing the IE inside the buffer seems like a hack to me since to locate it correctly we already need to be able to parse the IEs coming before it with the tlv definition currently built against. And we can't do that reliably if there are unknown IEs and if they do not have TLV_TYPE_TLV but another format like TLV_TYPE_FIXED, TLV_TYPE_T, TLV_TYPE_TV etc. So we may end up decoding garbage and finding what looks like the aoip transp addr IE by chance.

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