On Wed,  7 Jan 2015 14:03:29 +0100
David Marchand <david.marchand at 6wind.com> wrote:

> +     buf = strdup(devargs_str);
> +     if (buf == NULL) {
> +             RTE_LOG(ERR, EAL, "cannot allocate temp memory for devargs\n");
> +             goto fail;
> +     }
> +

If string is only used in same function you might consider using strdupa() 
which avoids
worrying about freeing in error paths.

Reply via email to