From patchwork Fri Jan 21 14:39:02 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: UBI: fix unaligned leb_start value Date: Fri, 21 Jan 2011 04:39:02 -0000 From: John Ogness X-Patchwork-Id: 79849 Message-Id: <87zkqu1fuh.fsf@vostro.fn.ogness.net> To: Artem Bityutskiy Cc: linux-mtd@lists.infradead.org, David Woodhouse , linux-kernel@vger.kernel.org From: John Ogness The wrong header size was used in calculating the data offset. The data offset should be VID header offset + VID header size (aligned to the minimum I/O unit). Signed-off-by: John Ogness --- This patch is against linux-next-20110121. Since the value is later aligned, this patch does not fix any real world problem. But it should be fixed nonetheless. drivers/mtd/ubi/build.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/mtd/ubi/build.c b/drivers/mtd/ubi/build.c index f49e49d..a4f47dd 100644 --- a/drivers/mtd/ubi/build.c +++ b/drivers/mtd/ubi/build.c @@ -737,7 +737,7 @@ static int io_init(struct ubi_device *ubi) } /* Similar for the data offset */ - ubi->leb_start = ubi->vid_hdr_offset + UBI_EC_HDR_SIZE; + ubi->leb_start = ubi->vid_hdr_offset + UBI_VID_HDR_SIZE; ubi->leb_start = ALIGN(ubi->leb_start, ubi->min_io_size); dbg_msg("vid_hdr_offset %d", ubi->vid_hdr_offset);