linux-kernel - Re: [PATCH 1/3] Merge mkubootimg tool for building U-Boot images

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

Message-ID: <20080103162613.09674907@zod.rchland.ibm.com>
Date:	Thu, 3 Jan 2008 16:26:13 -0600
From:	Josh Boyer <jwboyer@...ux.vnet.ibm.com>
To:	"Mike Frysinger" <vapier.adi@...il.com>
Cc:	sam@...nborg.org, paulus@...ba.org, linux-kernel@...r.kernel.org,
	wd@...x.de
Subject: Re: [PATCH 1/3] Merge mkubootimg tool for building U-Boot images

On Thu, 3 Jan 2008 17:15:48 -0500
"Mike Frysinger" <vapier.adi@...il.com> wrote:

> On Jan 3, 2008 5:02 PM, Josh Boyer <jwboyer@...ux.vnet.ibm.com> wrote:
> > Several platforms require the mkimage tool to generate a uImage file that is
> > used with U-Boot.  This brings the mkimage tool in-kernel to enable building
> > those platforms without having mkimage externally provided.  The tool is named
> > mkubootimg for better clarity.
> >
> > This is currently based off of the version found in U-Boot 1.3.1.
> >
> > Signed-off-by: Josh Boyer <jwboyer@...ux.vnet.ibm.com>
> >
> > ---
> >  scripts/Makefile             |    1
> >  scripts/mkubootimg/Makefile  |    6
> >  scripts/mkubootimg/crc32.c   |  199 +++++++++++
> >  scripts/mkubootimg/mkimage.c |  728 +++++++++++++++++++++++++++++++++++++++++++
> >  scripts/mkubootimg/sha1.c    |  413 ++++++++++++++++++++++++
> >  scripts/mkubootimg/sha1.h    |  115 ++++++
> >  scripts/mkubootimg/uimage.h  |  161 +++++++++
> >  7 files changed, 1623 insertions(+)
> 
> i'm fairly certain sha1 is not needed.  the u-boot makefile has a bug
> in the 1.3.1 release where mkimage depends on sha1.o but doesnt
> actually use sha1 functions.  i posted a patch to u-boot mailing list
> to get this dropped.  regardless, no need for the kernel to import it.

No need to yet anyway.  There are discussions on-going to make a new
image format that can do sha1 sums instead of crc32.  Either way is
fine with me, I just opted to include it now to keep it the same as
U-Boot and avoid having to include it in the future.

If you want an updated patch with the sha1 code removed, I can do
that.  Sam, Wolfgang?

josh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/