linux-ext4 - Re: [PATCH 0/3 V2] ext2fsprogs: Symlink support

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 4 Jan 2013 19:15:21 -0500
From:	Theodore Ts'o <tytso@....edu>
To:	Darren Hart <dvhart@...radead.org>
Cc:	linux-ext4@...r.kernel.org, adilger@...ger.ca, sgw@...ux.intel.com,
	darrick.wong@...cle.com
Subject: Re: [PATCH 0/3 V2] ext2fsprogs: Symlink support

On Fri, Jan 04, 2013 at 12:00:57PM -0800, Darren Hart wrote:
> The following series adds symlink support the libext2fs and adds the command to
> debugfs. Basic testing demonstrates functional duplication of a small test
> directory including fastlinks and slowlinks in both the root and a single
> subdirectory.

One thing I'd really appreciate is if you could write a regression
test for this new code.  You can use tests/d_loaddump and
tests/f_dup4/script as a model; I'd suggest calling it d_symlink, and
it would be good if it created a short symlink and a long symlink, and
that the script verified that you get the expected output when you
stat the small and long symlinks using debugfs.  Then run e2fsck on
the file system and make sure the file system is consistent.

Note the use of the debugfs commands:

	set_current_time 200704102100
	set_super_value lastcheck 0
	set_super_value hash_seed null
	set_super_value mkfs_time 0

and the environment variable: 

	E2FSCK_TIME=200704102100

in f_dup4's script.  These are used to control various variable
(random or time-specific) values such that so that you can use a
simple comparison of the output of programs such as debugfs, mke2fs,
and dumpe2fs using "diff".

Thanks again!!

						- Ted

P.S.  Actually, while you're at it, if you could also have the script
create some block and character device files using the debugfs mknod
command, and perhaps call the test d_create_spec_files instead of
d_symlink, that would be great!   (Since I don't think those functions
currently any good test coverage.) 

--
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