[Virtio-fs] [PATCH v3] virtiofsd: Prevent multiply running with same vhost_user_socket

[Stefan Hajnoczi]

On Tue, Aug 13, 2019 at 04:06:45PM -0400, Masayoshi Mizuma wrote:

Two minor comments below.  They can be squashed in when merging this
patch.

If users find it problematic that pid files are never deleted we could
make this feature optional with --pidfile=PATH.  But I think we can
merge this patch for now and see if anyone encounters issues before the
next virtio-fs release.

Reviewed-by: Stefan Hajnoczi <stefanha at redhat.com>

> +static int fv_socket_lock(struct fuse_session *se)
> +{
> +        char *dir, *sk_name;
> +        Error *local_err = NULL;
> +        int ret = -1;
> +
> +        dir = qemu_get_local_state_pathname("run/virtiofsd");
> +
> +        if (g_mkdir_with_parents(dir, S_IRWXU) < -1) {
> +                fuse_err("%s: Failed to create directory %s: %s",
> +                        __func__, dir, strerror(errno));

Missing g_free(dir).

> @@ -876,6 +916,17 @@ static int fv_create_listen_socket(struct fuse_session *se)
>                  return -1;
>          }
>  
> +        if (!strlen(se->vu_socket_path)) {
> +                fuse_err("Socket path is NULL\n");

s/NULL/empty/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virtio-fs/attachments/20190820/fc8c9f28/attachment.sig>