diff options
author | Artem Bityutskiy <dedekind1@gmail.com> | 2011-07-04 09:48:37 +0300 |
---|---|---|
committer | Artem Bityutskiy <dedekind1@gmail.com> | 2011-07-04 09:48:37 +0300 |
commit | 662edb6e7ddc01ffee53ffb320acd45e1814c10a (patch) | |
tree | e7fad5729f9df47bff5c463eb8354f1f1f28569a /mkfs.ubifs/mkfs.ubifs.c | |
parent | df1d8c90f634a200564ae21a76e9e6b84804d7fa (diff) |
mkfs.ubifs: remove root inode squash feature
Remove the -squash-rino-perm mkfs.ubifs option as this was planned. Remove
the corresponding record from the feature-removal-schedule.txt file.
Signed-off-by: Artem Bityutskiy <dedekind1@gmail.com>
Diffstat (limited to 'mkfs.ubifs/mkfs.ubifs.c')
-rw-r--r-- | mkfs.ubifs/mkfs.ubifs.c | 36 |
1 files changed, 0 insertions, 36 deletions
diff --git a/mkfs.ubifs/mkfs.ubifs.c b/mkfs.ubifs/mkfs.ubifs.c index 733a3bb..79cc71d 100644 --- a/mkfs.ubifs/mkfs.ubifs.c +++ b/mkfs.ubifs/mkfs.ubifs.c @@ -110,7 +110,6 @@ static char *output; static int out_fd; static int out_ubi; static int squash_owner; -static int squash_rino_perm; /* The 'head' (position) which nodes are written */ static int head_lnum; @@ -156,8 +155,6 @@ static const struct option longopts[] = { {"log-lebs", 1, NULL, 'l'}, {"orph-lebs", 1, NULL, 'p'}, {"squash-uids" , 0, NULL, 'U'}, - {"squash-rino-perm", 0, NULL, 'Q'}, - {"nosquash-rino-perm", 0, NULL, 'q'}, {NULL, 0, NULL, 0} }; @@ -197,15 +194,6 @@ static const char *helptext = "-V, --version display version information\n" "-g, --debug=LEVEL display debug information (0 - none, 1 - statistics,\n" " 2 - files, 3 - more details)\n" -"-Q, --squash-rino-perm ignore permissions of the FS image directory (the one\n" -" specified with --root) and make the UBIFS root inode\n" -" permissions to be {uid=gid=root, u+rwx,go+rx}; this is\n" -" a legacy compatibility option and it will be removed\n" -" at some point, do not use it\n" -"-q, --nosquash-rino-perm for the UBIFS root inode use permissions of the FS\n" -" image directory (the one specified with --root); this\n" -" is the default behavior; this option will be removed\n" -" at some point, do not use it, see clarifications below;\n" "-h, --help display this help text\n\n" "Note, SIZE is specified in bytes, but it may also be specified in Kilobytes,\n" "Megabytes, and Gigabytes if a KiB, MiB, or GiB suffix is used.\n\n" @@ -217,15 +205,6 @@ static const char *helptext = "or more percent better than \"lzo\", mkfs.ubifs chooses \"lzo\", otherwise it chooses\n" "\"zlib\". The \"--favor-percent\" may specify arbitrary threshold instead of the\n" "default 20%.\n\n" -"The -R parameter specifies amount of bytes reserved for the super-user.\n\n" -"Some clarifications about --squash-rino-perm and --nosquash-rino-perm options.\n" -"Originally, mkfs.ubifs did not have them, and it always set permissions for the UBIFS\n" -"root inode to be {uid=gid=root, u+rwx,go+rx}. This was a bug which was found too\n" -"late, when mkfs.ubifs had already been used in production. To fix this bug, 2 new\n" -"options were introduced: --squash-rino-perm which preserves the old behavior and\n" -"--nosquash-rino-perm which makes mkfs.ubifs use the right permissions for the root\n" -"inode. Now these options are considered depricated and they will be removed later, so\n" -"do not use them.\n\n" "The -F parameter is used to set the \"fix up free space\" flag in the superblock,\n" "which forces UBIFS to \"fixup\" all the free space which it is going to use. This\n" "option is useful to work-around the problem of double free space programming: if the\n" @@ -674,13 +653,6 @@ static int get_options(int argc, char**argv) case 'U': squash_owner = 1; break; - case 'Q': - squash_rino_perm = 1; - printf("WARNING: --squash-rino-perm is depricated, do not use it\n"); - break; - case 'q': - printf("WARNING: --nosquash-rino-perm is depricated, do not use it\n"); - break; } } @@ -709,10 +681,6 @@ static int get_options(int argc, char**argv) return err_msg("Maximum count of LEBs was not specified " "(use -h for help)"); - if (squash_rino_perm != -1 && !root) - return err_msg("--squash-rino-perm and nosquash-rino-perm options " - "can be used only with the --root option"); - if (c->max_bud_bytes == -1) { int lebs; @@ -1682,10 +1650,6 @@ static int write_data(void) if (err) return sys_err_msg("bad root file-system directory '%s'", root); - if (squash_rino_perm) { - root_st.st_uid = root_st.st_gid = 0; - root_st.st_mode = mode; - } } else { root_st.st_mtime = time(NULL); root_st.st_atime = root_st.st_ctime = root_st.st_mtime; |