neels has uploaded this change for review.

View Change

coverity: CID#27206

prevent negative bitshift.

Change-Id: Ic6068277add1e336c67409d48211d45493153924
---
M src/per_support.c
1 file changed, 3 insertions(+), 0 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/libasn1c refs/changes/89/37989/1
diff --git a/src/per_support.c b/src/per_support.c
index a1c5ae7..7e55263 100644
--- a/src/per_support.c
+++ b/src/per_support.c
@@ -490,6 +490,9 @@
buf[2] = bits >> 8,
buf[3] = bits;
else {
+ /* sanity: prevent negative bitshift (CID#27206) */
+ if (obits < 24)
+ return -1;
per_put_few_bits(po, bits >> (obits - 24), 24);
per_put_few_bits(po, bits, obits - 24);
}

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

Gerrit-MessageType: newchange
Gerrit-Project: libasn1c
Gerrit-Branch: master
Gerrit-Change-Id: Ic6068277add1e336c67409d48211d45493153924
Gerrit-Change-Number: 37989
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>