|
Zbigniew Jędrzejewski-Szmek |
126222 |
From b7b1af75b9f76f3bff35369b0c1666890c586144 Mon Sep 17 00:00:00 2001
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
From: Lennart Poettering <lennart@poettering.net>
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
Date: Tue, 10 Dec 2013 18:53:03 +0000
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
Subject: [PATCH] util: check for overflow in greedy_realloc()
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
Conflicts:
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
src/shared/util.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
---
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
src/shared/util.c | 8 ++++++++
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
1 file changed, 8 insertions(+)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
diff --git a/src/shared/util.c b/src/shared/util.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
index 1510c5e..02bae90 100644
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
--- a/src/shared/util.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+++ b/src/shared/util.c
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
@@ -5832,10 +5832,18 @@ void* greedy_realloc(void **p, size_t *allocated, size_t need) {
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
size_t a;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
void *q;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ assert(p);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ assert(allocated);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (*allocated >= need)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return *p;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
a = MAX(64u, need * 2);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ /* check for overflows */
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ if (a < need)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+ return NULL;
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
+
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
q = realloc(*p, a);
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
if (!q)
|
|
Zbigniew Jędrzejewski-Szmek |
126222 |
return NULL;
|