Project

General

Profile

Actions

Bug #2904

closed

TLV parser repeated IE handling incompliant with 3GPP Specs

Added by laforge about 6 years ago. Updated about 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
libosmogsm
Target version:
-
Start date:
01/31/2018
Due date:
% Done:

0%

Spec Reference:

Description

TS 24.008 and TS 44.018 Section 8.6.3 "Repeated IEs" state that unless otherwise specified, if a T/TV/TLV IE is repeated several times inside a message, the first occurrence shall be used.

The libosmogsm implementation in tlv_parse() however overwrites the dec->lv[] array with every subsequent occurrence, so that we're using the last occurrence, rather than the first.

We should change the behavior, or (if needed, which I doubt) make it configurable

We should also test for this, either as unit test in libosmogsm, or as TTCN-3 test for BSC and/or MSC. Both somehow would make sense to me.

Actions #1

Updated by stsp about 6 years ago

This issue was fixed by https://gerrit.osmocom.org/#/c/6327/, and https://gerrit.osmocom.org/#/c/7322/ adds a regression test for this issue.

Actions #2

Updated by stsp about 6 years ago

  • Status changed from New to Resolved

Both changes have been merged. Closing this issue.

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)