|
From: | Anthony Liguori |
Subject: | Re: [Qemu-devel] KVM call minutes for Feb 15 |
Date: | Wed, 16 Feb 2011 07:34:17 -0600 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101027 Lightning/1.0b1 Thunderbird/3.0.10 |
On 02/16/2011 04:24 AM, Avi Kivity wrote:
On 02/16/2011 01:13 AM, Anthony Liguori wrote:On 02/15/2011 10:26 AM, Chris Wright wrote:QAPI and QMP - Anthony adding a new wiki page to describe all of thishttp://wiki.qemu.org/Features/QAPI[ 'change', {'device': 'str', 'target': 'str'}, {'arg': 'str'}, 'none' ]->void qmp_change(const char *device, const char *target, bool has_arg, const char *arg, Error **errp);AFAICT a json-string allows embedded NULs ('\0000'). There translate to UTF-8 as '\0', terminating your char *s. Either we use some length/pointer structure, or the parser has to look for them and kill them, and we have to specify them as verboten.
I feel like it would be safer for us to not accept strings with embedded NULs. There's no way we're going to consistently handle this correctly in QEMU since we expect NUL terminated strings. They won't work for any of the standard C functions either.
BlockDeviceInfo *qmp_query_block_device_info(const char *device, Error **errp){ BlockDeviceInfo *info; BlockDriverState *bs; Error *local_err = NULL; bs = bdrv_find(device,&local_err); if (local_err) { error_propagate(errp, local_err); return NULL; } info->file = qemu_strdup(bs->filename); info->ro = bs->readonly; info->drv = qemu_strdup(bs->drv); info->encrypted = bs->encrypted; if (bs->backing_file[0]) { info->has_backing_file = true; info->backing_file = qemu_strdup(info->backing_file); } return info; }So, info and all its pointer-typed members are required to be qemu_free() compatible, with just a single pointer pointing to an object, and generated code will qemu_free() everything?
Yes.
Recommend translating '-' in identifiers to '_' so we can use '-' in the schema as a word separator.
Already do that and we make extensive use of that in the schema. Regards, Anthony Liguori
[Prev in Thread] | Current Thread | [Next in Thread] |