diff mbox

[U-Boot] md5: Export step-by-step API

Message ID 1882962205.333751.1342791280043.JavaMail.root@advansee.com
State Deferred
Delegated to: Wolfgang Denk
Headers show

Commit Message

Benoît Thébaudeau July 20, 2012, 1:34 p.m. UTC
Export the MD5 step-by-step API so that it is possible for the lib users to
digest files by chunks. This is especially useful for very large files that
would not fit into the embedded RAM.

Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau@advansee.com>
Cc: Wolfgang Denk <wd@denx.de>
---
 .../include/u-boot/md5.h                           |    4 ++++
 {u-boot-66714b1.orig => u-boot-66714b1}/lib/md5.c  |    6 +++---
 2 files changed, 7 insertions(+), 3 deletions(-)

Comments

Mike Frysinger July 20, 2012, 2:44 p.m. UTC | #1
Acked-by: Mike Frysinger <vapier@gentoo.org>
-mike
Benoît Thébaudeau Aug. 3, 2012, 11:09 p.m. UTC | #2
Dear Wolfgang Denk,

On Fri, Jul 20, 2012 at 04:44:03 PM, Mike Frysinger wrote:
> Acked-by: Mike Frysinger <vapier@gentoo.org>
> -mike

Can you apply it now that the merge window is open?

Thanks in advance.

Best regards,
Benoît
Benoît Thébaudeau Aug. 10, 2012, 2:40 p.m. UTC | #3
On Sat, Aug 4, 2012 at 01:09:02 AM, Benoît Thébaudeau wrote:
> Dear Wolfgang Denk,
> 
> On Fri, Jul 20, 2012 at 04:44:03 PM, Mike Frysinger wrote:
> > Acked-by: Mike Frysinger <vapier@gentoo.org>
> > -mike
> 
> Can you apply it now that the merge window is open?
> 
> Thanks in advance.

Can someone answer, please?

Best regards,
Benoît
Wolfgang Denk Sept. 2, 2012, 3:30 p.m. UTC | #4
Dear Benoît Thébaudeau,

In message <1882962205.333751.1342791280043.JavaMail.root@advansee.com> you wrote:
> Export the MD5 step-by-step API so that it is possible for the lib users to
> digest files by chunks. This is especially useful for very large files that
> would not fit into the embedded RAM.
>
> Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau@advansee.com>
> Cc: Wolfgang Denk <wd@denx.de>
> ---
>  .../include/u-boot/md5.h                           |    4 ++++
>  {u-boot-66714b1.orig => u-boot-66714b1}/lib/md5.c  |    6 +++---
>  2 files changed, 7 insertions(+), 3 deletions(-)

As is, there are no users for this code, and we avoid adding dead
code.

Please re-submit with the code that actually uses this feature.

Best regards,

Wolfgang Denk
diff mbox

Patch

diff --git u-boot-66714b1.orig/include/u-boot/md5.h u-boot-66714b1/include/u-boot/md5.h
index e09c16a..e5cb923 100644
--- u-boot-66714b1.orig/include/u-boot/md5.h
+++ u-boot-66714b1/include/u-boot/md5.h
@@ -17,6 +17,10 @@  struct MD5Context {
 	};
 };
 
+void MD5Init(struct MD5Context *ctx);
+void MD5Update(struct MD5Context *ctx, unsigned char const *buf, unsigned len);
+void MD5Final(unsigned char digest[16], struct MD5Context *ctx);
+
 /*
  * Calculate and store in 'output' the MD5 digest of 'len' bytes at
  * 'input'. 'output' must have enough space to hold 16 bytes.
diff --git u-boot-66714b1.orig/lib/md5.c u-boot-66714b1/lib/md5.c
index 2ae4a06..688b725 100644
--- u-boot-66714b1.orig/lib/md5.c
+++ u-boot-66714b1/lib/md5.c
@@ -55,7 +55,7 @@  byteReverse(unsigned char *buf, unsigned longs)
  * Start MD5 accumulation.  Set bit count to 0 and buffer to mysterious
  * initialization constants.
  */
-static void
+void
 MD5Init(struct MD5Context *ctx)
 {
 	ctx->buf[0] = 0x67452301;
@@ -71,7 +71,7 @@  MD5Init(struct MD5Context *ctx)
  * Update context to reflect the concatenation of another buffer full
  * of bytes.
  */
-static void
+void
 MD5Update(struct MD5Context *ctx, unsigned char const *buf, unsigned len)
 {
 	register __u32 t;
@@ -120,7 +120,7 @@  MD5Update(struct MD5Context *ctx, unsigned char const *buf, unsigned len)
  * Final wrapup - pad to 64-byte boundary with the bit pattern
  * 1 0* (64-bit count of bits processed, MSB-first)
  */
-static void
+void
 MD5Final(unsigned char digest[16], struct MD5Context *ctx)
 {
 	unsigned int count;