From 296038d49800c8657e7cf718f308541630748e7f Mon Sep 17 00:00:00 2001
From: Andrew Tridgell <tridge@samba.org>
Date: Tue, 1 Sep 1998 06:01:19 +0000
Subject: fixed a bug in the base64 hanlding that led to auth failures for some
 passwords with SWAT (This used to be commit
 edcde70108ab643a29f3e0e0cc97609287da6e87)

---
 source3/web/cgi.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

(limited to 'source3/web/cgi.c')

diff --git a/source3/web/cgi.c b/source3/web/cgi.c
index 20337e0459..ce1038231b 100644
--- a/source3/web/cgi.c
+++ b/source3/web/cgi.c
@@ -431,11 +431,11 @@ decode a base64 string in-place - simple and slow algorithm
 static void base64_decode(char *s)
 {
 	char *b64 = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/";
-	int bit_offset, byte_offset, idx, i;
+	int bit_offset, byte_offset, idx, i, n;
 	unsigned char *d = (unsigned char *)s;
 	char *p;
 
-	i=0;
+	n=i=0;
 
 	while (*s && (p=strchr(b64,*s))) {
 		idx = (int)(p - b64);
@@ -444,13 +444,17 @@ static void base64_decode(char *s)
 		d[byte_offset] &= ~((1<<(8-bit_offset))-1);
 		if (bit_offset < 3) {
 			d[byte_offset] |= (idx << (2-bit_offset));
+			n = byte_offset+1;
 		} else {
 			d[byte_offset] |= (idx >> (bit_offset-2));
 			d[byte_offset+1] = 0;
 			d[byte_offset+1] |= (idx << (8-(bit_offset-2))) & 0xFF;
+			n = byte_offset+2;
 		}
 		s++; i++;
 	}
+	/* null terminate */
+	d[n] = 0;
 }
 
 
-- 
cgit