]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
MFC r352404, r352413-r352414
authorAlan Somers <asomers@FreeBSD.org>
Wed, 30 Oct 2019 01:35:00 +0000 (01:35 +0000)
committerAlan Somers <asomers@FreeBSD.org>
Wed, 30 Oct 2019 01:35:00 +0000 (01:35 +0000)
commit642030e0fc9689e0450f610fc9f8e36fceb288ef
tree9a5c09f5caef5d866869172f41571e4a909ce829
parente2480e5d1fb0ca519d759966475d48af1dfc70d6
MFC r352404, r352413-r352414

r352404:
fusefs: fix some minor issues with fuse_vnode_setparent

* When unparenting a vnode, actually clear the flag. AFAIK this is basically
  a no-op because we only unparent a vnode when reclaiming it or when
  unlinking.

* There's no need to call fuse_vnode_setparent during reclaim, because we're
  about to free the vnode data anyway.

Reviewed by: emaste
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D21630

r352413:
fusefs: fix some minor Coverity CIDs in the tests

Where open(2) is expected to fail, the tests should assert or expect that
its return value is -1.  These tests all accepted too much but happened to
pass anyway.

Reported by: Coverity
Coverity CID: 1404512140437814045041404483
Sponsored by: The FreeBSD Foundation

r352414:
fusefs: initialize C++ classes the Coverity way

Coverity complained that I wasn't initializing some class members until the
SetUp method.  Do it in the constructor instead.

Reported by: Coverity
Coverity CIDs: 14043521404378
Sponsored by: The FreeBSD Foundation
sys/fs/fuse/fuse_node.h
sys/fs/fuse/fuse_vnops.c
tests/sys/fs/fusefs/create.cc
tests/sys/fs/fusefs/default_permissions.cc
tests/sys/fs/fusefs/io.cc
tests/sys/fs/fusefs/mknod.cc
tests/sys/fs/fusefs/opendir.cc