[Bug 224141] possible Off-by-one bug in the Base64 decoding in the wpa supplicant

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=224141

--- Comment #2 from Ivan Kostov <ikostov at gmail.com> ---
Hi Tim,

I have modified the source code a little bit before using it in my project. As
it turns out I have changed the line 


static const unsigned char base64_table[65] = ....

to 

static const unsigned char base64_encode_table[] =

which allocates indeed one byte less :). Sorry for the time lost :D 
However I don't find it sexy, to allocate 65 bytes and initialize only 64 of
them and rely on the fact that static variables usually land in the .bss.
The code will work by initializing 64 elements and removing the " -1 " from the
for loop.

Best regards,
Ivan

-- 
You are receiving this mail because:
You are the assignee for the bug.