[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] virtiofsd: Fix check of chown()'s return value
From: |
Greg Kurz |
Subject: |
Re: [PATCH] virtiofsd: Fix check of chown()'s return value |
Date: |
Mon, 10 May 2021 16:23:00 +0200 |
On Mon, 10 May 2021 15:16:23 +0100
"Dr. David Alan Gilbert" <dgilbert@redhat.com> wrote:
> * Greg Kurz (groug@kaod.org) wrote:
> > Otherwise you always get this warning when using --socket-group=users
> >
> > vhost socket failed to set group to users (100)
> >
> > While here, print out the error if chown() fails.
> >
> > Signed-off-by: Greg Kurz <groug@kaod.org>
>
> probably needs a fixes: but yes.
>
Fixes: f6698f2b03b0 ("tools/virtiofsd: add support for --socket-group")
> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
>
> > ---
> > tools/virtiofsd/fuse_virtio.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tools/virtiofsd/fuse_virtio.c b/tools/virtiofsd/fuse_virtio.c
> > index 3e13997406bf..638d3ffe2f8a 100644
> > --- a/tools/virtiofsd/fuse_virtio.c
> > +++ b/tools/virtiofsd/fuse_virtio.c
> > @@ -978,9 +978,9 @@ static int fv_create_listen_socket(struct fuse_session
> > *se)
> > if (se->vu_socket_group) {
> > struct group *g = getgrnam(se->vu_socket_group);
> > if (g) {
> > - if (!chown(se->vu_socket_path, -1, g->gr_gid)) {
> > + if (chown(se->vu_socket_path, -1, g->gr_gid) == -1) {
> > fuse_log(FUSE_LOG_WARNING,
> > - "vhost socket failed to set group to %s (%d)\n",
> > + "vhost socket failed to set group to %s (%d):
> > %m\n",
> > se->vu_socket_group, g->gr_gid);
> > }
> > }
> >
> >