From 566c1c41c8aeb5c83541058eaa4f41d33f861cb9 Mon Sep 17 00:00:00 2001 From: tabudz Date: Tue, 25 Feb 2025 11:36:43 +0800 Subject: [PATCH] fusermount: refuse unknown options Blacklists are notoriously fragile; especially if the kernel wishes to add some security-critical mount option at a later date, all existing systems with older versions of fusermount installed will suddenly have a security problem. Additionally, if the kernel's option parsing became a tiny bit laxer, the blacklist could probably be bypassed. Whitelist known-harmless flags instead, even if it's slightly more inconvenient. --- contrib/fuse-util/fusermount.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/contrib/fuse-util/fusermount.c b/contrib/fuse-util/fusermount.c index ff743f75a21..c1177b60a0d 100644 --- a/contrib/fuse-util/fusermount.c +++ b/contrib/fuse-util/fusermount.c @@ -773,10 +773,16 @@ static int do_mount(const char *mnt, char **typep, mode_t rootmode, flags |= flag; else flags &= ~flag; - } else { + } else if (opt_eq(s, len, "default_permissions") || + opt_eq(s, len, "allow_other") || + begins_with(s, "max_read=") || + begins_with(s, "blksize=")) { memcpy(d, s, len); d += len; *d++ = ','; + } else { + fprintf(stderr, "%s: unknown option '%.*s'\n", progname, len, s); + exit(1); } } }