vn_open_vnode(): handle error when fp == NULL
ClosedPublic
Actions

Authored by kib on Apr 19 2021, 10:32 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jun 14, 1:17 AM

	Unknown Object (File)
	Thu, Jun 13, 12:54 AM

	Unknown Object (File)
	Tue, Jun 11, 5:27 AM

	Unknown Object (File)
	Mon, Jun 10, 1:59 PM

	Unknown Object (File)
	Sun, Jun 9, 7:16 AM

	Unknown Object (File)
	Mon, May 20, 1:33 PM

	Unknown Object (File)
	May 3 2024, 2:07 PM

	Unknown Object (File)
	Mar 7 2024, 7:59 PM

View All 28 Files

Subscribers

emaste

pho

Details

Reviewers: None
Commits: rG54f98c4dbf9b: vn_open_vnode(): handle error when fp == NULL

Summary

If VOP_ADD_WRITECOUNT() or adv locking failed, so VOP_CLOSE() needs to be called, we cannot use fp fo_close() when there is no fp. This occurs when e.g. kernel code directly calls vn_open() instead of the open(2) syscall.

In this case, VOP_CLOSE() can be called directly, after possible lock upgrade.

Reported by: nvass@gmx.com
PR: 255119

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib requested review of this revision.Apr 19 2021, 10:32 AM

kib created this revision.

This fixes the issue for me. I have started a full test on mercat1.

emaste added a subscriber: emaste.Apr 20 2021, 2:59 PM

In D29830#669416, @pho wrote:

This fixes the issue for me. I have started a full test on mercat1.

I completed a full stress2 test without seeing any issues.

This revision was not accepted when it landed; it landed in state Needs Review.Apr 21 2021, 3:08 PM

Closed by commit rG54f98c4dbf9b: vn_open_vnode(): handle error when fp == NULL (authored by kib). · Explain Why

This revision was automatically updated to reflect the committed changes.

kib added a commit: rG54f98c4dbf9b: vn_open_vnode(): handle error when fp == NULL.

Revision Contents
Changeset List

Path

Size

sys/

kern/

vfs_vnops.c

32 lines

Diff 87871

View Options

vn_open_vnode(): handle error when fp == NULLClosedPublicActions