]> git.ipfire.org Git - thirdparty/suricata.git/commit
util/base64: don't reset decoded bytes in RFC4648 8660/head
authorShivani Bhardwaj <shivani@oisf.net>
Thu, 9 Mar 2023 07:13:41 +0000 (12:43 +0530)
committerVictor Julien <vjulien@oisf.net>
Fri, 31 Mar 2023 17:30:07 +0000 (19:30 +0200)
commit418ddba38e008f0a57c07e7a872d0771d36a9bbd
tree4454dbe2e427b84b6a675ac193e2687582ae50b9
parent8ef410e28452412f344256144dbb69d91a85b700
util/base64: don't reset decoded bytes in RFC4648

Old behavior:
With RFC4648, the decoded bytes were reset to 0 in case an unusual
character was encountered in the encoded string. This worked out fine
for small test cases where there weren't many bytes to be decoded.

Problem:
If a big encoded string had a character outside of the base alphabet,
the processing would stop and the number of decoded bytes were set to 0.
However, even though the processing should stop at the invalid
character, the number of decoded bytes should correctly store the bytes
decoded up until the point an invalid characted was encountered.

New behavor:
For any base64 encoded string given to the base64 decoder in RFC4648
mode, we make sure that the number of decoded bytes correctly reflect
the number of bytes processed up until the string was valid. This makes
sure any further calculations/use of the decoded data is done correctly.

Redmine ticket: 5885
src/util-base64.c