From: Dan Egnor <egnor@ofb.net>
Date: Wed, 6 Feb 2002 13:29:52 +0000 (+0100)
Subject: (base64_decode_init): New function.
X-Git-Tag: nettle_1.6_release_20021003~373
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cc64ece77b0f40d3f4bf1a1d2922f21e54ee940d;p=thirdparty%2Fnettle.git

(base64_decode_init): New function.
(base64_decode_update): New function, replacing base64_decode.
Takes a struct base64_ctx argument.

Rev: src/nettle/base64.c:1.2
---

diff --git a/base64.c b/base64.c
index 3e6609e2..4ec6c6af 100644
--- a/base64.c
+++ b/base64.c
@@ -94,38 +94,45 @@ base64_encode(uint8_t *dst,
   return out - dst;
 }
 
+void
+base64_decode_init(struct base64_ctx *ctx)
+{
+  ctx->shift = 10;
+  ctx->accum = 0;
+}
+
 unsigned
-base64_decode(uint8_t *dst,
-              unsigned src_length,
-              const uint8_t *src)
+base64_decode_update(struct base64_ctx *ctx,
+                     uint8_t *dst,
+                     unsigned src_length,
+                     const uint8_t *src)
 {
   uint8_t *out = dst;
-  uint32_t accum = 0;
-  int shift = 10;
 
-  while (src_length > 0)
+  for (;;) 
     {
-      const int data = decode_table[*src];
+      int data;
+      if (src_length == 0) return out - dst;
+      data = decode_table[*src];
       switch (data)
         {
         default:
-          accum |= data << shift;
-          shift -= 6;
-          if (shift <= 2)
+          ctx->accum |= data << ctx->shift;
+          ctx->shift -= 6;
+          if (ctx->shift <= 2)
             {
-              *out++ = accum >> 8;
-              accum = accum << 8;
-              shift += 8;
+              *out++ = ctx->accum >> 8;
+              ctx->accum <<= 8;
+              ctx->shift += 8;
             }
+	  /* Fall through */
         case TABLE_INVALID:
         case TABLE_SPACE:
         case TABLE_END:
-          break;
+	  /* FIXME: Silently ignores any invalid characters.
+	   * We need to detect and return errors, in some way. */
+          ++src;
+          --src_length;
         }
-
-        ++src;
-        --src_length;
     }
-
-  return out - dst;
 }