diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2016-11-07 10:55:25 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2016-11-11 10:03:48 +0100 |
| commit | 0649433da53c7165f839e24e889e131e2894dd32 (patch) | |
| tree | 7e516c1702fe87c09f190e5dc47ecd3a9bede1b8 /lib/escape.c | |
| parent | cdfda3ee827da069f1871722278fd82e7cbb4194 (diff) | |
| download | curl-0649433da53c7165f839e24e889e131e2894dd32.tar.gz | |
realloc: use Curl_saferealloc to avoid common mistakes
Discussed: https://curl.haxx.se/mail/lib-2016-11/0087.html
Diffstat (limited to 'lib/escape.c')
| -rw-r--r-- | lib/escape.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/lib/escape.c b/lib/escape.c index 66570076e..9fb8d3e15 100644 --- a/lib/escape.c +++ b/lib/escape.c @@ -31,6 +31,7 @@ #include "warnless.h" #include "non-ascii.h" #include "escape.h" +#include "strdup.h" /* The last 3 #include files should be in this order */ #include "curl_printf.h" #include "curl_memory.h" @@ -109,11 +110,9 @@ char *curl_easy_escape(struct Curl_easy *data, const char *string, newlen += 2; /* the size grows with two, since this'll become a %XX */ if(newlen > alloc) { alloc *= 2; - testing_ptr = realloc(ns, alloc); - if(!testing_ptr) { - free(ns); + testing_ptr = Curl_saferealloc(ns, alloc); + if(!testing_ptr) return NULL; - } else { ns = testing_ptr; } |