Re: [Qemu-devel] [patch] replace all strdup() with g_strdup()

[Peter Maydell]

On 16 December 2011 08:07, Jun Koi <address@hidden> wrote:
> This patch replaces all the strdup() with g_strdup()

I don't think you can do this as a pure search-and-replace.
For example this change:
> --- a/envlist.c
> +++ b/envlist.c
> @@ -109,7 +109,7 @@ envlist_parse(envlist_t *envlist, const char *env,
>        * We need to make temporary copy of the env string
>        * as strtok_r(3) modifies it while it tokenizes.
>        */
> -     if ((tmpenv = strdup(env)) == NULL)
> +     if ((tmpenv = g_strdup(env)) == NULL)
>               return (errno);
>
>       envvar = strtok_r(tmpenv, ",", &envsave);

means we're allocating memory with g_strdup but freeing it with
plain free rather than g_free.

You have to take each usage of memory, look for all the places which
allocate/free/strdup it, and change them all at once.

-- PMM