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

Fix a segfault in alpn_select_cb under OCaml 5 #89

Merged
merged 2 commits into from
Aug 11, 2022

Conversation

anmonteiro
Copy link
Collaborator

@anmonteiro anmonteiro commented Aug 11, 2022

This uses @gasche's suggestion from ocaml/ocaml#11485 (comment)

Further follow-up fixes might be needed too.

@anmonteiro anmonteiro changed the title Anmonteiro/fix segfault Fix a segfault in alpn_select_cb under OCaml 5 Aug 11, 2022
@anmonteiro anmonteiro changed the title Fix a segfault in alpn_select_cb under OCaml 5 Fix a segfault in alpn_select_cb under OCaml 5 Aug 11, 2022
@anmonteiro anmonteiro merged commit 16bf6cb into master Aug 11, 2022
@anmonteiro anmonteiro deleted the anmonteiro/fix-segfault branch August 11, 2022 23:38
anmonteiro added a commit to anmonteiro/opam-repository that referenced this pull request Aug 12, 2022
CHANGES:

- Add a few verification functions (savonet/ocaml-ssl#71):
  - `add_extra_chain_cert` to send additional chain certificates to the peer.
  - `add_cert_to_store`: to allow verification of the peer certificate CA.
  - `set_ip`: sets the expected IP address to be verified on a SSL socket.
- Improve `use_certificate_from_string` (savonet/ocaml-ssl#71) to read any type of key (rather
  than just RSA).
- Fix a segmentation fault in the ALPN selection callback under OCaml 5 (savonet/ocaml-ssl#89).
- Audit the C FFI and add `CAMLparamX` and `CAMLreturn` calls (savonet/ocaml-ssl#90).
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