[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] vnc: Clean up vncws_send_handshake_response()
From: |
Tim Hardeck |
Subject: |
Re: [Qemu-trivial] [PATCH] vnc: Clean up vncws_send_handshake_response() |
Date: |
Thu, 24 Jan 2013 16:51:46 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
Hi Markus,
On 01/23/2013 06:16 PM, Markus Armbruster wrote:
> Use appropriate types, drop superfluous casts, use sizeof, don't
> exploit that this particular call of gnutls_fingerprint() doesn't
> change its last argument.
>
> Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Tim Hardeck <address@hidden>
Regards
Tim
> ---
> ui/vnc-ws.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/ui/vnc-ws.c b/ui/vnc-ws.c
> index 9ccdc19..de7e74c 100644
> --- a/ui/vnc-ws.c
> +++ b/ui/vnc-ws.c
> @@ -120,8 +120,8 @@ static char *vncws_extract_handshake_entry(const char
> *handshake,
> static void vncws_send_handshake_response(VncState *vs, const char* key)
> {
> char combined_key[WS_CLIENT_KEY_LEN + WS_GUID_LEN + 1];
> - char hash[SHA1_DIGEST_LEN];
> - size_t hash_size = SHA1_DIGEST_LEN;
> + unsigned char hash[SHA1_DIGEST_LEN];
> + size_t hash_size = sizeof(hash);
> char *accept = NULL, *response = NULL;
> gnutls_datum_t in;
>
> @@ -133,7 +133,7 @@ static void vncws_send_handshake_response(VncState *vs,
> const char* key)
> in.size = WS_CLIENT_KEY_LEN + WS_GUID_LEN;
> if (gnutls_fingerprint(GNUTLS_DIG_SHA1, &in, hash, &hash_size)
> == GNUTLS_E_SUCCESS) {
> - accept = g_base64_encode((guchar *)hash, SHA1_DIGEST_LEN);
> + accept = g_base64_encode(hash, hash_size);
> }
> if (accept == NULL) {
> VNC_DEBUG("Hashing Websocket combined key failed\n");
>
--
SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix
Imendörffer, HRB 16746 (AG Nürnberg)
Maxfeldstr. 5, 90409 Nürnberg, Germany
T: +49 (0) 911 74053-0 F: +49 (0) 911 74053-483
http://www.suse.de/
signature.asc
Description: OpenPGP digital signature