Page MenuHomeFreeBSD

filemon should hold proctree_lock at least during pid check
Needs ReviewPublic

Authored by sjg on Jun 19 2015, 6:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 1:26 AM
Unknown Object (File)
Dec 18 2023, 8:44 PM
Unknown Object (File)
Sep 29 2023, 5:48 PM
Unknown Object (File)
Aug 20 2023, 8:47 AM
Unknown Object (File)
Aug 8 2023, 5:12 AM
Unknown Object (File)
Jun 26 2023, 11:27 PM
Unknown Object (File)
Jun 16 2023, 3:37 AM
Unknown Object (File)
Jan 12 2023, 11:10 AM
Subscribers
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

filemon's pid check needs to guard against changes to proc tree while it is walking up the list.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

sjg retitled this revision from to filemon should hold proctree_lock at least during pid check.
sjg updated this object.
sjg edited the test plan for this revision. (Show Details)
kib added inline comments.
sys/dev/filemon/filemon_wrapper.c
89

sx_slock should be enough there

89–90

This is wrong condition. You should use something like while (p != initproc).

97

This should be spelled p = proc_realparent(p);

implement suggested changes