Change in osmo-bsc[master]: log: fix logging in rsl_rx_chan_act_nack()

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Thu May 24 16:59:46 UTC 2018


Neels Hofmeyr has uploaded this change for review. ( https://gerrit.osmocom.org/9282


Change subject: log: fix logging in rsl_rx_chan_act_nack()
......................................................................

log: fix logging in rsl_rx_chan_act_nack()

In each code path within rsl_rx_chan_act_nack(), do separate logging of the
NACK to ensure proper termination of each log line.

When receiving a Chan Act NACK, we possibly mixed a LOGP() within an
unterminated other LOGP() that had not been ended with a LOGPC() yet.

Change-Id: Icd2772b21ef3a2ff5af11b7c92dff0ecb4d87ff0
---
M src/libbsc/abis_rsl.c
1 file changed, 9 insertions(+), 6 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/82/9282/1

diff --git a/src/libbsc/abis_rsl.c b/src/libbsc/abis_rsl.c
index ffbf630..39de1b7 100644
--- a/src/libbsc/abis_rsl.c
+++ b/src/libbsc/abis_rsl.c
@@ -1328,18 +1328,21 @@
 		return -1;
 	}
 
-	LOGP(DRSL, LOGL_ERROR, "%s CHANNEL ACTIVATE NACK ",
-		gsm_lchan_name(msg->lchan));
-
 	/* BTS has rejected channel activation ?!? */
-	if (dh->ie_chan != RSL_IE_CHAN_NR)
+	if (dh->ie_chan != RSL_IE_CHAN_NR) {
+		LOGP(DRSL, LOGL_ERROR, "%s CHANNEL ACTIVATE NACK, and chan nr mismatches\n",
+		     gsm_lchan_name(msg->lchan));
 		return -EINVAL;
+	}
 
 	rsl_tlv_parse(&tp, dh->data, msgb_l2len(msg)-sizeof(*dh));
 	if (TLVP_PRESENT(&tp, RSL_IE_CAUSE)) {
 		const uint8_t *cause = TLVP_VAL(&tp, RSL_IE_CAUSE);
+		LOGP(DRSL, LOGL_ERROR, "%s CHANNEL ACTIVATE NACK: ",
+		     gsm_lchan_name(msg->lchan));
 		print_rsl_cause(LOGL_ERROR, cause,
 				TLVP_LEN(&tp, RSL_IE_CAUSE));
+		LOGPC(DRSL, LOGL_ERROR, "\n");
 		msg->lchan->error_cause = *cause;
 		if (*cause != RSL_ERR_RCH_ALR_ACTV_ALLOC) {
 			rsl_lchan_mark_broken(msg->lchan, "NACK on activation");
@@ -1347,11 +1350,11 @@
 			rsl_rf_chan_release(msg->lchan, 1, SACCH_DEACTIVATE);
 
 	} else {
+		LOGP(DRSL, LOGL_ERROR, "%s CHANNEL ACTIVATE NACK, no cause IE\n",
+		     gsm_lchan_name(msg->lchan));
 		rsl_lchan_mark_broken(msg->lchan, "NACK on activation no IE");
 	}
 
-	LOGPC(DRSL, LOGL_ERROR, "\n");
-
 	send_lchan_signal(S_LCHAN_ACTIVATE_NACK, msg->lchan, NULL);
 	return 0;
 }

-- 
To view, visit https://gerrit.osmocom.org/9282
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icd2772b21ef3a2ff5af11b7c92dff0ecb4d87ff0
Gerrit-Change-Number: 9282
Gerrit-PatchSet: 1
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180524/ea9b9889/attachment.htm>


More information about the gerrit-log mailing list