From: Bill O'Donnell Date: Mon, 19 Sep 2016 06:05:45 +0000 (+1000) Subject: xfs_quota: add case for foreign fs, disabled regardless of foreign_allowed X-Git-Tag: v4.8.0-rc2~9 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=2ba39cd354b18cb1dbaf29115ec212edd92c9671;p=thirdparty%2Fxfsprogs-dev.git xfs_quota: add case for foreign fs, disabled regardless of foreign_allowed Some commands are disallowed for foreign filesystems, regardless of whether or not the -f flag is thrown. Add a case for this condition and improve commenting and output messaging accordingly in init_check_command. Signed-off-by: Bill O'Donnell Reviewed-by: Eric Sandeen Signed-off-by: Dave Chinner --- diff --git a/quota/init.c b/quota/init.c index 2c18c8b0e..3bebbb873 100644 --- a/quota/init.c +++ b/quota/init.c @@ -112,21 +112,29 @@ init_check_command( if (!fs_path) return 1; - /* Always run commands that we are told to skip here */ + /* Always run commands that are valid for all fs types. */ if (ct->flags & CMD_ALL_FSTYPES) return 1; - /* if it's an XFS filesystem, always run the command */ + /* If it's an XFS filesystem, always run the command. */ if (!(fs_path->fs_flags & FS_FOREIGN)) return 1; - /* If the user specified foreign filesysetms are ok, run it */ + /* If the user specified foreign filesystems are ok (-f), run cmd. */ if (foreign_allowed && (ct->flags & CMD_FLAG_FOREIGN_OK)) return 1; - /* foreign filesystem and it's not a valid command! */ - fprintf(stderr, _("%s command is for XFS filesystems only\n"), + /* If cmd not allowed on foreign fs, regardless of -f flag, skip it. */ + if (!(ct->flags & CMD_FLAG_FOREIGN_OK)) { + fprintf(stderr, _("%s: command is for XFS filesystems only\n"), + ct->name); + return 0; + } + + /* foreign fs, but cmd only allowed via -f flag. Skip it. */ + fprintf(stderr, + _("%s: foreign filesystem. Invoke xfs_quota with -f to enable.\n"), ct->name); return 0; }