Change in osmo-bsc[master]: OML: Fix encoded length value of OML GET ATTRIBUTES

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/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Tue Feb 12 18:53:56 UTC 2019


Harald Welte has uploaded this change for review. ( https://gerrit.osmocom.org/12890


Change subject: OML: Fix encoded length value of OML GET ATTRIBUTES
......................................................................

OML: Fix encoded length value of OML GET ATTRIBUTES

Before this patch, all OML GET ATTRIBUTES messages were encoded with
an erroneous OML header length value.  The length value was always three
bytes less than the actual message length.  This patch fixes the
problem.

Change-Id: I56068de0bb14a99ec39be587e542e27cddb7d1df
Closes: OS#3799
---
M src/osmo-bsc/abis_nm.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/90/12890/1

diff --git a/src/osmo-bsc/abis_nm.c b/src/osmo-bsc/abis_nm.c
index f1306fc..9b4af6e 100644
--- a/src/osmo-bsc/abis_nm.c
+++ b/src/osmo-bsc/abis_nm.c
@@ -1717,7 +1717,7 @@
 
 	msg = nm_msgb_alloc();
 	oh = (struct abis_om_hdr *) msgb_put(msg, ABIS_OM_FOM_HDR_SIZE);
-	fill_om_fom_hdr(oh, attr_len, NM_MT_GET_ATTR, obj_class,
+	fill_om_fom_hdr(oh, 3+attr_len, NM_MT_GET_ATTR, obj_class,
 			bts_nr, trx_nr, ts_nr);
 	msgb_tl16v_put(msg, NM_ATT_LIST_REQ_ATTR, attr_len, attr);
 

-- 
To view, visit https://gerrit.osmocom.org/12890
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I56068de0bb14a99ec39be587e542e27cddb7d1df
Gerrit-Change-Number: 12890
Gerrit-PatchSet: 1
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190212/e9bd29e2/attachment.htm>


More information about the gerrit-log mailing list