On 12/06/2015 14:57, Stefan Hajnoczi wrote:
> +    RockerSwitch *rocker = g_malloc0(sizeof(*rocker));
> +    Rocker *r;
> +
> +    r = rocker_find(name);
> +    if (!r) {
> +        error_set(errp, ERROR_CLASS_GENERIC_ERROR,
> +                  "rocker %s not found", name);
> +        return NULL;

Memory leak, please initialize rocker after the "if".

Paolo

> +    }
> +
> +    rocker->name = g_strdup(r->name);

Reply via email to