curl / Mailing Lists / curl-library / Single Mail

curl-library

Re: realloc: use Curl_saferealloc to avoid common mistakes

From: Kamil Dudka <kdudka_at_redhat.com>
Date: Tue, 08 Nov 2016 17:42:39 +0100

On Monday, November 07, 2016 15:16:36 Daniel Stenberg wrote:
> Hi team,
>
> I'm suggesting we introduce a Curl_saferealloc() function to be used instead
> of plain old realloc() where suitable.
>
> The replacement makes sure to free the passed in memory area if realloc()
> fails, but it does *not* free the memory area if the realloc'ed size is
> zero.
>
> The intent is to address a common problem pattern our code has shown in the
> past. We've allowed the size argument to get dynamically calculated and then
> passed to realloc() and in some cases it could wrap and become zero. A
> realloc() to 0 returns NULL *and* frees the memory area, which easily ends
> up in a free-twice scenario.
>
> This new function is meant to help us avoid this problem.
>
> The attached patch introduce the function and changes most uses of realloc()
> to use the new function. It didn't really fit in every case so there are
> still a handful calls to plain realloc() left.
>
> Comments or thoughts on this?

Is it intentional that the documentation of Curl_saferealloc() uses different
identifiers for parameters than the actual implementation of that function?

Otherwise it looks good to me.

Kamil
-------------------------------------------------------------------
List admin: https://cool.haxx.se/list/listinfo/curl-library
Etiquette: https://curl.haxx.se/mail/etiquette.html
Received on 2016-11-08