pespin submitted this change.

View Change

Approvals: lynxis lazus: Looks good to me, approved Jenkins Builder: Verified
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(-)

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 change 40160. To unsubscribe, or for help writing mail filters, visit settings.

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@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>