pespin has submitted this change. (
https://gerrit.osmocom.org/c/osmo-hnodeb/+/28401 )
Change subject: llsk: Fix previous audio sapi version passed instead of current
......................................................................
llsk: Fix previous audio sapi version passed instead of current
This bug showed up when running the whole TTCN3 hnodeb-test testsuite.
llsk_audio_sapi_version_confirmed() was setting the expected struct size
fo the version previously set.
Related: SYS#5516
Change-Id: Ic94881f46b20ba2817b56d88ada9da5d13b1e294
---
M src/osmo-hnodeb/llsk.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
diff --git a/src/osmo-hnodeb/llsk.c b/src/osmo-hnodeb/llsk.c
index f2d778f..ddb13c4 100644
--- a/src/osmo-hnodeb/llsk.c
+++ b/src/osmo-hnodeb/llsk.c
@@ -167,7 +167,7 @@
return -1;
if (rem_version > LLSK_SAPI_AUDIO_VERSION_MAX)
return LLSK_SAPI_AUDIO_VERSION_MAX;
- if (llsk_audio_sapi_version_confirmed(hnb->llsk.sapi_version_audio) < 0)
+ if (llsk_audio_sapi_version_confirmed(rem_version) < 0)
return -1;
hnb->llsk.sapi_version_audio = rem_version;
break;
--
To view, visit
https://gerrit.osmocom.org/c/osmo-hnodeb/+/28401
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnodeb
Gerrit-Branch: master
Gerrit-Change-Id: Ic94881f46b20ba2817b56d88ada9da5d13b1e294
Gerrit-Change-Number: 28401
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged