Page MenuHomeFreeBSD
Differential D58506

kern_execve(): avoid storing non-VDIR into p_textdvp
ClosedPublic
Actions

Authored by kib on Jul 28 2026, 2:35 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 11, 1:14 PM
Unknown Object (File)
Thu, Sep 10, 6:23 PM
Unknown Object (File)
Mon, Sep 7, 9:23 PM
Unknown Object (File)
Sun, Sep 6, 11:08 PM
Unknown Object (File)
Sat, Sep 5, 2:14 PM
Unknown Object (File)
Sat, Sep 5, 11:06 AM
Unknown Object (File)
Sat, Sep 5, 2:30 AM
Unknown Object (File)
Thu, Sep 3, 3:51 PM
View All 78 Files
Subscribers
imp
nick_spun.io
olce
pho

Details

Reviewers
markj
jah
Commits
rG930f2e4da964: kern_execve(): avoid storing non-VDIR into p_textdvp

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.Jul 28 2026, 2:35 AM
Herald added subscribers: olce, imp. · View Herald TranscriptJul 28 2026, 2:35 AM
kib requested review of this revision.Jul 28 2026, 2:35 AM
kib mentioned this in D58178: vfs: handle file mounts in vn_fullpath_hardlink().
pho added a comment.Jul 28 2026, 7:59 AM

Looks good to me. I have started a full stress2 test, just in case ..

09:54 /usr/src/tools/test/stress2/misc $ ./filemount_panic.sh
  PID COMM                OSREL PATH
67825 cover             1600019 /tmp/tmp.kEEk4GCRkj.renamed/cover
09:54 /usr/src/tools/test/stress2/misc $ ./filemount_panic2.sh 
mounted /bin/sleep over /tmp/filemount.VWEd7r/cover
querying kern.proc.pathname for pid 28160
pathname = /tmp/filemount.VWEd7r.renamed/cover
09:56 /usr/src/tools/test/stress2/misc $ uname -a
FreeBSD mercat1.netperf.freebsd.org 16.0-CURRENT FreeBSD 16.0-CURRENT #1 ufs-n287721-3b23d05c7781: Tue Jul 28 08:09:01 CEST 2026     pho@mercat1.netperf.freebsd.org:/var/tmp/deviant3/sys/amd64/compile/PHO amd64
09:56 /usr/src/tools/test/stress2/misc $
kib added reviewers: markj, jah.Jul 28 2026, 8:26 AM
markj added inline comments.Jul 28 2026, 1:56 PM
sys/kern/kern_exec.c
571

I think some comment explaining what this block does would be welcome.

589

vn_vptocnp() will drop the ref on dvp1 before returning an error, so I think this vrele() is wrong.

kib updated this revision to Diff 182981.Jul 28 2026, 7:52 PM
kib marked 2 inline comments as done.

Add comment.
Drop vrele().

markj added inline comments.Jul 28 2026, 10:27 PM
sys/kern/kern_exec.c
579
580
613

Is it okay to pass a NULL newtextdvp here?

kib updated this revision to Diff 182998.Jul 28 2026, 10:40 PM
kib marked 3 inline comments as done.

Grammar.
Guard vn_fullpath_hardlink() against NULL newtextdvp == NULL.

markj accepted this revision.Jul 28 2026, 10:44 PM
This revision is now accepted and ready to land.Jul 28 2026, 10:44 PM
nick_spun.io added a comment.Jul 29 2026, 1:15 AM

I’m late to the party but this covers every issue I’ve run into except one dumb one that I’m not sure should live in vfs to begin with

jah accepted this revision.Jul 29 2026, 5:21 AM
Closed by commit rG930f2e4da964: kern_execve(): avoid storing non-VDIR into p_textdvp (authored by kib). · Explain WhyJul 30 2026, 2:57 AM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rG930f2e4da964: kern_execve(): avoid storing non-VDIR into p_textdvp.

Revision Contents
Changeset List

PathSize
sys/
kern/
38 lines

Diff 183089

View Options

sys/kern/kern_exec.c

Loading...