[PATCH] osmo-pcu[master]: Don't access TBF internals in vty functions

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

Max gerrit-no-reply at lists.osmocom.org
Fri Jan 12 15:25:53 UTC 2018


Review at  https://gerrit.osmocom.org/5771

Don't access TBF internals in vty functions

Obtain corresponding window object of UL/DL TBF by using proper accessor
function instead of direct access to private member.

Change-Id: I89bcd2c2b0b6f120d40d20fd43c1e516de3e3950
---
M src/pcu_vty_functions.cpp
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/71/5771/1

diff --git a/src/pcu_vty_functions.cpp b/src/pcu_vty_functions.cpp
index 0a80a23..001164c 100644
--- a/src/pcu_vty_functions.cpp
+++ b/src/pcu_vty_functions.cpp
@@ -67,7 +67,7 @@
 	vty_out(vty, " CS=%s", tbf->current_cs().name());
 
 	if (ul_tbf) {
-		gprs_rlc_ul_window *win = &ul_tbf->m_window;
+		gprs_rlc_ul_window *win = ul_tbf->window();
 		vty_out(vty, " WS=%u V(Q)=%d V(R)=%d",
 			ul_tbf->window_size(), win->v_q(), win->v_r());
 		vty_out(vty, "%s", VTY_NEWLINE);
@@ -79,7 +79,7 @@
 		}
 	}
 	if (dl_tbf) {
-		gprs_rlc_dl_window *win = &dl_tbf->m_window;
+		gprs_rlc_dl_window *win = dl_tbf->window();
 		vty_out(vty, " WS=%u V(A)=%d V(S)=%d nBSN=%d%s",
 			dl_tbf->window_size(), win->v_a(), win->v_s(), win->resend_needed(),
 			win->window_stalled() ? " STALLED" : "");

-- 
To view, visit https://gerrit.osmocom.org/5771
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I89bcd2c2b0b6f120d40d20fd43c1e516de3e3950
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>



More information about the gerrit-log mailing list