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: fix CURLOPT_SSLCERT_BLOB

Closed
wants to merge 1 commit into from
Closed

Conversation

bagder
Copy link
Member

@bagder bagder commented Dec 14, 2021

The memory passed to mbedTLS for this needs to be null terminated.

Reported-by: Florian Van Heghe

The memory passed to mbedTLS for this needs to be null terminated.

Reported-by: Florian Van Heghe
@bagder bagder added the TLS label Dec 14, 2021
@bagder bagder closed this in 867ad1c Dec 14, 2021
@bagder bagder deleted the bagder/mbedtls-ssl_cert_blob branch Dec 14, 2021
jay pushed a commit to jay/curl that referenced this issue Jan 11, 2022
- Increase the buffer length passed to mbedtls_x509_crt_parse to account
  for the null byte appended to the temporary blob.

Follow-up to 867ad1c which uses a null terminated copy of the
certificate blob, because mbedtls_x509_crt_parse requires PEM data
to be null terminated.

Ref: curl@867ad1c#r63439893
Ref: curl#8146

Closes #xxxx
jay pushed a commit to jay/curl that referenced this issue Jan 12, 2022
- Increase the buffer length passed to mbedtls_x509_crt_parse to account
  for the null byte appended to the temporary blob.

Follow-up to 867ad1c which uses a null terminated copy of the
certificate blob, because mbedtls_x509_crt_parse requires PEM data
to be null terminated.

Ref: curl@867ad1c#r63439893
Ref: curl#8146

Closes #xxxx
jay pushed a commit that referenced this issue Jan 15, 2022
- Increase the buffer length passed to mbedtls_x509_crt_parse to account
  for the null byte appended to the temporary blob.

Follow-up to 867ad1c which uses a null terminated copy of the
certificate blob, because mbedtls_x509_crt_parse requires PEM data
to be null terminated.

Ref: 867ad1c#r63439893
Ref: #8146

Closes #8260
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant