pespin has submitted this change. (
https://gerrit.osmocom.org/c/osmo-pcu/+/30535 )
Change subject: pdch_ulc_get_node(): assert if ulc pointer is null
......................................................................
pdch_ulc_get_node(): assert if ulc pointer is null
Make crah more obvious if this condition is reached.
Related: SYS#6231
Change-Id: I2689ceaf39009c5e708d5c72e41744658681c917
---
M src/pdch_ul_controller.c
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
osmith: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
pespin: Verified
diff --git a/src/pdch_ul_controller.c b/src/pdch_ul_controller.c
index 2ae553f..1746333 100644
--- a/src/pdch_ul_controller.c
+++ b/src/pdch_ul_controller.c
@@ -71,6 +71,7 @@
struct pdch_ulc_node *pdch_ulc_get_node(struct pdch_ulc *ulc, uint32_t fn)
{
+ OSMO_ASSERT(ulc);
struct rb_node *node = ulc->tree_root.rb_node;
struct pdch_ulc_node *it;
int res;
--
To view, visit
https://gerrit.osmocom.org/c/osmo-pcu/+/30535
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I2689ceaf39009c5e708d5c72e41744658681c917
Gerrit-Change-Number: 30535
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged