io: monitor encoutput buffer size from websocket GSource
commita7b20a8efa28e5f22c26c06cd06c2f12bc863493
authorDaniel P. Berrange <berrange@redhat.com>
Mon, 9 Oct 2017 13:43:42 +0000 (9 14:43 +0100)
committerDaniel P. Berrange <berrange@redhat.com>
Mon, 16 Oct 2017 15:57:08 +0000 (16 16:57 +0100)
tree9491f2cefa582584002726191257a42bb520948c
parent9cf961bba74d433db76a110917ac70aecc2ebcc4
io: monitor encoutput buffer size from websocket GSource

The websocket GSource is monitoring the size of the rawoutput
buffer to determine if the channel can accepts more writes.
The rawoutput buffer, however, is merely a temporary staging
buffer before data is copied into the encoutput buffer. Thus
its size will always be zero when the GSource runs.

This flaw causes the encoutput buffer to grow without bound
if the other end of the underlying data channel doesn't
read data being sent. This can be seen with VNC if a client
is on a slow WAN link and the guest OS is sending many screen
updates. A malicious VNC client can act like it is on a slow
link by playing a video in the guest and then reading data
very slowly, causing QEMU host memory to expand arbitrarily.

This issue is assigned CVE-2017-15268, publically reported in

  https://bugs.launchpad.net/qemu/+bug/1718964

Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
io/channel-websock.c