logstorage: Truncate ECUid in Logstorage filter to prevent crash (#402)

Signed-off-by: Andrei-Mircea Rusu <[email protected]> Signed-off-by: Andrei-Mircea Rusu <[email protected]> Co-authored-by: Andrei-Mircea Rusu <[email protected]>
COVESA · Sep 21, 2022 · 6170416 · 6170416
1 parent 995ba51
commit 6170416
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/offlinelogstorage/dlt_offline_logstorage.c b/src/offlinelogstorage/dlt_offline_logstorage.c
@@ -498,7 +498,7 @@ DLT_STATIC void dlt_logstorage_create_keys_only_ctid(char *ecuid, char *ctid,
     int curr_len = 0;
 
     if (ecuid != NULL) {
-        strncpy(curr_str, ecuid, strlen(ecuid));
+        strncpy(curr_str, ecuid, DLT_ID_SIZE);
         strncat(curr_str, "::", 2);
     }
     else {
@@ -530,7 +530,7 @@ DLT_STATIC void dlt_logstorage_create_keys_only_apid(char *ecuid, char *apid,
     int curr_len = 0;
 
     if (ecuid != NULL) {
-        strncpy(curr_str, ecuid, strlen(ecuid));
+        strncpy(curr_str, ecuid, DLT_ID_SIZE);
         strncat(curr_str, ":", 1);
     }
     else {
@@ -564,7 +564,7 @@ DLT_STATIC void dlt_logstorage_create_keys_multi(char *ecuid, char *apid,
     int curr_len = 0;
 
     if (ecuid != NULL) {
-        strncpy(curr_str, ecuid, strlen(ecuid));
+        strncpy(curr_str, ecuid, DLT_ID_SIZE);
         strncat(curr_str, ":", 1);
     }
     else {
@@ -595,7 +595,7 @@ DLT_STATIC void dlt_logstorage_create_keys_only_ecu(char *ecuid, char *key)
 {
     char curr_str[DLT_OFFLINE_LOGSTORAGE_MAX_KEY_LEN + 1] = { 0 };
 
-    strncpy(curr_str, ecuid, strlen(ecuid));
+    strncpy(curr_str, ecuid, DLT_ID_SIZE);
     strncat(curr_str, "::", 2);
 
     strncpy(key, curr_str, strlen(curr_str));