Change in osmocom-bb[master]: virt_l1_sched_execute: talloc_free() TDMA item after unlinking it

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

laforge gerrit-no-reply at lists.osmocom.org
Sat Mar 7 15:26:04 UTC 2020


laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/17412 )


Change subject: virt_l1_sched_execute: talloc_free() TDMA item after unlinking it
......................................................................

virt_l1_sched_execute: talloc_free() TDMA item after unlinking it

We so far relied on it being free'd once the TDMA item is free'd,
but let's make it more explicit.  After we've unlinked it from the
list, nobody is going to reference it ever again.

Change-Id: I57a596428be10ce720e0b528ecfc44a70e3e3078
---
M src/host/virt_phy/src/virt_l1_sched_simple.c
1 file changed, 1 insertion(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/12/17412/1

diff --git a/src/host/virt_phy/src/virt_l1_sched_simple.c b/src/host/virt_phy/src/virt_l1_sched_simple.c
index a14a115..486d319 100644
--- a/src/host/virt_phy/src/virt_l1_sched_simple.c
+++ b/src/host/virt_phy/src/virt_l1_sched_simple.c
@@ -97,6 +97,7 @@
 				ti_next->handler_cb(ms, mi_next->fn, ti_next->ts, ti_next->msg);
 				/* remove handled tdma sched item */
 				llist_del(&ti_next->tdma_item_entry);
+				talloc_free(ti_next);
 			}
 			/* remove handled mframe sched item */
 			llist_del(&mi_next->mframe_item_entry);

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I57a596428be10ce720e0b528ecfc44a70e3e3078
Gerrit-Change-Number: 17412
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200307/eea228e9/attachment.htm>


More information about the gerrit-log mailing list