Bug 94192 - spice server crashes when not using qxl
Summary: spice server crashes when not using qxl
Status: RESOLVED FIXED
Alias: None
Product: Spice
Classification: Unclassified
Component: server (show other bugs)
Version: unspecified
Hardware: Other Linux (All)
: medium normal
Assignee: Spice Bug List
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-02-17 15:40 UTC by Lukas Venhoda
Modified: 2016-02-19 10:08 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Script used to run QEMU (1.47 KB, text/plain)
2016-02-17 15:40 UTC, Lukas Venhoda
Details

Description Lukas Venhoda 2016-02-17 15:40:42 UTC
Created attachment 121809 [details]
Script used to run QEMU

When running qemu with upstream spice server, and without qxl enabled, qemu crashes. The crash was introduced with commit 1f210080609f2c00b4d1859eb0b363a38 "Remove use of global 'reds' from AgentMsgFilter" in spice server, specifically this part:

reds->agent_state.write_filter.use_client_monitors_config=
     reds_use_client_monitors_config(reds);
reds->agent_state.read_filter.use_client_monitors_config =
     reds_use_client_monitors_config(reds);

The crash happens in function dpy_ui_info_supported() from qemu ui/console.c, because QemuConsole *con == NULL.

The issue might be connected to commit aadb94557011af2466dd7e678a1de45415ce424a "Remove global 'dispatchers', 'num_active_workers' variables"

Callstack:
dpy_ui_info_supported(), qemu ui/console.c
interface_client_monitors_config(), qemu ui/spice-display.c
red_dispatcher_use_client_monitors_config(), spice server/red-dispatcher.c
reds_use_client_monitors_config(), spice server/reds.c

Script for runnign QEMU attached
Comment 1 Lukas Venhoda 2016-02-17 15:49:56 UTC
Even this causes the crash:
export LD_PRELOAD="/home/lvenhoda/jhbuild/src/spice/server/.libs/libspice-server.so"
qemu-system-x86_64 -spice disable-ticketing,port=5900

While this doesn't:
export LD_PRELOAD="/home/lvenhoda/jhbuild/src/spice/server/.libs/libspice-server.so"
qemu-system-x86_64 -spice disable-ticketing,port=5900 -vga qxl
Comment 2 Frediano Ziglio 2016-02-19 10:08:17 UTC
Fixed in last master version.

Thanks for prompt report.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.