I would squash this in the next commit, but Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> if you prefer to keep them separate. Christophe On Mon, Jul 09, 2018 at 09:51:40AM +0100, Frediano Ziglio wrote: > For some reason kernel handles in Windows have 2 invalid values > depending on the type. Files/devices use INVALID_HANDLE_VALUE (-1), > while others use NULL (0). As _vio_serial is a file, created > with CreateFile, use INVALID_HANDLE_VALUE for consistency. > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > --- > vdagent/vdagent.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/vdagent/vdagent.cpp b/vdagent/vdagent.cpp > index dbf6556..010e73b 100644 > --- a/vdagent/vdagent.cpp > +++ b/vdagent/vdagent.cpp > @@ -196,7 +196,7 @@ VDAgent::VDAgent() > , _session_is_locked (false) > , _desktop_switch (false) > , _display_setting (VD_AGENT_REGISTRY_KEY) > - , _vio_serial (NULL) > + , _vio_serial (INVALID_HANDLE_VALUE) > , _read_pos (0) > , _write_pos (0) > , _logon_desktop (false) > -- > 2.17.1 > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel