[S] Change in libosmocore[master]: gsm0503_tch_hr_decode(): look at all 8 stealing bits

falconia has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32960 ) Change subject: gsm0503_tch_hr_decode(): look at all 8 stealing bits ...................................................................... gsm0503_tch_hr_decode(): look at all 8 stealing bits This function needs to look at hl and hu stealing bits in order to decide if it should decode FACCH/H instead of TCH/HS traffic. However, out of the 8 (in total) hl and hu bits that get set when FACCH/H stealing takes place, the function only looked at 7 of them - one was missed. Fix this bug. Change-Id: I08c4358b26d69910190f89a53b654bc58c2efea9 --- M src/coding/gsm0503_coding.c 1 file changed, 16 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved diff --git a/src/coding/gsm0503_coding.c b/src/coding/gsm0503_coding.c index 3fd440f..ce24650 100644 --- a/src/coding/gsm0503_coding.c +++ b/src/coding/gsm0503_coding.c @@ -1997,7 +1997,7 @@ steal -= h; } - for (i = 2; i < 5; i++) { + for (i = 2; i < 6; i++) { gsm0503_tch_burst_unmap(NULL, &bursts[i * 116], &h, 1); steal -= h; } -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/32960 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I08c4358b26d69910190f89a53b654bc58c2efea9 Gerrit-Change-Number: 32960 Gerrit-PatchSet: 1 Gerrit-Owner: falconia &lt;falcon(a)freecalypso.org&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia &lt;falcon(a)freecalypso.org&gt; Gerrit-Reviewer: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged