Tobias Stoeckmann
Tobias Stoeckmann
There are multiple occurrences in code where error messages are not set, which result in `(null)` output: ``` $ base64 -d
If realloc fails, keep track of currently allocated memory instead of provoking memory leaks in error paths.
Prevent out of boundary accesses by revalidating offset every time it is incremented.
This actually disabled CRC checks on regular builds and enabled it if CRC checks were supposed to be disabled. All other occurrences properly use ifndef instead of ifdef.
Reaching EOF for first time sets the correct amount of available bytes, but each subsequent call returns 0. Do not forget that the copy buffer can already contain data and...
Turning window_mask into ssize_t and adjusting the signature of circular_memcpy reduces the amount of required casts. Since window_mask depends on value window_size, which is ssize_t, this change is safe. Also...
The parsers for 7zip and xar contain memory leaks. Also the testsuite 7zip leaks memory. Plug them. Noticed while running test suite with address sanitizer on Linux.
*Issue #, if available:* - https://github.com/aws/aws-cli/issues/7143 *Description of changes:* The `relative_path` function can also fail on Unix-like systems if the current working directory has been unlinked. Add this except block...
Some tests are active even if support for tested features is disabled through configuration. To better detect missing decryption support in bsdunzip, I have added a special return value, otherwise...