diff options
| author | Michael S. Tsirkin | 2014-04-03 18:52:13 +0200 |
|---|---|---|
| committer | Juan Quintela | 2014-05-05 22:15:02 +0200 |
| commit | 52f91c3723932f8340fe36c8ec8b18a757c37b2b (patch) | |
| tree | f331a24f6d414648c3994cc0a25e3bc6ae3059e4 /include/exec | |
| parent | tsc210x: fix buffer overrun on invalid state load (diff) | |
| download | qemu-52f91c3723932f8340fe36c8ec8b18a757c37b2b.tar.gz qemu-52f91c3723932f8340fe36c8ec8b18a757c37b2b.tar.xz qemu-52f91c3723932f8340fe36c8ec8b18a757c37b2b.zip | |
zaurus: fix buffer overrun on invalid state load
CVE-2013-4540
Within scoop_gpio_handler_update, if prev_level has a high bit set, then
we get bit > 16 and that causes a buffer overrun.
Since prev_level comes from wire indirectly, this can
happen on invalid state load.
Similarly for gpio_level and gpio_dir.
To fix, limit to 16 bit.
Reported-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Diffstat (limited to 'include/exec')
0 files changed, 0 insertions, 0 deletions
