X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/4db03d266bba0da0fa2d96c408fb3bb697b96128..0bbe61b8c18a1795189f0cf73cc61c14a0fb846d:/src/common/base64.cpp?ds=inline diff --git a/src/common/base64.cpp b/src/common/base64.cpp index 89025b0b56..2832bffb3b 100644 --- a/src/common/base64.cpp +++ b/src/common/base64.cpp @@ -88,7 +88,7 @@ wxBase64Decode(void *dst_, size_t dstLen, PAD }; - static const char decode[256] = + static const unsigned char decode[256] = { WSP,INV,INV,INV,INV,INV,INV,INV,INV,WSP,WSP,INV,WSP,WSP,INV,INV, INV,INV,INV,INV,INV,INV,INV,INV,INV,INV,INV,INV,INV,INV,INV,INV, @@ -118,7 +118,7 @@ wxBase64Decode(void *dst_, size_t dstLen, const char *p; for ( p = src; srcLen; p++, srcLen-- ) { - const char c = decode[(int)*p]; // cast just to suppress warnings + const unsigned char c = decode[wx_static_cast(unsigned char, *p)]; switch ( c ) { case WSP: @@ -132,7 +132,7 @@ wxBase64Decode(void *dst_, size_t dstLen, // force the loop to stop and an error to be returned n = -1; - srcLen = 0; + srcLen = 1; break; case PAD: @@ -156,7 +156,7 @@ wxBase64Decode(void *dst_, size_t dstLen, { // force the loop terminate with an error n = -1; - srcLen = 0; + srcLen = 1; } break; @@ -165,7 +165,7 @@ wxBase64Decode(void *dst_, size_t dstLen, { // nothing is allowed after the end so provoke error return n = -1; - srcLen = 0; + srcLen = 1; break; } @@ -194,7 +194,11 @@ wxBase64Decode(void *dst_, size_t dstLen, if ( n ) { if ( posErr ) - *posErr = p - src; + { + // notice that the error was on a previous position as we did one + // extra "p++" in the loop line after it + *posErr = p - src - 1; + } return wxCONV_FAILED; }