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]

Message-ID: <50E77343.4010807@infradead.org>
Date:	Fri, 04 Jan 2013 16:26:43 -0800
From:	Darren Hart <dvhart@...radead.org>
To:	Theodore Ts'o <tytso@....edu>
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 01/04/2013 04:15 PM, Theodore Ts'o wrote:
> 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.) 


Ah, great. I'm all for built-in tests. I'll get on it next week.

Thanks Ted.

--
Darren Hart
Intel Open Source Technology Center
--
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