laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified tnt: Looks good to me, but someone else must approve laforge: Looks good to me, approved
ctl: Prevent clients from opening TS0

This doesn't work, as the mux_demux.c code doesn't pass the TS0 bitstream
to users anyway. So let's reject clients attempting this.

Change-Id: Idb2d20da7de72dad38ae2fccdd7630677d0f0cc8
---
M src/ctl.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ctl.c b/src/ctl.c
index b5fdee4..54d37c5 100644
--- a/src/ctl.c
+++ b/src/ctl.c
@@ -45,7 +45,7 @@
static struct e1_ts *
_e1d_get_ts(struct e1_line *line, uint8_t ts)
{
- if (ts < 32)
+ if (ts > 0 && ts < 32)
return &line->ts[ts];
else if (ts == E1DP_TS_SUPERCHAN)
return &line->superchan;

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

Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: Idb2d20da7de72dad38ae2fccdd7630677d0f0cc8
Gerrit-Change-Number: 29951
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: tnt <tnt@246tNt.com>
Gerrit-MessageType: merged