lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20130422031720.GA12221@thunk.org> Date: Sun, 21 Apr 2013 23:17:20 -0400 From: Theodore Ts'o <tytso@....edu> To: Akira Fujita <a-fujita@...jp.nec.com> Cc: ext4 development <linux-ext4@...r.kernel.org> Subject: Re: [PATCH] tune2fs: Disallow tune2fs to set inode size larger than block size On Thu, Apr 18, 2013 at 05:10:38PM +0900, Akira Fujita wrote: > Disallow tune2fs command to set inode size larger than > block size to ext3 and ext4 (^flex_bg). > Without this patch, tune2fs makes FS unmountable. > > Steps to reproduce: > > 1.Create ext4 without flex_bg (or just create ext3) > # mke2fs -t ext4 -O ^flex_bg DEV > > 2.Set inode size larger than block size > # tune2fs -I 8192 DEV > > 3. We failed to mount FS > # mount DEV MP > mount: wrong fs type, bad option, bad superblock on /dev/sda7, > missing codepage or helper program, or other error > In some cases useful info is found in syslog - try > dmesg | tail or so > > Signed-off-by: Akira Fujita <a-fujita@...jp.nec.com> Thanks for reporting this. Although at the moment we don't support changing the inode size for flex_bg file systems, we may at some point in the future --- and the inode size can never be larger than the block size. So I simplified your change by removing the ^flex_bg check. - Ted >From fab0d286ec8539988551271c5faa864ea38dc4d4 Mon Sep 17 00:00:00 2001 From: Akira Fujita <a-fujita@...jp.nec.com> Date: Sun, 21 Apr 2013 23:14:28 -0400 Subject: [PATCH] tune2fs: disallow tune2fs to set inode size larger than block size Disallow tune2fs command to set the inode size to be larger than the block size. Without this patch, tune2fs makes the file system to be unmountable. Steps to reproduce: 1.Create ext4 without flex_bg (or just create ext3) # mke2fs -t ext4 -O ^flex_bg DEV 2.Set inode size larger than block size # tune2fs -I 8192 DEV 3. We failed to mount FS # mount DEV MP mount: wrong fs type, bad option, bad superblock on /dev/sda7, missing codepage or helper program, or other error In some cases useful info is found in syslog - try dmesg | tail or so Signed-off-by: Akira Fujita <a-fujita@...jp.nec.com> Signed-off-by: "Theodore Ts'o" <tytso@....edu> --- misc/tune2fs.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/misc/tune2fs.c b/misc/tune2fs.c index 849f3a0..47a7711 100644 --- a/misc/tune2fs.c +++ b/misc/tune2fs.c @@ -1944,6 +1944,12 @@ retry_open: rc = 1; goto closefs; } + if (new_inode_size > fs->blocksize) { + fprintf(stderr, _("Invalid inode size %lu (max %d)\n"), + new_inode_size, fs->blocksize); + rc = 1; + goto closefs; + } /* * If inode resize is requested use the -- 1.7.12.rc0.22.gcdd159b -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists