[PATCH] UBI: fix bug in image sequence number handling
Artem Bityutskiy
dedekind at infradead.org
Tue Jul 14 01:43:20 EDT 2009
On Mon, 2009-07-13 at 16:47 +0200, Holger Brunck wrote:
> Hi all,
> I have seen a bug on the current 2.6.28 backport for UBI. But the same
> bug seems to be present in the current ubi-2.6.git.
>
>
> This patch fixes a bug in the image seq. number handling in the
> scanning level. The assignment of the image_seq was incorrect.
>
> Signed-off-by: Holger Brunck <holger.brunck at keymile.com>
> ---
> drivers/mtd/ubi/scan.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mtd/ubi/scan.c b/drivers/mtd/ubi/scan.c
> index 065c4ba..e337b0b 100644
> --- a/drivers/mtd/ubi/scan.c
> +++ b/drivers/mtd/ubi/scan.c
> @@ -781,7 +781,7 @@ static int process_eb(struct ubi_device *ubi, struct
> ubi_scan_info *si,
> return -EINVAL;
> }
>
> - image_seq = be32_to_cpu(ech->ec);
> + image_seq = be32_to_cpu(ech->image_seq);
> if (!si->image_seq_set) {
> ubi->image_seq = image_seq;
> si->image_seq_set = 1;
Yes, this is me who did this in the
"UBI: nicify image sequence number handling" commit. Sorry, really
shame bug. Where is my brown paper-bag? I've pushed your patch to
all trees.
BTW, the patch was line-wrapped.
--
Best regards,
Artem Bityutskiy (Битюцкий Артём)
More information about the linux-mtd
mailing list