Re: [Qemu-trivial] [Qemu-devel] [PATCH] Fix crash when connecting to VNC through websocket

[Peter Crosthwaite]

The subject should read V2 as this is a resubmit. Use git format-patch
--subject-prefix="PATCH v2".

On Mon, Feb 23, 2015 at 4:29 PM, Jorge Acereda Maciá <address@hidden> wrote:
> Please, forget my previous patch, worked somehow with Chrome but failed with 
> Safari (with a good reason, the sent headers were incorrect). This one should 
> be correct and simpler.
>

You should also start a fresh with your commit message content. Don't
mention discarded versions of the patch or changes in your patch
content. Just say what the patch does. Your v1 commit message looks
good and coulld probably be reused as-is.

> Signed-off-by: Jorge Acereda Macia <address@hidden>
>
> ---

But you put you patch change log here, below the "---". This allows
reviewers to see your change information, but git will discard the
changlog on application of the final accepted version.

Regards,
Peter

>  ui/vnc-ws.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/ui/vnc-ws.c b/ui/vnc-ws.c
> index e304baf..d75950d 100644
> --- a/ui/vnc-ws.c
> +++ b/ui/vnc-ws.c
> @@ -207,8 +207,7 @@ static void vncws_send_handshake_response(VncState *vs, 
> const char* key)
>      }
>
>      response = g_strdup_printf(WS_HANDSHAKE, accept);
> -    vnc_write(vs, response, strlen(response));
> -    vnc_flush(vs);
> +    vnc_client_write_buf(vs, (const uint8_t *)response, strlen(response));
>
>      g_free(accept);
>      g_free(response);
> --
> 1.9.3 (Apple Git-50)
>
>
>