7 Jun
2023
7 Jun
'23
11:56 a.m.
Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/33179 )
Change subject: include Global RNC-ID in RESET ......................................................................
Patch Set 4: Code-Review+1
(1 comment)
File src/osmo-hnbgw/cnlink.c:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/33179/comment/7586cbc1_441ddb35 PS4, Line 147: / why are we handling that case if it is in violation of the 3GPP specs? For backwards compatibility with old config files? In that case it is ok but we should probably warn the user (via the log) that his config is missing vital information and hence we're sending non-compliant messages.
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/33179
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I2cd5d7ea86a6ed0916befe219dbf21373afbd95b
Gerrit-Change-Number: 33179
Gerrit-PatchSet: 4
Gerrit-Owner: neels nhofmeyr@sysmocom.de
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge laforge@osmocom.org
Gerrit-Reviewer: pespin pespin@sysmocom.de
Gerrit-Attention: neels nhofmeyr@sysmocom.de
Gerrit-Comment-Date: Wed, 07 Jun 2023 09:56:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment