qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] input: free InputEvent when it can't be inserte


From: Marc-André Lureau
Subject: Re: [Qemu-devel] [PATCH] input: free InputEvent when it can't be inserted into a full kdb queue
Date: Wed, 6 Dec 2017 10:46:54 +0100

Hi

On Wed, Dec 6, 2017 at 3:29 AM, 田殿臣 <address@hidden> wrote:
> From e8c03f405c2112428e79bb82064c7b7743d0cc86 Mon Sep 17 00:00:00 2001
> From: Tian Dianchen <address@hidden>
> Date: Tue, 5 Dec 2017 14:03:53 +0800
> Subject: [PATCH] input: free InputEvent when it can't be inserted into a
> full
>  kdb queue
>
> When the kdb queue is full, the evt can't be placed in it, so it should
> be released to free the memory.
>
> Impact: Without this limit vnc clients can exhaust host memory by keep
> sending keyboard events when kdb queue is full.

You may add "Leak introduced in commit fa18f36a461984eae50ab957e47ec78dae3c14fc"

>
> Reviewed-by: Zhang Chao <address@hidden>
> Reviewed-by: Quan Xu <address@hidden>
> Signed-off-by: Tian Dianchen <address@hidden>

Reviewed-by: Marc-André Lureau <address@hidden>


> ---
>  ui/input.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/ui/input.c b/ui/input.c
> index 3e2d324..e5b78aa 100644
> --- a/ui/input.c
> +++ b/ui/input.c
> @@ -421,6 +421,8 @@ void qemu_input_event_send_key(QemuConsole *src,
> KeyValue *key, bool down)
>      } else if (queue_count < queue_limit) {
>          qemu_input_queue_event(&kbd_queue, src, evt);
>          qemu_input_queue_sync(&kbd_queue);
> +    } else {
> +        qapi_free_InputEvent(evt);
>      }
>  }
>
> --
> 1.8.3.1



-- 
Marc-André Lureau



reply via email to

[Prev in Thread] Current Thread [Next in Thread]