Re: [Qemu-block] [PATCH 1/2] blockdev: Fix regression with the default naming of throttling groups

[Alberto Garcia]

On Fri 08 Jul 2016 04:23:03 PM CEST, Max Reitz wrote:
>> +    blk_id = qemu_opts_id(opts);
>> +
>
> Side note: The "id" variable is supposed to contain the exact same
> value, but the string it points to is invalidated by the
> qdict_del(bs_opts, "id") call.
>
> So indeed we need to obtain the ID anew here (or we'd have to do
> g_strdup() before and g_free() after, which is cumbersome). But
> regarding the variable itself, you could have actually reused "id" (I
> only noticed that just now).
>
> But that's just a minor thing, so:
>
> Reviewed-by: Max Reitz <address@hidden>

Ok, if this series doesn't get a new version feel free to modify the
patch when you apply it.

Berto