* [Qemu-devel] [6390] re-fix screendump (Stefano Stabellini)
@ 2009-01-21 19:18 Anthony Liguori
0 siblings, 0 replies; only message in thread
From: Anthony Liguori @ 2009-01-21 19:18 UTC (permalink / raw)
To: qemu-devel
Revision: 6390
http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=6390
Author: aliguori
Date: 2009-01-21 19:18:00 +0000 (Wed, 21 Jan 2009)
Log Message:
-----------
re-fix screendump (Stefano Stabellini)
Removing the assumption about a single graphic console made
get_graphic_console return NULL when called by vga_screen_dump.
In this case returning NULL is correct but since NULL is not handled in
qemu_console_resize it causes a segmentation fault.
Just returning immediately from qemu_console_resize is sufficient to fix the
problem.
Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Modified Paths:
--------------
trunk/console.c
Modified: trunk/console.c
===================================================================
--- trunk/console.c 2009-01-21 18:59:12 UTC (rev 6389)
+++ trunk/console.c 2009-01-21 19:18:00 UTC (rev 6390)
@@ -1431,6 +1431,8 @@
void qemu_console_resize(DisplayState *ds, int width, int height)
{
TextConsole *s = get_graphic_console(ds);
+ if (!s) return;
+
s->g_width = width;
s->g_height = height;
if (is_graphic_console()) {
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2009-01-21 19:18 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-21 19:18 [Qemu-devel] [6390] re-fix screendump (Stefano Stabellini) Anthony Liguori
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.