sysstat: fix CVE-2019-19725

(From OE-Core rev: 2bf6fdc460526feb0955b3b789e7be78f6e8ec4a)

(From OE-Core rev: 66c428a676acd4dd2c7c7589ac651faef77c1323)

Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Anuj Mittal
2019-12-12 07:10:42 +08:00
committed by Richard Purdie
parent e61dfe872e
commit 0a9ba3a53b
2 changed files with 29 additions and 0 deletions

View File

@@ -0,0 +1,28 @@
From a5c8abd4a481ee6e27a3acf00e6d9b0f023e20ed Mon Sep 17 00:00:00 2001
From: Sebastien GODARD <sysstat@users.noreply.github.com>
Date: Mon, 9 Dec 2019 17:54:07 +0100
Subject: [PATCH] Fix #242: Double free in check_file_actlst()
Avoid freeing buffer() twice.
Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>
Upstream-Status: Backport [https://github.com/sysstat/sysstat/commit/a5c8abd4a481ee6e27a3acf00e6d9b0f023e20ed]
CVE: CVE-2019-19725
Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
---
sa_common.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sa_common.c b/sa_common.c
index cf52aefe..856a3715 100644
--- a/sa_common.c
+++ b/sa_common.c
@@ -2153,6 +2153,7 @@ void check_file_actlst(int *ifd, char *dfile, struct activity *act[], uint64_t f
}
free(buffer);
+ buffer = NULL;
/* Check that at least one activity selected by the user is available in file */
for (i = 0; i < NR_ACT; i++) {

View File

@@ -4,6 +4,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=a23a74b3f4caf9616230789d94217acb"
SRC_URI += "file://0001-Include-needed-headers-explicitly.patch \
file://0001-Fix-232-Memory-corruption-bug-due-to-Integer-Overflo.patch \
file://CVE-2019-19725.patch \
"
SRC_URI[md5sum] = "d8e3bbb9c873dd370f6d33664e326570"