Patch Set 1: Code-Review-2

(1 comment)

https://gerrit.osmocom.org/#/c/3090/1/src/common/oml.c
File src/common/oml.c:

Line 1126:              bts->mo.obj_inst.trx_nr = foh->obj_inst.trx_nr;
this most certainly is completely wrong.  The bts->mo is the instance of the 
managed object for the BTS. It's "MO Address" is configured once statically at 
BTS startup and never modified - *particularly* not by a get/read request.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8b5a5ab8bd07daf2500b66dec428b89b7f8cd852
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Max <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-HasComments: Yes

Reply via email to