Re: [Qemu-devel] [PATCH 15/22] qapi: add new QMP server that uses CharDriverState

[Stefan Hajnoczi]

On Mon, Mar 7, 2011 at 1:22 AM, Anthony Liguori <address@hidden> wrote:
> +static void qmp_chr_send_greeting(QmpSession *s)
> +{
> +    VersionInfo *info;
> +    QObject *vers;
> +    QObject *greeting;
> +    QString *str;
> +
> +    info = qmp_query_version(NULL);
> +    vers = qmp_marshal_type_VersionInfo(info);
> +    qmp_free_version_info(info);
> +
> +    greeting = qobject_from_jsonf("{'QMP': {'version': %p, 'capabilities': 
> []} }",
> +                                  vers);
> +    str = qobject_to_json(greeting);
> +    qobject_decref(greeting);
> +
> +    qemu_chr_write(s->chr, (void *)str->string, str->length);
> +    qemu_chr_write(s->chr, (void *)"\n", 1);
> +    QDECREF(str);
> +}

Is vers leaked?

Stefan