Do not assume we allocated the handle doing the cleanup. This utility will be reused in next patch. Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> Acked-by: Christophe de Dinechin <dinechin@xxxxxxxxxx> --- vdagent/vdagent.cpp | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/vdagent/vdagent.cpp b/vdagent/vdagent.cpp index 010e73b..d603dda 100644 --- a/vdagent/vdagent.cpp +++ b/vdagent/vdagent.cpp @@ -110,6 +110,7 @@ private: bool write_message(uint32_t type, uint32_t size, void* data); bool write_clipboard(VDAgentMessage* msg, uint32_t size); bool init_vio_serial(); + void close_vio_serial(); bool send_input(); void set_display_depth(uint32_t depth); void load_display_setting(); @@ -340,7 +341,7 @@ bool VDAgent::run() void VDAgent::cleanup() { - CloseHandle(_vio_serial); + close_vio_serial(); } void VDAgent::set_control_event(control_command_t control_command) @@ -1224,6 +1225,14 @@ bool VDAgent::init_vio_serial() return true; } +void VDAgent::close_vio_serial() +{ + if (_vio_serial != INVALID_HANDLE_VALUE) { + CloseHandle(_vio_serial); + _vio_serial = INVALID_HANDLE_VALUE; + } +} + void VDAgent::dispatch_message(VDAgentMessage* msg, uint32_t port) { bool res = true; -- 2.17.1 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel