2007-02-20 15:19:51

by Adrian Bunk

[permalink] [raw]
Subject: [RFC: 2.6.16 patch] ext3: fix ext3 block bitmap leakage

On Mon, Feb 19, 2007 at 06:03:14PM -0500, Chuck Ebbert wrote:

> This was dropped from 2.6.16-stable somehow:
>
> http://lkml.org/lkml/2006/6/30/54

Thanks for forwarding it.

This patch didn't went in this form into Linus' tree, I'd therefore like
to get an ACK from the ext3 maintainers (Cc'ed) before applying it.

> Below is probably mangled by Thunderbird, you'll have
> to fetch it yourself if you want to merge it:

patch(1) has a nice -l option for applying such patches. :-)

> From: Kirill Korotaev <>
>
> This patch fixes ext3 block bitmap leakage,
> which leads to the following fsck messages on
> _healthy_ filesystem:
> Block bitmap differences: -64159 -73707
>
> All kernels up to 2.6.17 have this bug.
>
> Found by
> Vasily Averin <[email protected]> and Andrey Savochkin <[email protected]>
> Test case triggered the issue was created by
> Dmitry Monakhov <[email protected]>
>
> Signed-Off-By: Vasiliy Averin <[email protected]>
> Signed-Off-By: Andrey Savochkin <[email protected]>
> Signed-Off-By: Kirill Korotaev <[email protected]>
> CC: Dmitry Monakhov <[email protected]>
>
> --- ./fs/ext3/inode.c.e3crp 2006-06-28 05:22:40.000000000 +0400
> +++ ./fs/ext3/inode.c 2006-06-27 13:31:20.000000000 +0400
> @@ -585,6 +585,7 @@ static int ext3_alloc_branch(handle_t *h
>
> branch[0].key = cpu_to_le32(parent);
> if (parent) {
> + keys = 1;
> for (n = 1; n < num; n++) {
> struct buffer_head *bh;
> /* Allocate the next block */

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed


2007-02-20 17:13:19

by Eric Sandeen

[permalink] [raw]
Subject: Re: [RFC: 2.6.16 patch] ext3: fix ext3 block bitmap leakage

Adrian Bunk wrote:
> On Mon, Feb 19, 2007 at 06:03:14PM -0500, Chuck Ebbert wrote:
>
>> This was dropped from 2.6.16-stable somehow:
>>
>> http://lkml.org/lkml/2006/6/30/54
>
> Thanks for forwarding it.
>
> This patch didn't went in this form into Linus' tree, I'd therefore like
> to get an ACK from the ext3 maintainers (Cc'ed) before applying it.

Yep, ACK from me for 2.6.16-stable.

-Eric

2007-02-26 14:35:44

by Adrian Bunk

[permalink] [raw]
Subject: Re: [RFC: 2.6.16 patch] ext3: fix ext3 block bitmap leakage

On Tue, Feb 20, 2007 at 11:08:37AM -0600, Eric Sandeen wrote:
> Adrian Bunk wrote:
> > On Mon, Feb 19, 2007 at 06:03:14PM -0500, Chuck Ebbert wrote:
> >
> >> This was dropped from 2.6.16-stable somehow:
> >>
> >> http://lkml.org/lkml/2006/6/30/54
> >
> > Thanks for forwarding it.
> >
> > This patch didn't went in this form into Linus' tree, I'd therefore like
> > to get an ACK from the ext3 maintainers (Cc'ed) before applying it.
>
> Yep, ACK from me for 2.6.16-stable.

Thanks, I've applied it.

> -Eric

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed