Paths

Table of Contentst

Fix vnode locking bug in fuse_vnop_copy_file_range
ClosedPublic
Actions

Authored by asomers on Jan 3 2021, 3:32 AM.

Tags

None

Referenced Files

	F132471164: D27938.diff
	Fri, Oct 17, 5:14 AM

	Unknown Object (File)
	Sun, Oct 5, 5:01 AM

	Unknown Object (File)
	Thu, Sep 25, 3:41 PM

	Unknown Object (File)
	Sep 12 2025, 7:15 AM

	Unknown Object (File)
	Sep 9 2025, 2:41 AM

	Unknown Object (File)
	Sep 4 2025, 10:03 AM

	Unknown Object (File)
	Aug 29 2025, 11:04 AM

	Unknown Object (File)
	Aug 13 2025, 10:24 PM

View All Files

Subscribers

imp

Details

Reviewers

cem

Commits

rG17a82e6af8e2: Fix vnode locking bug in fuse_vnop_copy_file_range

Summary

MFC-With: 92bbfe1f0d1f1c4436d1f064a16e5aaf682526ba

Test Plan

fusefs test suite with DEBUG_VFS_LOCKS

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

asomers created this revision.Jan 3 2021, 3:32 AM

Herald added a subscriber: imp. · View Herald TranscriptJan 3 2021, 3:32 AM

asomers requested review of this revision.Jan 3 2021, 3:32 AM

Harbormaster completed remote builds in B35902: Diff 81568.Jan 3 2021, 3:32 AM

cem added inline comments.Jan 3 2021, 12:43 PM

sys/fs/fuse/fuse_vnops.c
685–726	Need `goto unlock` instead of `return (err);` to avoid leaking the locks, I think.

Don't leak locks on fufh lookup failure

Harbormaster completed remote builds in B35911: Diff 81588.Jan 3 2021, 4:34 PM

cem accepted this revision.Jan 3 2021, 4:49 PM

This revision is now accepted and ready to land.Jan 3 2021, 4:49 PM

Closed by commit rG17a82e6af8e2: Fix vnode locking bug in fuse_vnop_copy_file_range (authored by asomers). · Explain WhyJan 3 2021, 6:20 PM

This revision was automatically updated to reflect the committed changes.

asomers added a commit: rG17a82e6af8e2: Fix vnode locking bug in fuse_vnop_copy_file_range.

Revision Contents
Changeset List

Path

Size

sys/

fs/

fuse/

fuse_vnops.c

16 lines

Diff 81594

View Options

Fix vnode locking bug in fuse_vnop_copy_file_rangeClosedPublicActions