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)
Jan 1 2026, 3:13 AM
Unknown Object (File)
Dec 20 2025, 6:00 PM
Unknown Object (File)
Nov 28 2025, 10:29 AM
Unknown Object (File)
Nov 27 2025, 8:26 AM
Unknown Object (File)
Nov 27 2025, 5:25 AM
Unknown Object (File)
Nov 25 2025, 4:44 PM
Unknown Object (File)
Nov 22 2025, 3:35 AM
Unknown Object (File)
Nov 21 2025, 11:03 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