base64: Stop decoding at the first sequence of pad characters
The base64 encoded data cannot included pad characters in the middle, so we can stop the loop at the first sequence of pad characters. If the sequence includes more than two pad characters, the encoding is invalid and we can indicate failure.
This commit is contained in:
parent
c3b75919fa
commit
619e6726ba
1 changed files with 17 additions and 10 deletions
|
@ -1,6 +1,6 @@
|
|||
/*
|
||||
* Base64 encoding/decoding (RFC1341)
|
||||
* Copyright (c) 2005, Jouni Malinen <j@w1.fi>
|
||||
* Copyright (c) 2005-2011, Jouni Malinen <j@w1.fi>
|
||||
*
|
||||
* This program is free software; you can redistribute it and/or modify
|
||||
* it under the terms of the GNU General Public License version 2 as
|
||||
|
@ -103,8 +103,9 @@ unsigned char * base64_encode(const unsigned char *src, size_t len,
|
|||
unsigned char * base64_decode(const unsigned char *src, size_t len,
|
||||
size_t *out_len)
|
||||
{
|
||||
unsigned char dtable[256], *out, *pos, in[4], block[4], tmp;
|
||||
unsigned char dtable[256], *out, *pos, block[4], tmp;
|
||||
size_t i, count, olen;
|
||||
int pad = 0;
|
||||
|
||||
os_memset(dtable, 0x80, 256);
|
||||
for (i = 0; i < sizeof(base64_table) - 1; i++)
|
||||
|
@ -131,7 +132,8 @@ unsigned char * base64_decode(const unsigned char *src, size_t len,
|
|||
if (tmp == 0x80)
|
||||
continue;
|
||||
|
||||
in[count] = src[i];
|
||||
if (src[i] == '=')
|
||||
pad++;
|
||||
block[count] = tmp;
|
||||
count++;
|
||||
if (count == 4) {
|
||||
|
@ -139,14 +141,19 @@ unsigned char * base64_decode(const unsigned char *src, size_t len,
|
|||
*pos++ = (block[1] << 4) | (block[2] >> 2);
|
||||
*pos++ = (block[2] << 6) | block[3];
|
||||
count = 0;
|
||||
}
|
||||
}
|
||||
|
||||
if (pos > out) {
|
||||
if (in[2] == '=')
|
||||
pos -= 2;
|
||||
else if (in[3] == '=')
|
||||
if (pad) {
|
||||
if (pad == 1)
|
||||
pos--;
|
||||
else if (pad == 2)
|
||||
pos -= 2;
|
||||
else {
|
||||
/* Invalid padding */
|
||||
os_free(out);
|
||||
return NULL;
|
||||
}
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
*out_len = pos - out;
|
||||
|
|
Loading…
Reference in a new issue