Fixes a UB issue when built with GCC and _GLIBCXX_ASSERTIONS turned on. It is specifically that we are passing a nullptr to a function where it is not used, it is only used in the S3 client. Although it will not actually cause runtime UB a compiler will flag it as such, and we should fix it.
Check all that applies:
[x] Did a review by yourself.
[x] Added proper tests to cover this PR. (If tests are not applicable, explain.)
[x] Checked if this PR is a breaking (APIs have been changed) change.
[x] Checked if this PR will not introduce cross-platform inconsistent behavior.
[x] Checked if this PR would require a ReadMe/Wiki update.
Check which platforms you have built SDK on to verify the correctness of this PR.
[x] Linux
[x] Windows
[ ] Android
[x] MacOS
[ ] IOS
[ ] Other Platforms
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Issue #, if available:
issues/2862
Description of changes:
Fixes a UB issue when built with GCC and
_GLIBCXX_ASSERTIONS
turned on. It is specifically that we are passing a nullptr to a function where it is not used, it is only used in the S3 client. Although it will not actually cause runtime UB a compiler will flag it as such, and we should fix it.Check all that applies:
Check which platforms you have built SDK on to verify the correctness of this PR.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.