[PATCH] openbsc[master]: LLC: Fixup elemnt order in LLC-XID

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

dexter gerrit-no-reply at lists.osmocom.org
Mon Dec 5 11:48:51 UTC 2016


Review at  https://gerrit.osmocom.org/1359

LLC: Fixup elemnt order in LLC-XID

When the LLC-XID request is constructed the order of the elements
in the TLV structure is reversed. This is in theory not a problem,
but differs from what we know from our practical experience. This
commint fixes the problem.

Change-Id: I1d71c947350d3c5a85ff36b71c1b8f036071d162
---
M openbsc/src/gprs/gprs_llc.c
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/59/1359/1

diff --git a/openbsc/src/gprs/gprs_llc.c b/openbsc/src/gprs/gprs_llc.c
index e5a80e5..0215029 100644
--- a/openbsc/src/gprs/gprs_llc.c
+++ b/openbsc/src/gprs/gprs_llc.c
@@ -77,9 +77,9 @@
 	xid_n201i.data_len = 2;
 
 	/* Add locally managed XID Fields */
-	llist_add(&xid_n201i.list, &xid_fields);
-	llist_add(&xid_n201u.list, &xid_fields);
 	llist_add(&xid_version.list, &xid_fields);
+	llist_add(&xid_n201u.list, &xid_fields);
+	llist_add(&xid_n201i.list, &xid_fields);
 
 	/* Append layer 3 XID field (if present) */
 	if (l3_xid_field) {

-- 
To view, visit https://gerrit.osmocom.org/1359
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d71c947350d3c5a85ff36b71c1b8f036071d162
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>



More information about the gerrit-log mailing list