9pfs: local: simplify file opening
commit3dbcf27334b6c41e74a476b55d76f60df1c4007b
authorGreg Kurz <groug@kaod.org>
Thu, 25 May 2017 08:30:14 +0000 (25 10:30 +0200)
committerGreg Kurz <groug@kaod.org>
Thu, 25 May 2017 08:30:14 +0000 (25 10:30 +0200)
tree4fda76c5b7b96a449362455813968bedddffdd3f
parentf57f5878578af19f72344439154234c6d6ba8ccc
9pfs: local: simplify file opening

The logic to open a path currently sits between local_open_nofollow() and
the relative_openat_nofollow() helper, which has no other user.

For the sake of clarity, this patch moves all the code of the helper into
its unique caller. While here we also:
- drop the code to skip leading "/" because the backend isn't supposed to
  pass anything but relative paths without consecutive slashes. The assert()
  is kept because we really don't want a buggy backend to pass an absolute
  path to openat().
- use strchrnul() to get a simpler code. This is ok since virtfs is for
  linux+glibc hosts only.
- don't dup() the initial directory and add an assert() to ensure we don't
  return the global mountfd to the caller. BTW, this would mean that the
  caller passed an empty path, which isn't supposed to happen either.

Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Eric Blake <eblake@redhat.com>
[groug: fixed typos in changelog]
hw/9pfs/9p-local.c
hw/9pfs/9p-util.c
hw/9pfs/9p-util.h