diff options
author | Chris Webb <chris@arachsys.com> | 2010-03-08 14:34:49 +0000 |
---|---|---|
committer | Anthony Liguori <aliguori@us.ibm.com> | 2010-03-17 10:42:11 -0500 |
commit | cc39a92cbfc80c70d2b83708a4c9b309c3126ac3 (patch) | |
tree | dbab4f5553c9e9bf6428667b3c6ac49ddf082672 /vnc.c | |
parent | 3c9c706c3b66d838942aba53c0d3fdcdf06c7423 (diff) | |
download | qemu-cc39a92cbfc80c70d2b83708a4c9b309c3126ac3.tar.gz qemu-cc39a92cbfc80c70d2b83708a4c9b309c3126ac3.tar.bz2 qemu-cc39a92cbfc80c70d2b83708a4c9b309c3126ac3.zip |
Fix SIGFPE for vnc display of width/height = 1
During boot, the screen gets resized to height 1 and a mouse click at this
point will cause a division by zero when calculating the absolute pointer
position from the pixel (x, y). Return a click in the middle of the screen
instead in this case.
Signed-off-by: Chris Webb <chris@arachsys.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'vnc.c')
-rw-r--r-- | vnc.c | 6 |
1 files changed, 4 insertions, 2 deletions
@@ -1457,8 +1457,10 @@ static void pointer_event(VncState *vs, int button_mask, int x, int y) dz = 1; if (vs->absolute) { - kbd_mouse_event(x * 0x7FFF / (ds_get_width(vs->ds) - 1), - y * 0x7FFF / (ds_get_height(vs->ds) - 1), + kbd_mouse_event(ds_get_width(vs->ds) > 1 ? + x * 0x7FFF / (ds_get_width(vs->ds) - 1) : 0x4000, + ds_get_height(vs->ds) > 1 ? + y * 0x7FFF / (ds_get_height(vs->ds) - 1) : 0x4000, dz, buttons); } else if (vnc_has_feature(vs, VNC_FEATURE_POINTER_TYPE_CHANGE)) { x -= 0x7FFF; |