pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/40160?usp=email ) Change subject: mgw_fsm: use LOG_MAP() to log error ...................................................................... mgw_fsm: use LOG_MAP() to log error Change-Id: Ic543a3c17eddf4b1da51172223eccaa62a441fd3 --- M src/osmo-hnbgw/mgw_fsm.c 1 file changed, 6 insertions(+), 6 deletions(-) Approvals: lynxis lazus: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/osmo-hnbgw/mgw_fsm.c b/src/osmo-hnbgw/mgw_fsm.c index d44a6e3..3d230c6 100644 --- a/src/osmo-hnbgw/mgw_fsm.c +++ b/src/osmo-hnbgw/mgw_fsm.c @@ -846,9 +846,9 @@ * into account under the assumption that voice calls typically require a single RAB only. Nevertheless, we * will block all incoming RAB assignments that try to assign more (or less) than one RAB. */ if (ranap_rab_ass_req_ies_get_count(&message->msg.raB_AssignmentRequestIEs) != 1) { - LOGP(DMGW, LOGL_ERROR, - "%s() rua_ctx_id=%d, RAB-AssignmentRequest with more than one RAB assignment -- abort!\n", - __func__, map->rua_ctx_id); + LOG_MAP(map, DMGW, LOGL_ERROR, + "%s() RAB-AssignmentRequest with more than one RAB assignment -- abort!\n", + __func__); tx_release_req(map); return -1; } @@ -886,9 +886,9 @@ /* NOTE: This situation is a corner-case. We may end up here when the co-located MGW caused a problem * on the way between RANAP RAB Assignment Request and RANAP RAB Assignment Response. */ - LOGP(DMGW, LOGL_ERROR, - "%s() rua_ctx_id=%d, no MGW fsm -- sending Iu-Release-Request!\n", - __func__, map->rua_ctx_id); + LOG_MAP(map, DMGW, LOGL_ERROR, + "%s() no MGW fsm -- sending Iu-Release-Request!\n", + __func__); /* Send a release request, to make sure that the MSC is aware of the problem. */ tx_release_req(map); -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40160?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: Ic543a3c17eddf4b1da51172223eccaa62a441fd3 Gerrit-Change-Number: 40160 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>