Change in osmo-gbproxy[master]: gbproxy: Remove unused variable and improve log message

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/.

daniel gerrit-no-reply at lists.osmocom.org
Wed Sep 29 09:50:13 UTC 2021


daniel has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-gbproxy/+/25637 )


Change subject: gbproxy: Remove unused variable and improve log message
......................................................................

gbproxy: Remove unused variable and improve log message

Change-Id: I8b8a761a828b29dacacd6803427a27e16f041f18
---
M src/gb_proxy.c
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gbproxy refs/changes/37/25637/1

diff --git a/src/gb_proxy.c b/src/gb_proxy.c
index 90d587f..31766c0 100644
--- a/src/gb_proxy.c
+++ b/src/gb_proxy.c
@@ -1324,7 +1324,6 @@
 	uint16_t bvci;
 	char log_pfx[32];
 	int rc = 0;
-	int cause;
 	int i;
 	bool paging_bc = false;
 
@@ -1498,7 +1497,7 @@
 	return rc;
 
 err_no_bvc:
-	LOGPNSE(nse, LOGL_ERROR, "Rx %s: Cannot find BVC\n", pdut_name);
+	LOGPNSE(nse, LOGL_ERROR, "Rx %s: Cannot find BVC %05u\n", pdut_name, bvci);
 	rate_ctr_inc(rate_ctr_group_get_ctr(cfg->ctrg, GBPROX_GLOB_CTR_INV_BVCI));
 	return tx_status(nse, ns_bvci, BSSGP_CAUSE_INV_MAND_INF, NULL, msg);
 }

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-gbproxy/+/25637
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gbproxy
Gerrit-Branch: master
Gerrit-Change-Id: I8b8a761a828b29dacacd6803427a27e16f041f18
Gerrit-Change-Number: 25637
Gerrit-PatchSet: 1
Gerrit-Owner: daniel <dwillmann at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210929/204645be/attachment.htm>


More information about the gerrit-log mailing list