aboutsummaryrefslogtreecommitdiff
path: root/ui
diff options
context:
space:
mode:
authorBlue Swirl <blauwirbel@gmail.com>2012-07-08 06:56:53 +0000
committerBlue Swirl <blauwirbel@gmail.com>2012-07-28 09:23:11 +0000
commit0ed8b6f67f26acc2e88dfc6be4954e63f943bd28 (patch)
tree8490bfb39362e56f8238ba9208114764bc938d7e /ui
parent16fd921bd3a266d321071739cf40785abce4bcb1 (diff)
downloadqemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.zip
qemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.tar.gz
qemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.tar.bz2
Avoid returning void
It's silly and non-conforming to standards to return void, don't do it. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
Diffstat (limited to 'ui')
-rw-r--r--ui/vnc-auth-vencrypt.c3
-rw-r--r--ui/vnc.c2
2 files changed, 3 insertions, 2 deletions
diff --git a/ui/vnc-auth-vencrypt.c b/ui/vnc-auth-vencrypt.c
index 674ba97..c59b188 100644
--- a/ui/vnc-auth-vencrypt.c
+++ b/ui/vnc-auth-vencrypt.c
@@ -47,7 +47,8 @@ static void start_auth_vencrypt_subauth(VncState *vs)
case VNC_AUTH_VENCRYPT_TLSSASL:
case VNC_AUTH_VENCRYPT_X509SASL:
VNC_DEBUG("Start TLS auth SASL\n");
- return start_auth_sasl(vs);
+ start_auth_sasl(vs);
+ break;
#endif /* CONFIG_VNC_SASL */
default: /* Should not be possible, but just in case */
diff --git a/ui/vnc.c b/ui/vnc.c
index cf1cae2..cfc61a7 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -3089,5 +3089,5 @@ void vnc_display_add_client(DisplayState *ds, int csock, int skipauth)
{
VncDisplay *vs = ds ? (VncDisplay *)ds->opaque : vnc_display;
- return vnc_connect(vs, csock, skipauth);
+ vnc_connect(vs, csock, skipauth);
}