From: Sameeh Jubran <sameeh@xxxxxxxxxx> update_dev_mode_position() should propagate error of update_mode_position() in case it fails. Signed-off-by: Sameeh Jubran <sameeh@xxxxxxxxxx> Signed-off-by: Dmitry Fleytman <dmitry@xxxxxxxxxx> --- vdagent/display_configuration.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/vdagent/display_configuration.cpp b/vdagent/display_configuration.cpp index 98b3d82..59dc032 100644 --- a/vdagent/display_configuration.cpp +++ b/vdagent/display_configuration.cpp @@ -361,8 +361,7 @@ bool WDDMInterface::update_dev_mode_position(LPCTSTR device_name, DEVMODE* dev_m //Convert the position so that the primary is always at (0,0) dev_mode->dmPosition.x = x; dev_mode->dmPosition.y = y; - _ccd.update_mode_position(device_name, dev_mode); - return true; + return _ccd.update_mode_position(device_name, dev_mode); } bool WDDMInterface::init_d3d_api() -- 1.8.3.1 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel