laforge has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-abis/+/38130?usp=email )
Change subject: osmo_trau_frame_decode_8k: check all sync bits
......................................................................
osmo_trau_frame_decode_8k: check all sync bits
The decoding of unknown TRAU-8k frames (the direction and 8 kbit/s
submux are known, but nothing else) begins by trial-and-error
checking of 4 possible GSM 08.61 sync patterns. In the case of
classic HR speech/data and AMR-low sync patterns, not all bits
were checked.
Change-Id: I47b055b4d3b92f018508fd7baa0784dfcabe6262
---
M src/trau/trau_frame.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/src/trau/trau_frame.c b/src/trau/trau_frame.c
index 509021d..9cb8a66 100644
--- a/src/trau/trau_frame.c
+++ b/src/trau/trau_frame.c
@@ -1376,7 +1376,7 @@
return false;
if (bits[16] != 0 || bits[17] != 1)
return false;
- for (i = 24; i < 20 * 8; i += 16) {
+ for (i = 24; i < 20 * 8; i += 8) {
if (bits[i] != 1)
return false;
}
@@ -1398,7 +1398,7 @@
return false;
if (bits[24] != 0 || bits[25] != 1)
return false;
- for (i = 32; i < 20 * 8; i += 16) {
+ for (i = 32; i < 20 * 8; i += 8) {
if (bits[i] != 1)
return false;
}
--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/38130?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I47b055b4d3b92f018508fd7baa0784dfcabe6262
Gerrit-Change-Number: 38130
Gerrit-PatchSet: 2
Gerrit-Owner: falconia <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
