diff options
| author | Blue Swirl <blauwirbel@gmail.com> | 2012-07-08 06:56:53 +0000 |
|---|---|---|
| committer | Blue Swirl <blauwirbel@gmail.com> | 2012-07-28 09:23:11 +0000 |
| commit | 0ed8b6f67f26acc2e88dfc6be4954e63f943bd28 (patch) | |
| tree | 8490bfb39362e56f8238ba9208114764bc938d7e /ui | |
| parent | 16fd921bd3a266d321071739cf40785abce4bcb1 (diff) | |
| download | focaccia-qemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.tar.gz focaccia-qemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.zip | |
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.c | 3 | ||||
| -rw-r--r-- | ui/vnc.c | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/ui/vnc-auth-vencrypt.c b/ui/vnc-auth-vencrypt.c index 674ba97dc7..c59b188602 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 cf1cae2de4..cfc61a7756 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); } |