[RFC PATCH vdagent 16/16] Fix return value of update_dev_mode_position

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]