linux-ext4

Open Source and information security mailing list archives

Message-id: <20090409043121.GR17302@webber.adilger.int>
Date:	Wed, 08 Apr 2009 22:31:21 -0600
From:	Andreas Dilger <adilger@....com>
To:	Christian Kujau <lists@...dbynature.de>
Cc:	ext-users <ext3-users@...hat.com>, linux-ext4@...r.kernel.org
Subject: Re: external journal lost [PATCH]

On Apr 08, 2009  01:40 -0700, Christian Kujau wrote:
> On Thu, 2 Apr 2009, Christian Kujau wrote:
> > Hm, I never had to forcefully remove a journal (yet), but from reading the 
> > manpage I'd indeed expect to get this removed even if the filesystem is in 
> > error. The attached patch to e2fsprogs (latest git) makes tune2fs removing 
> > the journal when "-f" is supplied. And it seems to work so far:
> > 
> > http://nerdbynature.de/bits/tune2fs/
> > 
> > Comments?
> 
> Hm, I was really hoping someone would comment on this, as I'm a) puzzled 
> as well, why the -force option does not work as expected and b) if this is 
> the right thing to do:

Proabably the right thing to do.

> --- e2fsprogs-git/misc/tune2fs.c.ORIG	2009-04-02 22:40:58.740218188 +0200
> +++ e2fsprogs-git/misc/tune2fs.c	2009-04-02 22:42:37.221830335 +0200
> @@ -374,8 +374,8 @@ static void update_feature_set(ext2_fils
>  				"read-only.\n"), stderr);
>  			exit(1);
>  		}
> -		if (sb->s_feature_incompat &
> -		    EXT3_FEATURE_INCOMPAT_RECOVER) {
> +		if ((sb->s_feature_incompat &
> +		    EXT3_FEATURE_INCOMPAT_RECOVER) && (!f_flag)) {
>  			fputs(_("The needs_recovery flag is set.  "
>  				"Please run e2fsck before clearing\n"
>  				"the has_journal flag.\n"), stderr);
> 
> -- 
> Bruce Schneier knows the secret formula for Coca-Cola.
> 
> _______________________________________________
> Ext3-users mailing list
> Ext3-users@...hat.com
> https://www.redhat.com/mailman/listinfo/ext3-users

Cheers, Andreas
--
Andreas Dilger
Sr. Staff Engineer, Lustre Group
Sun Microsystems of Canada, Inc.

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

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