|
 |
a60c34 |
From f73ad90e797824569008a054bea6c8215883a3a0 Mon Sep 17 00:00:00 2001
|
|
|
a60c34 |
From: Christos Zoulas <christos@zoulas.com>
|
|
|
a60c34 |
Date: Mon, 26 Aug 2019 14:31:39 +0000
|
|
|
a60c34 |
Subject: [PATCH] Limit the number of elements in a vector (found by oss-fuzz)
|
|
|
a60c34 |
|
|
|
a60c34 |
Upstream-commit: 46a8443f76cec4b41ec736eca396984c74664f84
|
|
|
a60c34 |
Signed-off-by: Kamil Dudka <kdudka@redhat.com>
|
|
|
a60c34 |
---
|
|
|
a60c34 |
src/cdf.c | 7 +++----
|
|
|
a60c34 |
src/cdf.h | 1 +
|
|
|
a60c34 |
2 files changed, 4 insertions(+), 4 deletions(-)
|
|
|
a60c34 |
|
|
|
a60c34 |
diff --git a/src/cdf.c b/src/cdf.c
|
|
|
a60c34 |
index 556a3ff..8bb0a6d 100644
|
|
|
a60c34 |
--- a/src/cdf.c
|
|
|
a60c34 |
+++ b/src/cdf.c
|
|
|
a60c34 |
@@ -1013,8 +1013,9 @@ cdf_read_property_info(const cdf_stream_t *sst, const cdf_header_t *h,
|
|
|
a60c34 |
goto out;
|
|
|
a60c34 |
}
|
|
|
a60c34 |
nelements = CDF_GETUINT32(q, 1);
|
|
|
a60c34 |
- if (nelements == 0) {
|
|
|
a60c34 |
- DPRINTF(("CDF_VECTOR with nelements == 0\n"));
|
|
|
a60c34 |
+ if (nelements > CDF_ELEMENT_LIMIT || nelements == 0) {
|
|
|
a60c34 |
+ DPRINTF(("CDF_VECTOR with nelements == %"
|
|
|
a60c34 |
+ SIZE_T_FORMAT "u\n", nelements));
|
|
|
a60c34 |
goto out;
|
|
|
a60c34 |
}
|
|
|
a60c34 |
slen = 2;
|
|
|
a60c34 |
@@ -1056,8 +1057,6 @@ cdf_read_property_info(const cdf_stream_t *sst, const cdf_header_t *h,
|
|
|
a60c34 |
goto out;
|
|
|
a60c34 |
inp += nelem;
|
|
|
a60c34 |
}
|
|
|
a60c34 |
- DPRINTF(("nelements = %" SIZE_T_FORMAT "u\n",
|
|
|
a60c34 |
- nelements));
|
|
|
a60c34 |
for (j = 0; j < nelements && i < sh.sh_properties;
|
|
|
a60c34 |
j++, i++)
|
|
|
a60c34 |
{
|
|
|
a60c34 |
diff --git a/src/cdf.h b/src/cdf.h
|
|
|
a60c34 |
index 2f7e554..0505666 100644
|
|
|
a60c34 |
--- a/src/cdf.h
|
|
|
a60c34 |
+++ b/src/cdf.h
|
|
|
a60c34 |
@@ -48,6 +48,7 @@
|
|
|
a60c34 |
typedef int32_t cdf_secid_t;
|
|
|
a60c34 |
|
|
|
a60c34 |
#define CDF_LOOP_LIMIT 10000
|
|
|
a60c34 |
+#define CDF_ELEMENT_LIMIT 100000
|
|
|
a60c34 |
|
|
|
a60c34 |
#define CDF_SECID_NULL 0
|
|
|
a60c34 |
#define CDF_SECID_FREE -1
|
|
|
a60c34 |
--
|
|
|
a60c34 |
2.20.1
|
|
|
a60c34 |
|