Re: [PATCH spice-server v2] RFC: Support abstract Unix sockets

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

 



On Wed, Nov 22, 2017 at 12:11:33PM +0000, Frediano Ziglio wrote:
> Allows to specify abstract Unix sockets addresses.
> These Unix sockets are supported on Linux and allows to not
> have file system names.

What would be the use-case? Just cleaner not to have a dummy path in the
FS? Or does it bring more? I'd say why not, though a spice-gtk patch
will be needed too.


> 
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/reds.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> Changes since v1:
> - do not unlink no file socket.
> 
> diff --git a/server/reds.c b/server/reds.c
> index ebcbe496..ca0bb75a 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -2585,8 +2585,12 @@ static int reds_init_socket(const char *addr, int portnr, int family)
>  
>          local.sun_family = AF_UNIX;
>          g_strlcpy(local.sun_path, addr, sizeof(local.sun_path));
> -        unlink(local.sun_path);
>          len = SUN_LEN(&local);
> +        if (local.sun_path[0] == '@') {
> +            local.sun_path[0] = 0;
> +        } else {
> +            unlink(local.sun_path);
> +        }
>          if (bind(slisten, (struct sockaddr *)&local, len) == -1) {
>              perror("bind");
>              return -1;
> -- 
> 2.14.3
> 
> _______________________________________________
> 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

[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]