Change in libosmo-netif[master]: amr: fix off-by-one in osmo_amr_bwe_to_oa()

[dexter]

Hello laforge, Jenkins Builder, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/c/libosmo-netif/+/18451

to look at the new patch set (#2).

Change subject: amr: fix off-by-one in osmo_amr_bwe_to_oa()
......................................................................

amr: fix off-by-one in osmo_amr_bwe_to_oa()

The for loop in osmo_amr_bwe_to_oa, that converts the body part of the
AMR payload runs one byte too far. This may cause that some of the
padding bits in the end are not set to zero. The loop is designed to
convert n-1 bytes and the nth byte is done separately at the end.

Change-Id: I91e755b83aaac722079879c026d913cc446812d1
---
M src/amr.c
M tests/amr/amr_test.c
M tests/amr/amr_test.ok
3 files changed, 10 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/51/18451/2
-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/18451
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I91e755b83aaac722079879c026d913cc446812d1
Gerrit-Change-Number: 18451
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200525/6d0ced74/attachment.htm>