Attention is currently required from: fixeria, pespin, lynxis lazus.
osmith has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-iuh/+/28208
)
Change subject: Make logging message about received RANAP message more meaningful
......................................................................
Patch Set 4: -Code-Review
(1 comment)
File src/ranap_common_ran.c:
https://gerrit.osmocom.org/c/osmo-iuh/+/28208/comment/1dba2a3b_f865fc5b
PS4, Line 73: LOGP(DRANAP, LOGL_NOTICE, "Not freeing suspicious RANAP "
needs to be adjusted here as well? and similar wording seems to be in a lot of places:
$ git grep "Received suspicious"
src/ranap_common_cn.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_cn.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_cn.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_cn.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_cn.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_cn.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_cn.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_ran.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
src/ranap_common_ran.c: LOGP(DRANAP, LOGL_NOTICE, "Received suspicious
RANAP "
--
To view, visit
https://gerrit.osmocom.org/c/osmo-iuh/+/28208
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: If63d942496491f1e9ee454034ec97d25764fde65
Gerrit-Change-Number: 28208
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Thu, 02 Jun 2022 12:06:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment