Page MenuHomeFreeBSD

ufs: change isrmdir type to bool or u_int as appropriate
ClosedPublic

Authored by kib on Jul 29 2025, 6:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 25, 11:44 PM
Unknown Object (File)
Wed, May 20, 1:29 PM
Unknown Object (File)
Sun, May 17, 9:28 AM
Unknown Object (File)
Tue, May 12, 10:17 AM
Unknown Object (File)
Mon, May 11, 9:42 PM
Unknown Object (File)
Mon, May 11, 9:05 PM
Unknown Object (File)
Wed, Apr 29, 2:34 PM
Unknown Object (File)
Wed, Apr 29, 2:29 PM
Subscribers

Details

Summary
Use bool for isrmdir argument to
ufs_dirremove()/softdep_setup_remove()/newdirrem(), where it is used as
bool.

Use u_int for isrmdir argument to
ufs_dirrewrite()/softdep_setup_directory_change()
where it is 0/1/ino.  Without the change to unsigned, the
        if (isrmdir > 1)
test is broken on volumes with many inodes.
Use newparent instead of isrmdir for the argument name in this case.

Reviewed by:    markj
Fixes:  610319c766e941de96e52f2d28fea9f8cfc51aeb
Fixes:  98eb6f0eaa50d8bd9a6794f0a9da2eddeae5bcd8

Diff Detail

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