Change in osmo-bsc[master]: lchan: call reset() upon alloc

neels gerrit-no-reply at lists.osmocom.org
Thu Jul 8 04:49:19 UTC 2021


neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/24882 )


Change subject: lchan: call reset() upon alloc
......................................................................

lchan: call reset() upon alloc

call lchan_reset() after allocation, to make sure that lchan->* fields
that have a nonzero default value get initialized properly.

In particular, a future patch adds interference measurements, and an
lchan that never received any Resource Indication info should always
indicate that there are no measurements (a nonzero constant).

Change-Id: I700a969f5b11c21dacda9a7cad00c943dce554b3
---
M src/osmo-bsc/lchan_fsm.c
1 file changed, 3 insertions(+), 0 deletions(-)



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

diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c
index 6350481..47a8f9e 100644
--- a/src/osmo-bsc/lchan_fsm.c
+++ b/src/osmo-bsc/lchan_fsm.c
@@ -433,6 +433,8 @@
 	lchan_rtp_fsm_init();
 }
 
+static void lchan_reset(struct gsm_lchan *lchan);
+
 void lchan_fsm_alloc(struct gsm_lchan *lchan)
 {
 	OSMO_ASSERT(lchan->ts);
@@ -444,6 +446,7 @@
 	lchan->fi->priv = lchan;
 	lchan_fsm_update_id(lchan);
 	LOGPFSML(lchan->fi, LOGL_DEBUG, "new lchan\n");
+	lchan_reset(lchan);
 }
 
 /* Clear volatile state of the lchan. Clear all except

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/24882
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I700a969f5b11c21dacda9a7cad00c943dce554b3
Gerrit-Change-Number: 24882
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210708/9982c20c/attachment.htm>


More information about the gerrit-log mailing list