pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, approved
e1_input: Fix e1i_ts pointing to old line after line_clone

Change-Id: I8f8e1fd67a63b46d59f433ad01bb2ab880cdf910
---
M src/e1_input.c
1 file changed, 4 insertions(+), 0 deletions(-)

diff --git a/src/e1_input.c b/src/e1_input.c
index 145fccf..76adbd9 100644
--- a/src/e1_input.c
+++ b/src/e1_input.c
@@ -675,6 +675,10 @@
OSMO_ASSERT(clone->sock_path);
}

+ /* Update TS to point to the cloned line: */
+ for (unsigned int i = 0; i < line->num_ts; i++)
+ clone->ts[i].line = clone;
+
/*
* Rate counters and driver data are shared between clones. These are pointers
* to dynamic memory so we use reference counting to avoid a double-free (see OS#3137).

To view, visit change 38965. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I8f8e1fd67a63b46d59f433ad01bb2ab880cdf910
Gerrit-Change-Number: 38965
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>