Attention is currently required from: fixeria, pespin.

View Change

1 comment:

• File doc/manuals/abis/rsl.adoc:

  • Patch Set #1, Line 1110: This information element requests the use of non-standard enhanced RTP

    @pespin@sysmocom.de: thank you for suggesting the idea of a Redmine ticket. […]

    @vyanitskiy@sysmocom.de: I just studied Osmocom TLV parser in libosmocore/src/gsm/tlv_parser.c, and I don't see where it treats unknown IEIs as having TLV format. Instead my reading of the code is that it stops on the first unknown IEI and returns OSMO_TLVP_ERR_UNKNOWN_TLV_TYPE - see function tlv_parse_one(), line 321 in that source file.

    With this observation, I see no advantage with defining new IEs as TLV rather than TV, hence we can use either format for the new IE under consideration here. I'll add this question to OS#6448, so we can continue the discussion there.

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