summaryrefslogtreecommitdiffstats
path: root/ui
diff options
context:
space:
mode:
authorDaniel P. Berrange2015-05-01 12:44:46 +0200
committerGerd Hoffmann2015-05-20 10:23:08 +0200
commit2b2c1a38eeaba5d8bfe92281e9e680361e09ee3b (patch)
treee41b8764c5e29ca94a6f5d2810485d082c9c0355 /ui
parentStrip brackets from vnc host (diff)
downloadqemu-2b2c1a38eeaba5d8bfe92281e9e680361e09ee3b.tar.gz
qemu-2b2c1a38eeaba5d8bfe92281e9e680361e09ee3b.tar.xz
qemu-2b2c1a38eeaba5d8bfe92281e9e680361e09ee3b.zip
ui: remove check for failure of qemu_acl_init()
The qemu_acl_init() function has long since stopped being able to return NULL, since g_malloc will abort on OOM. As such the checks for NULL were unreachable code. Signed-off-by: Daniel P. Berrange <berrange@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Diffstat (limited to 'ui')
-rw-r--r--ui/vnc.c8
1 files changed, 0 insertions, 8 deletions
diff --git a/ui/vnc.c b/ui/vnc.c
index 2dccae96c6..076355d9c5 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -3609,10 +3609,6 @@ void vnc_display_open(const char *id, Error **errp)
aclname = g_strdup_printf("vnc.%s.x509dname", vs->id);
}
vs->tls.acl = qemu_acl_init(aclname);
- if (!vs->tls.acl) {
- fprintf(stderr, "Failed to create x509 dname ACL\n");
- exit(1);
- }
g_free(aclname);
}
#endif
@@ -3626,10 +3622,6 @@ void vnc_display_open(const char *id, Error **errp)
aclname = g_strdup_printf("vnc.%s.username", vs->id);
}
vs->sasl.acl = qemu_acl_init(aclname);
- if (!vs->sasl.acl) {
- fprintf(stderr, "Failed to create username ACL\n");
- exit(1);
- }
g_free(aclname);
}
#endif