dlt-daemon: fix CVE-2023-26257

An issue was discovered in the Connected Vehicle Systems Alliance (COVESA; formerly GENIVI)
dlt-daemon through 2.18.8. Dynamic memory is not released after
it is allocated in dlt-control-common.c.

References:
https://nvd.nist.gov/vuln/detail/CVE-2023-26257
https://github.com/COVESA/dlt-daemon/issues/440

Signed-off-by: Yogita Urade <yogita.urade@windriver.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
This commit is contained in:
Yogita Urade 2023-04-14 05:57:35 +00:00 committed by Khem Raj
parent e64fda2f96
commit ad73ee286f
2 changed files with 35 additions and 0 deletions

View File

@ -0,0 +1,34 @@
From b6149e203f919c899fefc702a17fbb78bdec3700 Mon Sep 17 00:00:00 2001
From: Le Van Khanh <Khanh.LeVan@vn.bosch.com>
Date: Thu, 9 Feb 2023 03:17:13 -0500
Subject: [PATCH] Fix memory leak
Free the ecuid_conf in case of memory alllocated
CVE: CVE-2023-26257
Upstream-Status: Backport
[https://github.com/COVESA/dlt-daemon/pull/441/commits/b6149e203f919c899fefc702a17fbb78bdec3700]
Signed-off-by: Le Van Khanh <Khanh.LeVan@vn.bosch.com>
Signed-off-by: Yogita Urade <yogita.urade@windriver.com>
---
src/console/dlt-control-common.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/console/dlt-control-common.c b/src/console/dlt-control-common.c
index abcaf92..64951c1 100644
--- a/src/console/dlt-control-common.c
+++ b/src/console/dlt-control-common.c
@@ -124,6 +124,8 @@ void set_ecuid(char *ecuid)
if (dlt_parse_config_param("ECUId", &ecuid_conf) == 0) {
memset(local_ecuid, 0, DLT_CTRL_ECUID_LEN);
strncpy(local_ecuid, ecuid_conf, DLT_CTRL_ECUID_LEN);
+ if (ecuid_conf !=NULL)
+ free(ecuid_conf);
local_ecuid[DLT_CTRL_ECUID_LEN - 1] = '\0';
}
else {
--
2.34.1

View File

@ -19,6 +19,7 @@ SRC_URI = "git://github.com/GENIVI/${BPN}.git;protocol=https;branch=master \
file://0004-Modify-systemd-config-directory.patch \
file://0001-cmake-Link-with-libatomic-on-rv32-rv64.patch \
file://0001-dlt-system-Fix-buffer-overflow-detection-on-32bit-ta.patch \
file://0001-Fix-memory-leak.patch \
"
SRCREV = "6a3bd901d825c7206797e36ea98e10a218f5aad2"