pespin submitted this change.

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved

gsm0808: Fix decoding of IE GSM0808_IE_LCS_CLIENT_TYPE

Change-Id: I7acafdefd105e330c466df2e17e884327651e348
---
M src/gsm/gsm0808.c
M tests/gsm0808/gsm0808_test.c
2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/gsm/gsm0808.c b/src/gsm/gsm0808.c
index 7013d8d..27ea47b 100644
--- a/src/gsm/gsm0808.c
+++ b/src/gsm/gsm0808.c
@@ -1525,6 +1525,7 @@
 		[GSM0808_IE_LOCATION_ESTIMATE]		= { TLV_TYPE_TLV },
 		[GSM0808_IE_POSITIONING_DATA]		= { TLV_TYPE_TLV },
 		[GSM0808_IE_LCS_CAUSE]			= { TLV_TYPE_TLV },
+		[GSM0808_IE_LCS_CLIENT_TYPE]		= { TLV_TYPE_TLV },
 		[GSM0808_IE_APDU]			= { TLV_TYPE_TLV },
 		[GSM0808_IE_NETWORK_ELEMENT_IDENTITY]	= { TLV_TYPE_TLV },
 		[GSM0808_IE_GPS_ASSISTANCE_DATA]	= { TLV_TYPE_TLV },
diff --git a/tests/gsm0808/gsm0808_test.c b/tests/gsm0808/gsm0808_test.c
index 38c68bf..158aeba 100644
--- a/tests/gsm0808/gsm0808_test.c
+++ b/tests/gsm0808/gsm0808_test.c
@@ -474,7 +474,7 @@
 	printf("Testing decoding Perform Location Report SYS#5891\n");
 
 	rc = tlv_parse(&tp, gsm0808_att_tlvdef(), hex+1, sizeof(hex)-1, 0, 0);
-	OSMO_ASSERT(rc < 0);
+	OSMO_ASSERT(rc == 5);
 }
 
 static void test_create_ass()

3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

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