helium-3rd-party/meta-openembedded
1
0
Fork 0
mirror of git://git.openembedded.org/meta-openembedded synced 2026-04-02 02:49:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

keepalived: patch CVE-2024-41184

Browse Source 
Details: https://nvd.nist.gov/vuln/detail/CVE-2024-41184

Backport the patches referenced by upstream in the bug
mentioned by the NVD advisory.

Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
Signed-off-by: Anuj Mittal <anuj.mittal@oss.qualcomm.com>
This commit is contained in:
Gyorgy Sarvari 2026-03-05 15:15:13 +01:00 committed by Anuj Mittal
parent ad6ea218ae
commit a0a3169b2b
No known key found for this signature in database
GPG Key ID: 4340AEFE69F5085C
5 changed files with 317 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

98
meta-networking/recipes-daemons/keepalived/keepalived/CVE-2024-41184-1.patch Normal file
View File

@ -0,0 +1,98 @@
From 3ff3427da746c4641e1718dcec7a2f7041ae6961 Mon Sep 17 00:00:00 2001
From: Quentin Armitage <quentin@armitage.org.uk>
Date: Fri, 12 Jul 2024 15:16:47 +0100
Subject: [PATCH] vrrp: Handle empty ipset names with vrrp_ipsets keyword
We now handle empty ipset names and return a config error.
Signed-off-by: Quentin Armitage <quentin@armitage.org.uk>
CVE: CVE-2024-41184
Upstream-Status: Backport [https://github.com/acassen/keepalived/commit/e78513fe0ce5d83c226ea2c0bd222f375c2438e7]
Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
---
keepalived/core/global_parser.c | 36 +++++++++++++++++++--------------
1 file changed, 21 insertions(+), 15 deletions(-)
diff --git a/keepalived/core/global_parser.c b/keepalived/core/global_parser.c
index a3f2329..504b9a1 100644
--- a/keepalived/core/global_parser.c
+++ b/keepalived/core/global_parser.c
@@ -1086,6 +1086,22 @@ vrrp_iptables_handler(const vector_t *strvec)
}
}
#ifdef _HAVE_LIBIPSET_
+static bool
+check_valid_ipset_name(const vector_t *strvec, unsigned entry, const char *log_name)
+{
+ if (strlen(strvec_slot(strvec, entry)) >= IPSET_MAXNAMELEN - 1) {
+ report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset %s name too long - ignored", log_name);
+ return false;
+ }
+
+ if (strlen(strvec_slot(strvec, entry)) == 0) {
+ report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset %s name empty - ignored", log_name);
+ return false;
+ }
+
+ return true;
+}
+
static void
vrrp_ipsets_handler(const vector_t *strvec)
{
@@ -1103,17 +1119,13 @@ vrrp_ipsets_handler(const vector_t *strvec)
return;
}
- if (strlen(strvec_slot(strvec,1)) >= IPSET_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset address name too long - ignored");
+ if (!check_valid_ipset_name(strvec, 1, "address"))
return;
- }
global_data->vrrp_ipset_address = STRDUP(strvec_slot(strvec,1));
if (vector_size(strvec) >= 3) {
- if (strlen(strvec_slot(strvec,2)) >= IPSET_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset IPv6 address name too long - ignored");
+ if (!check_valid_ipset_name(strvec, 2, "IPv6 address"))
return;
- }
global_data->vrrp_ipset_address6 = STRDUP(strvec_slot(strvec,2));
}
else {
@@ -1124,10 +1136,8 @@ vrrp_ipsets_handler(const vector_t *strvec)
global_data->vrrp_ipset_address6 = STRDUP(set_name);
}
if (vector_size(strvec) >= 4) {
- if (strlen(strvec_slot(strvec,3)) >= IPSET_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset IPv6 address_iface name too long - ignored");
+ if (!check_valid_ipset_name(strvec, 3, "IPv6 address_iface"))
return;
- }
global_data->vrrp_ipset_address_iface6 = STRDUP(strvec_slot(strvec,3));
}
else {
@@ -1142,10 +1152,8 @@ vrrp_ipsets_handler(const vector_t *strvec)
}
if (vector_size(strvec) >= 5) {
- if (strlen(strvec_slot(strvec,4)) >= IPSET_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset IGMP name too long - ignored");
+ if (!check_valid_ipset_name(strvec, 4, "IGMP"))
return;
- }
global_data->vrrp_ipset_igmp = STRDUP(strvec_slot(strvec,4));
}
else {
@@ -1156,10 +1164,8 @@ vrrp_ipsets_handler(const vector_t *strvec)
global_data->vrrp_ipset_igmp = STRDUP(set_name);
}
if (vector_size(strvec) >= 6) {
- if (strlen(strvec_slot(strvec,5)) >= IPSET_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset MLD name too long - ignored");
+ if (!check_valid_ipset_name(strvec, 5, "MLD"))
return;
- }
global_data->vrrp_ipset_mld = STRDUP(strvec_slot(strvec,5));
}
else {

88
meta-networking/recipes-daemons/keepalived/keepalived/CVE-2024-41184-2.patch Normal file
View File

@ -0,0 +1,88 @@
From 19e9f69359c204805de4d1467d9f96940910a7dd Mon Sep 17 00:00:00 2001
From: Quentin Armitage <quentin@armitage.org.uk>
Date: Fri, 12 Jul 2024 15:18:20 +0100
Subject: [PATCH] vrrp: handle empty iptables chain names - vrrp_iptables
keyword
We now return an error if a chain name is empty.
Signed-off-by: Quentin Armitage <quentin@armitage.org.uk>
CVE: CVE-2024-41184
Upstream-Status: Backport [https://github.com/acassen/keepalived/commit/281de3aa8a0990fa3cd694a9addc0bf28953da0b]
Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
---
keepalived/core/global_parser.c | 42 ++++++++++++++++++++-------------
1 file changed, 25 insertions(+), 17 deletions(-)
diff --git a/keepalived/core/global_parser.c b/keepalived/core/global_parser.c
index 504b9a1..c49e771 100644
--- a/keepalived/core/global_parser.c
+++ b/keepalived/core/global_parser.c
@@ -1059,6 +1059,28 @@ vrrp_higher_prio_send_advert_handler(const vector_t *strvec)
global_data->vrrp_higher_prio_send_advert = true;
}
#ifdef _WITH_IPTABLES_
+static bool
+check_valid_iptables_ipset_name(const vector_t *strvec, unsigned entry, unsigned max_len, const char *type_name, const char *log_name)
+{
+ if (strlen(strvec_slot(strvec, entry)) >= max_len - 1) {
+ report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : %s %s name too long - ignored", type_name, log_name);
+ return false;
+ }
+
+ if (strlen(strvec_slot(strvec, entry)) == 0) {
+ report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : %s %s name empty - ignored", type_name, log_name);
+ return false;
+ }
+
+ return true;
+}
+
+static bool
+check_valid_iptables_chain_name(const vector_t *strvec, unsigned entry, const char *log_name)
+{
+ return check_valid_iptables_ipset_name(strvec, entry, XT_EXTENSION_MAXNAMELEN, "iptables", log_name);
+}
+
static void
vrrp_iptables_handler(const vector_t *strvec)
{
@@ -1068,16 +1090,12 @@ vrrp_iptables_handler(const vector_t *strvec)
}
if (vector_size(strvec) >= 2) {
- if (strlen(strvec_slot(strvec,1)) >= XT_EXTENSION_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : iptables in chain name too long - ignored");
+ if (!check_valid_iptables_chain_name(strvec, 1, "in chain"))
return;
- }
global_data->vrrp_iptables_inchain = STRDUP(strvec_slot(strvec,1));
if (vector_size(strvec) >= 3) {
- if (strlen(strvec_slot(strvec,2)) >= XT_EXTENSION_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : iptables out chain name too long - ignored");
+ if (!check_valid_iptables_chain_name(strvec, 2, "out chain"))
return;
- }
global_data->vrrp_iptables_outchain = STRDUP(strvec_slot(strvec,2));
}
} else {
@@ -1089,17 +1107,7 @@ vrrp_iptables_handler(const vector_t *strvec)
static bool
check_valid_ipset_name(const vector_t *strvec, unsigned entry, const char *log_name)
{
- if (strlen(strvec_slot(strvec, entry)) >= IPSET_MAXNAMELEN - 1) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset %s name too long - ignored", log_name);
- return false;
- }
-
- if (strlen(strvec_slot(strvec, entry)) == 0) {
- report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : ipset %s name empty - ignored", log_name);
- return false;
- }
-
- return true;
+ return check_valid_iptables_ipset_name(strvec, entry, IPSET_MAXNAMELEN, "ipset", log_name);
}
static void

94
meta-networking/recipes-daemons/keepalived/keepalived/CVE-2024-41184-3.patch Normal file
View File

@ -0,0 +1,94 @@
From 87f8c80bbbf9ceb3340cc89720fe9aa2284fad3a Mon Sep 17 00:00:00 2001
From: Quentin Armitage <quentin@armitage.org.uk>
Date: Fri, 12 Jul 2024 15:32:35 +0100
Subject: [PATCH] vrrp and ipvs: handle empty nftables chain names
We now return an error if a chain name is empty.
Signed-off-by: Quentin Armitage <quentin@armitage.org.uk>
CVE: CVE-2024-41184
Upstream-Status: Backport [https://github.com/acassen/keepalived/commit/1e5902c4793ac01b810f0faa3b5cf47b41ae95c1]
Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
---
keepalived/core/global_parser.c | 25 +++++++++++++++----------
1 file changed, 15 insertions(+), 10 deletions(-)
diff --git a/keepalived/core/global_parser.c b/keepalived/core/global_parser.c
index c49e771..1a145f9 100644
--- a/keepalived/core/global_parser.c
+++ b/keepalived/core/global_parser.c
@@ -1058,9 +1058,10 @@ vrrp_higher_prio_send_advert_handler(const vector_t *strvec)
else
global_data->vrrp_higher_prio_send_advert = true;
}
-#ifdef _WITH_IPTABLES_
+
+#if defined _WITH_IPTABLES_ || defined _WITH_NFTABLES_
static bool
-check_valid_iptables_ipset_name(const vector_t *strvec, unsigned entry, unsigned max_len, const char *type_name, const char *log_name)
+check_valid_iptables_ipset_nftables_name(const vector_t *strvec, unsigned entry, unsigned max_len, const char *type_name, const char *log_name)
{
if (strlen(strvec_slot(strvec, entry)) >= max_len - 1) {
report_config_error(CONFIG_GENERAL_ERROR, "VRRP Error : %s %s name too long - ignored", type_name, log_name);
@@ -1074,11 +1075,13 @@ check_valid_iptables_ipset_name(const vector_t *strvec, unsigned entry, unsigned
return true;
}
+#endif
+#ifdef _WITH_IPTABLES_
static bool
check_valid_iptables_chain_name(const vector_t *strvec, unsigned entry, const char *log_name)
{
- return check_valid_iptables_ipset_name(strvec, entry, XT_EXTENSION_MAXNAMELEN, "iptables", log_name);
+ return check_valid_iptables_ipset_nftables_name(strvec, entry, XT_EXTENSION_MAXNAMELEN, "iptables", log_name);
}
static void
@@ -1107,7 +1110,7 @@ vrrp_iptables_handler(const vector_t *strvec)
static bool
check_valid_ipset_name(const vector_t *strvec, unsigned entry, const char *log_name)
{
- return check_valid_iptables_ipset_name(strvec, entry, IPSET_MAXNAMELEN, "ipset", log_name);
+ return check_valid_iptables_ipset_nftables_name(strvec, entry, IPSET_MAXNAMELEN, "ipset", log_name);
}
static void
@@ -1201,6 +1204,12 @@ vrrp_iptables_handler(__attribute__((unused)) const vector_t *strvec)
#ifdef _WITH_NFTABLES_
#ifdef _WITH_VRRP_
+static bool
+check_valid_nftables_chain_name(const vector_t *strvec, unsigned entry, const char *log_name)
+{
+ return check_valid_iptables_ipset_nftables_name(strvec, entry, NFT_TABLE_MAXNAMELEN, "nftables", log_name);
+}
+
static void
vrrp_nftables_handler(__attribute__((unused)) const vector_t *strvec)
{
@@ -1212,10 +1221,8 @@ vrrp_nftables_handler(__attribute__((unused)) const vector_t *strvec)
}
if (vector_size(strvec) >= 2) {
- if (strlen(strvec_slot(strvec, 1)) >= NFT_TABLE_MAXNAMELEN) {
- report_config_error(CONFIG_GENERAL_ERROR, "nftables table name too long - ignoring");
+ if (!check_valid_nftables_chain_name(strvec, 1, "chain"))
return;
- }
name = strvec_slot(strvec, 1);
}
else {
@@ -1255,10 +1262,8 @@ ipvs_nftables_handler(__attribute__((unused)) const vector_t *strvec)
}
if (vector_size(strvec) >= 2) {
- if (strlen(strvec_slot(strvec, 1)) >= NFT_TABLE_MAXNAMELEN) {
- report_config_error(CONFIG_GENERAL_ERROR, "ipvs nftables table name too long - ignoring");
+ if (!check_valid_nftables_chain_name(strvec, 1, "ipvs chain"))
return;
- }
name = strvec_slot(strvec, 1);
}
else {

33
meta-networking/recipes-daemons/keepalived/keepalived/CVE-2024-41184-4.patch Normal file
View File

@ -0,0 +1,33 @@
From 5e5fde051d75ffe0449418466832b8b50baeb95e Mon Sep 17 00:00:00 2001
From: Quentin Armitage <quentin@armitage.org.uk>
Date: Fri, 12 Jul 2024 15:11:13 +0100
Subject: [PATCH] lib: don't return subtracted addresses for rb_find() compare
function
If sizeof(int) < sizeof(void *) returning the difference between two
addresses in an int can cause an overflow.
Use less_equal_greater_than() for comparing addresses.
Signed-off-by: Quentin Armitage <quentin@armitage.org.uk>
CVE: CVE-2024-41184
Upstream-Status: Backport [https://github.com/acassen/keepalived/commit/f3a32e3557520dccb298b36b4952eff3e236fb86]
Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
---
lib/memory.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/memory.c b/lib/memory.c
index c7217fd..4b250ac 100644
--- a/lib/memory.c
+++ b/lib/memory.c
@@ -200,7 +200,7 @@ static unsigned free_list_size;
static inline int
memcheck_ptr_cmp(const void *key, const struct rb_node *a)
{
- return (const char *)key - (char *)rb_entry_const(a, MEMCHECK, t)->ptr;
+ return less_equal_greater_than((const char *)key, (char *)rb_entry_const(a, MEMCHECK, t)->ptr);
}
static inline bool

4
meta-networking/recipes-daemons/keepalived/keepalived_2.2.8.bb
View File

@ -12,6 +12,10 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
SRC_URI = "http://www.keepalived.org/software/${BP}.tar.gz \
file://0001-configure.ac-Do-not-emit-compiler-flags-into-object-.patch \
file://CVE-2024-41184-1.patch \
file://CVE-2024-41184-2.patch \
file://CVE-2024-41184-3.patch \
file://CVE-2024-41184-4.patch \
"
SRC_URI[sha256sum] = "85882eb62974f395d4c631be990a41a839594a7e62fbfebcb5649a937a7a1bb6"
UPSTREAM_CHECK_URI = "https://github.com/acassen/keepalived/releases"