pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved
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(-)

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 change 28401. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-hnodeb
Gerrit-Branch: master
Gerrit-Change-Id: Ic94881f46b20ba2817b56d88ada9da5d13b1e294
Gerrit-Change-Number: 28401
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged