From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Webb Subject: [PATCH] Fix SIGFPE for vnc display of width/height = 1 Date: Mon, 8 Mar 2010 14:34:49 +0000 Message-ID: <20100308143449.GA4084@arachsys.com> References: <20100306085332.GF7180@arachsys.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii To: qemu-devel@nongnu.org, kvm@vger.kernel.org Return-path: Received: from alpha.arachsys.com ([91.203.57.7]:44173 "EHLO alpha.arachsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754425Ab0CHOew (ORCPT ); Mon, 8 Mar 2010 09:34:52 -0500 Content-Disposition: inline In-Reply-To: <20100306085332.GF7180@arachsys.com> Sender: kvm-owner@vger.kernel.org List-ID: 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 --- vnc.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/vnc.c b/vnc.c index 01353a9..676a707 100644 --- a/vnc.c +++ b/vnc.c @@ -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; -- 1.7.0.1