Merge pull request #15404 from IVOES/fix-overrunning-write

Fix potentially overrunning write of sprintf
ARMmbed · Apr 26, 2023 · 0ea5dec · 0ea5dec
2 parents 9a82ad6 + c958d1d
commit 0ea5dec
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/connectivity/drivers/cellular/GEMALTO/CINTERION/GEMALTO_CINTERION_CellularStack.cpp b/connectivity/drivers/cellular/GEMALTO/CINTERION/GEMALTO_CINTERION_CellularStack.cpp
@@ -186,8 +186,8 @@ nsapi_error_t GEMALTO_CINTERION_CellularStack::socket_open_defer(CellularSocket
                         }
                     }
                     if (strcmp(paramTag, "conId") == 0) {
-                        char buf[10];
-                        std::sprintf(buf, "%d", _cid);
+                        char buf[12];
+                        std::snprintf(buf, sizeof(buf), "%d", _cid);
                         if (strcmp(paramValue, buf) == 0) {
                             foundConIdType = true;
                         }