fixeria submitted this change.

View Change


Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve
library/RLCMAC_EncDec: fix segfault in encode_trailing_padding_spb()

Change-Id: Ib0992a50026f0e0deebb959788ed6e79c5286fec
Related: OS#6310
---
M library/RLCMAC_EncDec.cc
1 file changed, 11 insertions(+), 0 deletions(-)

diff --git a/library/RLCMAC_EncDec.cc b/library/RLCMAC_EncDec.cc
index 1b56984..dd32d8f 100644
--- a/library/RLCMAC_EncDec.cc
+++ b/library/RLCMAC_EncDec.cc
@@ -455,6 +455,7 @@
if (data_len > blk_len_no_spb) {
fprintf(stderr, "Buffer too large for requested CS! %s (%s:%u)\n", __func__, __FILE__, __LINE__);
// TODO: throw exception?
+ return;
}

for (int i = 0; i < blk_len_no_spb - data_len; i++)

To view, visit change 35373. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ib0992a50026f0e0deebb959788ed6e79c5286fec
Gerrit-Change-Number: 35373
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged