Skip to content
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.

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

mbedtls/ssl: free cert chain when mbedtls_client_preload_filepath ena… #3335

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ThePassionate
Copy link

@ThePassionate ThePassionate commented Feb 12, 2025

hello, I have encountered some issues while using the mbedtls-wrapper for lws, and I would like to propose incorporating the following fixes:

  1. fix the alignment in SSL_CTX_new
  2. release memory if failed to parse cert chain in SSL_CTX_new
  3. release cert chain during SSL_free call

I would appreciate it if these changes could be considered for inclusion in the project. Thank you for your time and consideration.
Best regards.

…bled

Signed-off-by: makejian <makejian@xiaomi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant