svn commit: r270390 - head/sys/dev/vt
Jean-Sebastien Pedron
dumbbell at FreeBSD.org
Sat Aug 23 07:41:09 UTC 2014
Author: dumbbell
Date: Sat Aug 23 07:41:08 2014
New Revision: 270390
URL: http://svnweb.freebsd.org/changeset/base/270390
Log:
vt(4): Fix a crash in vt_mark_mouse_position_as_dirty() when in textmode
In textmode, no font is loaded, thus the page fault in
vt_mark_mouse_position_as_dirty() when it wants the font width/height.
For now, create a fake area for the textmode. This needs to be modified
if vt_vga gains mouse support in textmode.
While here, fix a build failure when SC_NO_CUTPASTE is defined:
vt_mark_mouse_position_as_dirty() must not be included in this case.
MFC after: 1 week
Modified:
head/sys/dev/vt/vt_core.c
Modified: head/sys/dev/vt/vt_core.c
==============================================================================
--- head/sys/dev/vt/vt_core.c Sat Aug 23 07:03:04 2014 (r270389)
+++ head/sys/dev/vt/vt_core.c Sat Aug 23 07:41:08 2014 (r270390)
@@ -818,6 +818,7 @@ vt_determine_colors(term_char_t c, int c
}
}
+#ifndef SC_NO_CUTPASTE
static void
vt_mark_mouse_position_as_dirty(struct vt_device *vd, int x, int y)
{
@@ -828,17 +829,33 @@ vt_mark_mouse_position_as_dirty(struct v
vw = vd->vd_curwindow;
vf = vw->vw_font;
- area.tr_begin.tp_col = (x - vw->vw_offset.tp_col) / vf->vf_width;
- area.tr_begin.tp_row = (y - vw->vw_offset.tp_row) / vf->vf_height;
- area.tr_end.tp_col =
- ((x + vd->vd_mcursor->width - vw->vw_offset.tp_col) /
- vf->vf_width) + 1;
- area.tr_end.tp_row =
- ((y + vd->vd_mcursor->height - vw->vw_offset.tp_row) /
- vf->vf_height) + 1;
+ if (vf != NULL) {
+ area.tr_begin.tp_col = (x - vw->vw_offset.tp_col) /
+ vf->vf_width;
+ area.tr_begin.tp_row = (y - vw->vw_offset.tp_row) /
+ vf->vf_height;
+ area.tr_end.tp_col =
+ ((x + vd->vd_mcursor->width - vw->vw_offset.tp_col) /
+ vf->vf_width) + 1;
+ area.tr_end.tp_row =
+ ((y + vd->vd_mcursor->height - vw->vw_offset.tp_row) /
+ vf->vf_height) + 1;
+ } else {
+ /*
+ * No font loaded (ie. vt_vga operating in textmode).
+ *
+ * FIXME: This fake area needs to be revisited once the
+ * mouse cursor is supported in vt_vga's textmode.
+ */
+ area.tr_begin.tp_col = x;
+ area.tr_begin.tp_row = y;
+ area.tr_end.tp_col = x + 2;
+ area.tr_end.tp_row = y + 2;
+ }
vtbuf_dirty(&vw->vw_buf, &area);
}
+#endif
static void
vt_bitblt_char(struct vt_device *vd, struct vt_font *vf, term_char_t c,
More information about the svn-src-all
mailing list