ryantimwilson / rpms / systemd

Forked from rpms/systemd a month ago
Clone
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;