From 6cd1b2d427a8c7e487d40d371435a2b30cf94ba3 Mon Sep 17 00:00:00 2001 From: Tavian Barnes Date: Wed, 13 Jan 2021 09:35:45 -0500 Subject: -perm: Use +t instead of ug+t The chmod spec says that ug+t is unspecified, and only +t or a+t is guaranteed to actually set the sticky bit. In practice GNU tools respect o+t as well, but ignore u+t/g+t. Fix the implementation to match GNU, and only test the POSIX required parse. --- parse.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'parse.c') diff --git a/parse.c b/parse.c index 4e326ec..fa95e69 100644 --- a/parse.c +++ b/parse.c @@ -1896,8 +1896,10 @@ static int parse_mode(const struct parser_state *state, const char *mode, struct } break; case 't': - file_change |= S_ISVTX; - dir_change |= S_ISVTX; + if (who & 0007) { + file_change |= S_ISVTX; + dir_change |= S_ISVTX; + } break; default: mstate = MODE_ACTION_APPLY; -- cgit v1.2.3