New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix NULL config clean-up #620

Merged

waahm7 merged 1 commit into main from fix-s2n-destroy

Jan 29, 2024

+3 −2

Contributor

waahm7 commented Jan 29, 2024

Description of changes:

The s2n cleanup function https://github.com/aws/s2n-tls/blob/c74f442b55589872b1374559bb03878c49761ca6/tls/s2n_config.c#L122 doesn't check for NULL, so we should do it before calling the cleanup function.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.


          fix null config clean-up

116be91

DmitriyMusatkin approved these changes

View reviewed changes

waahm7 merged commit 4c65ce5 into main

33 checks passed

waahm7 deleted the fix-s2n-destroy branch

January 29, 2024 21:18

alfred2g pushed a commit that referenced this pull request


          Fix NULL config clean-up (#620)

aa00ac0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet