[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening |
Date: |
Thu, 18 May 2017 10:42:47 +0200 |
On Tue, 9 May 2017 11:23:05 +0200
Greg Kurz <address@hidden> wrote:
> On Fri, 5 May 2017 12:01:55 -0500
> Eric Blake <address@hidden> wrote:
>
> > On 05/05/2017 09:37 AM, Greg Kurz wrote:
> > > All paths in the virtfs directory now start with "./" (except the virtfs
> > > root itself which is exactly ".").
> > >
> > > We hence don't need to skip leading '/' characters anymore, nor to handle
> > > the empty path case. Also, since virtfs will only ever be supported on
> > > linux+glibc hosts, we can use strchrnul() and come up with a much simplier
> > > code to walk through the path elements. And we don't need to dup() the
> > > passed directory fd.
> > >
> > > Signed-off-by: Greg Kurz <address@hidden>
> > > ---
> > > hw/9pfs/9p-local.c | 5 -----
> > > hw/9pfs/9p-util.c | 26 ++++++++++----------------
> > > 2 files changed, 10 insertions(+), 21 deletions(-)
> > >
> > > diff --git a/hw/9pfs/9p-local.c b/hw/9pfs/9p-local.c
> > > index 92262f3c3e37..bb6e296df317 100644
> > > --- a/hw/9pfs/9p-local.c
> > > +++ b/hw/9pfs/9p-local.c
> > > @@ -54,11 +54,6 @@ int local_open_nofollow(FsContext *fs_ctx, const char
> > > *path, int flags,
> > > {
> > > LocalData *data = fs_ctx->private;
> > >
> > > - /* All paths are relative to the path data->mountfd points to */
> > > - while (*path == '/') {
> > > - path++;
> > > - }
> >
> > Is it worth adding any assert()s in place of the deleted code?
> >
>
> The assert() added by this patch ensures that we never pass an empty
> string to relative_openat_nofollow(), which isn't related to this
> hunk of deleted code... so I'm not sure I understand the question :-\
>
Ping ?
> > Otherwise looks okay.
> >
>
pgpooBX72mzmr.pgp
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH 1/5] 9pfs: check return value of v9fs_co_name_to_path(), (continued)
[Qemu-devel] [PATCH 3/5] 9pfs: local: simplify file opening, Greg Kurz, 2017/05/05
[Qemu-devel] [PATCH 4/5] 9pfs: local: metadata file for the VirtFS root, Greg Kurz, 2017/05/05
[Qemu-devel] [PATCH 5/5] 9pfs: local: forbid client access to metadata, Greg Kurz, 2017/05/05
Re: [Qemu-devel] [PATCH 0/5] 9pfs: local: fix metadata of mapped-file security mode, no-reply, 2017/05/05
Re: [Qemu-devel] [PATCH 0/5] 9pfs: local: fix metadata of mapped-file security mode, Leo Gaspard, 2017/05/08