linux-ext4 - Re: [PATCH 1/3] lib/ext2fs: Add ext2fs

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] [day] [month] [year] [list]

Message-ID: <50FA2F2D.8010901@infradead.org>
Date:	Fri, 18 Jan 2013 21:29:17 -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 1/3] lib/ext2fs: Add ext2fs_symlink

On 01/15/2013 11:43 AM, Theodore Ts'o wrote:
> I'll fix up the this patch before I commit it, but this is a perfect
> exhibit about why I request that code submissions come with test
> cases.  It turns out that there were a couple of problems with
> ext2fs_symlink(), that showed up very quickly as soon as I started
> writing a test case (where it's important to run e2fsck on the
> resulting file system after creating the symlinks --- e2fsck is a
> wonderful rep invariant checkers for ext[234] file systems. :-)
> 
> *) i_blocks must be set to 0 for fast symlinks
> *) The last argument of ext2fs_inode_alloc_stats() indicates whether
>    the new inode is a directory or not.  So when you cut and pasted
>    the code from ext2fs_mkdir(), it needed to be changed.
> *) Zeroing the entire block before setting the symlink in the case
>    where it needs to use an external data block makes it a lot
>    easier to write the regression test.
> 
> So here's the patch I needed to apply on top of your submission....
> 

Ted,

I apologize for not getting to the test case, I meant to, and was still
meaning to. I was pulled away on a number of other tasks, took a family
vacation etc. I figured I was the only one that cared and it would be me
who had to pay the price if things changed while I delayed.

THANK YOU, THANK YOU! I really appreciate you writing the test cases and
fixing up the remaining issues with the patch.

This should now allow us to work with a debugfs script to validate this
tooling to populate filesystem images. Once we have done that, we'll
want to look at pushing some things (like file copy wrappers) from
debugfs into libext2fs and implement the -i (initial dir) option we
discused for mke2fs.

We'll do our validation and then resurface here.

Thanks again!

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