Harald Welte has submitted this change and it was merged. Change subject: mobile/gsm322.c: prevent buffer over-/under-run ......................................................................
mobile/gsm322.c: prevent buffer over-/under-run Change-Id: Ic12587a6c6456b8663e5357cf68a22c6d1927a07 --- M src/host/layer23/src/mobile/gsm322.c 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/host/layer23/src/mobile/gsm322.c b/src/host/layer23/src/mobile/gsm322.c index 0e2c9af..91729d8 100644 --- a/src/host/layer23/src/mobile/gsm322.c +++ b/src/host/layer23/src/mobile/gsm322.c @@ -320,6 +320,9 @@ else value -= min; + /* Prevent 'bar' buffer over-/under-run */ + OSMO_ASSERT(value >= 0 && value < 128); + /* Prevent calling memset() with zero length */ if (value == 0) return ""; -- To view, visit https://gerrit.osmocom.org/4729 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic12587a6c6456b8663e5357cf68a22c6d1927a07 Gerrit-PatchSet: 4 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Vadim Yanitskiy <axilira...@gmail.com> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Vadim Yanitskiy <axilira...@gmail.com>