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

	F109384927: D27938.diff
	Tue, Feb 4, 9:10 AM

	Unknown Object (File)
	Sun, Feb 2, 9:29 AM

	Unknown Object (File)
	Mon, Jan 27, 1:03 PM

	Unknown Object (File)
	Sun, Jan 26, 6:23 PM

	Unknown Object (File)
	Sat, Jan 18, 10:08 PM

	Unknown Object (File)
	Sat, Jan 18, 10:03 PM

	Unknown Object (File)
	Dec 10 2024, 7:57 PM

	Unknown Object (File)
	Nov 30 2024, 5:55 PM

View All 72 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