diff options
| author | Eric Blake <eblake@redhat.com> | 2015-10-26 16:34:56 -0600 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2015-11-02 08:30:27 +0100 |
| commit | 8d0bcba8370a4e8606dee602393a14d0c48e8bfc (patch) | |
| tree | 1df87a06cd1806f9216d51b51164ed82546a83f1 /net/vde.c | |
| parent | 2d32addae70987521578d8bb27c6b3f52cdcbdcb (diff) | |
| download | focaccia-qemu-8d0bcba8370a4e8606dee602393a14d0c48e8bfc.tar.gz focaccia-qemu-8d0bcba8370a4e8606dee602393a14d0c48e8bfc.zip | |
net: Convert to new qapi union layout
We have two issues with our qapi union layout: 1) Even though the QMP wire format spells the tag 'type', the C code spells it 'kind', requiring some hacks in the generator. 2) The C struct uses an anonymous union, which places all tag values in the same namespace as all non-variant members. This leads to spurious collisions if a tag value matches a non-variant member's name. Make the conversion to the new layout for net-related code. Signed-off-by: Eric Blake <eblake@redhat.com> Message-Id: <1445898903-12082-18-git-send-email-eblake@redhat.com> [Commit message tweaked slightly] Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'net/vde.c')
| -rw-r--r-- | net/vde.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/vde.c b/net/vde.c index dacaa64b47..4475d929e6 100644 --- a/net/vde.c +++ b/net/vde.c @@ -115,8 +115,8 @@ int net_init_vde(const NetClientOptions *opts, const char *name, /* FIXME error_setg(errp, ...) on failure */ const NetdevVdeOptions *vde; - assert(opts->kind == NET_CLIENT_OPTIONS_KIND_VDE); - vde = opts->vde; + assert(opts->type == NET_CLIENT_OPTIONS_KIND_VDE); + vde = opts->u.vde; /* missing optional values have been initialized to "all bits zero" */ if (net_vde_init(peer, "vde", name, vde->sock, vde->port, vde->group, |