diff options
| author | Daniel P. Berrangé | 2018-02-05 12:49:37 +0100 |
|---|---|---|
| committer | Gerd Hoffmann | 2018-02-16 12:33:02 +0100 |
| commit | cf0706581bc0c24ab2e9a81ff0fc3efa9482c812 (patch) | |
| tree | e3cb993a75f3bc55d57c64ab8a884418edc547f6 /scripts/qapi | |
| parent | ui: avoid 'local_err' variable shadowing in VNC SASL auth (diff) | |
| download | qemu-cf0706581bc0c24ab2e9a81ff0fc3efa9482c812.tar.gz qemu-cf0706581bc0c24ab2e9a81ff0fc3efa9482c812.tar.xz qemu-cf0706581bc0c24ab2e9a81ff0fc3efa9482c812.zip | |
ui: check VNC audio frequency limit at time of reading from client
The 'vs->as.freq' value is a signed integer, which is read from an
unsigned 32-bit int field on the wire. There is thus a risk of overflow
on 32-bit platforms. Move the frequency limit checking to be done at
time of read before casting to a signed integer.
Reported-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-id: 20180205114938.15784-4-berrange@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Diffstat (limited to 'scripts/qapi')
0 files changed, 0 insertions, 0 deletions