Re: [phodav PATCH 1/2] spice-webdavd: Show error msg when CreateFile fails

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

 



hi

----- Original Message -----
> Adds a popup error message on Windows, when CreateFile() fails.
> 
> CreateFile can fail when webdavd is run without administrator privileges
> on windows. Assert doesn't provide any kind of error message to user,
> when started without those privileges.
> ---
>  spice/spice-webdavd.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/spice/spice-webdavd.c b/spice/spice-webdavd.c
> index c8fb795..368bb44 100644
> --- a/spice/spice-webdavd.c
> +++ b/spice/spice-webdavd.c
> @@ -722,7 +722,12 @@ open_mux_path (const char *path)
>                           OPEN_EXISTING,
>                           FILE_FLAG_OVERLAPPED,
>                           NULL);
> -  g_assert (port_handle != INVALID_HANDLE_VALUE);
> +
> +  if (port_handle == INVALID_HANDLE_VALUE)
> +    {
> +      g_error ("%s", g_win32_error_message (GetLastError ()));
> +      exit (1);

g_error should be no return already, I think exit(1) is unnecessary,

ack anyway

> +    }
> 
>    mux_ostream = G_OUTPUT_STREAM (g_win32_output_stream_new (port_handle,
>    TRUE));
>    mux_istream = G_INPUT_STREAM (g_win32_input_stream_new (port_handle,
>    TRUE));
> --
> 2.4.3
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]