Change in libosmocore[master]: gprs_ns2: replace OSMO_ASSERT()

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

lynxis lazus gerrit-no-reply at lists.osmocom.org
Tue Mar 2 09:15:15 UTC 2021


lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/23183 )


Change subject: gprs_ns2: replace OSMO_ASSERT()
......................................................................

gprs_ns2: replace OSMO_ASSERT()

When freeing the nsvc there are cases when the NSE is still alive.

Change-Id: I4f32dee705c9886717339fe7cce5b4cd8d2c18f7
---
M src/gb/gprs_ns2.c
1 file changed, 3 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/83/23183/1

diff --git a/src/gb/gprs_ns2.c b/src/gb/gprs_ns2.c
index a79dd05..c56b0b5 100644
--- a/src/gb/gprs_ns2.c
+++ b/src/gb/gprs_ns2.c
@@ -1455,8 +1455,9 @@
 		if (ns2_vc_is_unblocked(nsvc))
 			active_nsvcs++;
 	}
-	/* an alive nse should always have active_nsvcs */
-	OSMO_ASSERT(active_nsvcs);
+
+	if (!active_nsvcs)
+		return 0;
 
 	active_binds = talloc_zero_array(nse, struct gprs_ns2_vc_bind*, active_nsvcs);
 	if (!active_binds)

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I4f32dee705c9886717339fe7cce5b4cd8d2c18f7
Gerrit-Change-Number: 23183
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210302/8e1f95aa/attachment.htm>


More information about the gerrit-log mailing list