Blame SOURCES/0017-netlink-fix-memory-leak.patch
|
 |
c96cf6 |
From f1bb0c6263ca11bedea9e18bc9753c6b1d4c924d Mon Sep 17 00:00:00 2001
|
|
|
c96cf6 |
From: Ivan Vecera <cera@cera.cz>
|
|
|
c96cf6 |
Date: Thu, 24 Sep 2020 20:10:31 +0200
|
|
|
c96cf6 |
Subject: [PATCH 17/17] netlink: fix memory leak
|
|
|
c96cf6 |
|
|
|
c96cf6 |
Potentially allocated memory allocated for mask is not freed when
|
|
|
c96cf6 |
the allocation for value fails.
|
|
|
c96cf6 |
|
|
|
c96cf6 |
Fixes: 81a30f416ec7 ("netlink: add bitset command line parser handlers")
|
|
|
c96cf6 |
|
|
|
c96cf6 |
Cc: Michal Kubecek <mkubecek@suse.cz>
|
|
|
c96cf6 |
Signed-off-by: Ivan Vecera <cera@cera.cz>
|
|
|
c96cf6 |
---
|
|
|
c96cf6 |
netlink/parser.c | 4 +++-
|
|
|
c96cf6 |
1 file changed, 3 insertions(+), 1 deletion(-)
|
|
|
c96cf6 |
|
|
|
c96cf6 |
diff --git a/netlink/parser.c b/netlink/parser.c
|
|
|
c96cf6 |
index c5a368a65a7a..3b25f5d5a88e 100644
|
|
|
c96cf6 |
--- a/netlink/parser.c
|
|
|
c96cf6 |
+++ b/netlink/parser.c
|
|
|
c96cf6 |
@@ -630,8 +630,10 @@ static int parse_numeric_bitset(struct nl_context *nlctx, uint16_t type,
|
|
|
c96cf6 |
}
|
|
|
c96cf6 |
|
|
|
c96cf6 |
value = calloc(nwords, sizeof(uint32_t));
|
|
|
c96cf6 |
- if (!value)
|
|
|
c96cf6 |
+ if (!value) {
|
|
|
c96cf6 |
+ free(mask);
|
|
|
c96cf6 |
return -ENOMEM;
|
|
|
c96cf6 |
+ }
|
|
|
c96cf6 |
ret = __parse_num_string(arg, len1, value, force_hex1);
|
|
|
c96cf6 |
if (ret < 0) {
|
|
|
c96cf6 |
parser_err_invalid_value(nlctx, arg);
|
|
|
c96cf6 |
--
|
|
|
c96cf6 |
2.26.2
|
|
|
c96cf6 |
|