diff options
| author | Jason A. Harmening <jah@FreeBSD.org> | 2025-11-08 04:23:02 +0000 |
|---|---|---|
| committer | Jason A. Harmening <jah@FreeBSD.org> | 2025-11-09 03:47:15 +0000 |
| commit | 8b92a6ad597e224b616a8b1d6983f3b55c85308e (patch) | |
| tree | f11feafe3938fb4b4b78c4edd9892c449901278b /sbin | |
| parent | 335fc09ab8d25c3ced027d46f5a0f4103d5c8bee (diff) | |
Commit 2ed053cde5 changed UFS' VOP_RMDIR() behavior to no longer
ignore whiteouts when determining whether a directory is empty,
unless explicitly requested by the caller. However, this also
necessitates a change to rm(1) to avoid breaking the expected
behavior when forcibly removing directory hierarchies via `rm -fr`.
I neglected to make this follow-on change despite discussing it
in the review for the breaking commit (D45987).
Finally address the breakage by making `rm -fr` imply FTS_WHITEOUT
when rm(1) reads directory contents via fts_read(3). While here,
also fix a logic error which produces a spurious 'No error' warning
message on stdout for each deleted whiteout.
Reported by: csjp
Reviewed by: csjp, kib, olce
MFC after: 3 days
Differential Revision: https://reviews.freebsd.org/D53640
Diffstat (limited to 'sbin')
0 files changed, 0 insertions, 0 deletions
