diff options
| author | Markus Armbruster <armbru@redhat.com> | 2017-03-06 20:00:44 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2017-03-07 14:53:28 +0100 |
| commit | fc29458dee2b4206eba90ed210342105e1556fd0 (patch) | |
| tree | 892c4fe84dd513a886086689aef6f9ce3beebecf /block/gluster.c | |
| parent | 831acdc95e9c0faaf0bd0ced50f168afd7ad92ac (diff) | |
| download | qemu-fc29458dee2b4206eba90ed210342105e1556fd0.tar.gz | |
gluster: Drop assumptions on SocketTransport names
qemu_gluster_glfs_init() passes the names of QAPI enumeration type
SocketTransport to glfs_set_volfile_server(). Works, because they
were chosen to match. But the coupling is artificial. Use the
appropriate literal strings instead.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Niels de Vos <ndevos@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/gluster.c')
| -rw-r--r-- | block/gluster.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/block/gluster.c b/block/gluster.c index 56b4abe3a7..7236d592ae 100644 --- a/block/gluster.c +++ b/block/gluster.c @@ -412,8 +412,7 @@ static struct glfs *qemu_gluster_glfs_init(BlockdevOptionsGluster *gconf, for (server = gconf->server; server; server = server->next) { if (server->value->type == GLUSTER_TRANSPORT_UNIX) { - ret = glfs_set_volfile_server(glfs, - GlusterTransport_lookup[server->value->type], + ret = glfs_set_volfile_server(glfs, "unix", server->value->u.q_unix.path, 0); } else { if (parse_uint_full(server->value->u.tcp.port, &port, 10) < 0 || @@ -423,8 +422,7 @@ static struct glfs *qemu_gluster_glfs_init(BlockdevOptionsGluster *gconf, errno = EINVAL; goto out; } - ret = glfs_set_volfile_server(glfs, - GlusterTransport_lookup[server->value->type], + ret = glfs_set_volfile_server(glfs, "tcp", server->value->u.tcp.host, (int)port); } |