Commit fa0cfdf2 authored by Stefan Weil's avatar Stefan Weil Committed by Anthony Liguori
Browse files

vnc: Set invalid buffer pointers to NULL 



After qemu_free, the pointers for input and output
buffers are no longer valid, so set them to NULL
(most other calls of qemu_free in vnc.c use this
pattern, too).

Signed-off-by: default avatarStefan Weil <weil@mail.berlios.de>
Signed-off-by: default avatarAnthony Liguori <aliguori@us.ibm.com>
parent 7f1e9d4e

vnc.c

+8 −2
Original line number Diff line number Diff line
@@ -894,8 +894,14 @@ static void vnc_disconnect_start(VncState *vs) 


static void vnc_disconnect_finish(VncState *vs)

{

    if (vs->input.buffer) qemu_free(vs->input.buffer);

    if (vs->output.buffer) qemu_free(vs->output.buffer);

    if (vs->input.buffer) {

        qemu_free(vs->input.buffer);

        vs->input.buffer = NULL;

    }

    if (vs->output.buffer) {

        qemu_free(vs->output.buffer);

        vs->output.buffer = NULL;

    }

#ifdef CONFIG_VNC_TLS

    vnc_tls_client_cleanup(vs);

#endif /* CONFIG_VNC_TLS */