diff options
| author | Olivier Certner <olce@FreeBSD.org> | 2026-09-07 16:09:54 +0000 |
|---|---|---|
| committer | Olivier Certner <olce@FreeBSD.org> | 2026-09-23 11:37:45 +0000 |
| commit | 72bb9eb7b6decc34f343bdcbf0a8df0c73f9edd4 (patch) | |
| tree | 337b5e228a8e2a61cb7e48ab3c091bdced7261a2 /sys/kern/kern_ctf.c | |
| parent | 9d0859637f99160e17b656df153effae3df31f8b (diff) | |
In getcwd_logical(), test for a '.' or '..' component in one of the most
straightforward and intelligible ways possible.
In particular, this removes a superfluous re-test of the the component's
first character being '.' when the first one did not pass and, more
importantly, prevents the second test from relying on a side-effect in
the first.
While here, for better clarity, replace the loop that searches for '/'
with a simple call to strchrnul().
Add high-level comments about what is going on.
While here, test explicitly that pointed 'char' values are not 0 ('\0')
(style(9)).
While here, separate the successive steps of getcwd_logical() with blank
lines.
No functional change (intended).
Discussed with: emaste
Fixes: 2df923c5d2d0 ("pwd: Clean up and adopt POSIX semantics")
MFC after: 3 days
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D59709
Diffstat (limited to 'sys/kern/kern_ctf.c')
0 files changed, 0 insertions, 0 deletions
