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)
Fri, Nov 28, 10:29 AM
Unknown Object (File)
Thu, Nov 27, 8:26 AM
Unknown Object (File)
Thu, Nov 27, 5:25 AM
Unknown Object (File)
Tue, Nov 25, 4:44 PM
Unknown Object (File)
Sat, Nov 22, 3:35 AM
Unknown Object (File)
Fri, Nov 21, 11:03 PM
Unknown Object (File)
Tue, Nov 18, 8:28 PM
Unknown Object (File)
Tue, Nov 18, 3:16 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 Skipped
Unit
Tests Skipped