From 1f2da926260fb1b2f7b0f4a567becbcf9a2e564c Mon Sep 17 00:00:00 2001
From: Michael Zillgith <michael.zillgith@mz-automation.de>
Date: Thu, 3 Mar 2022 00:09:01 +0100
Subject: [PATCH] - fixed position range check in MmsValue_getOctetStringOctet

---
 src/mms/iso_mms/common/mms_value.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mms/iso_mms/common/mms_value.c b/src/mms/iso_mms/common/mms_value.c
index 2b604f85..f04670b6 100644
--- a/src/mms/iso_mms/common/mms_value.c
+++ b/src/mms/iso_mms/common/mms_value.c
@@ -1464,7 +1464,7 @@ MmsValue_getOctetStringOctet(MmsValue* self, int octetPos)
 {
     uint8_t octet = 0x00; /* default value, for out of range request */
 
-    if (octetPos < self->value.octetString.size) {
+    if ((octetPos >= 0) && (octetPos < self->value.octetString.size)) {
         octet = self->value.octetString.buf[octetPos];
     }