Skip to content

[3.12] gh-106687: _ssl: use uint64_t for SSL options (GH-106700)#106827

Merged
vstinner merged 1 commit intopython:3.12from
miss-islington:backport-ad95c72-3.12
Jul 17, 2023
Merged

[3.12] gh-106687: _ssl: use uint64_t for SSL options (GH-106700)#106827
vstinner merged 1 commit intopython:3.12from
miss-islington:backport-ad95c72-3.12

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Jul 17, 2023

SSL_CTX_get_options() uses uint64_t for options:
https://www.openssl.org/docs/man3.1/man3/SSL_CTX_get_options.html

Fix this compiler warning on Windows with MSC:

conversion from 'uint64_t' to 'long', possible loss of data

(cherry picked from commit ad95c72)

Co-authored-by: Victor Stinner vstinner@python.org

SSL_CTX_get_options() uses uint64_t for options:
https://www.openssl.org/docs/man3.1/man3/SSL_CTX_get_options.html

Fix this compiler warning on Windows with MSC:

    conversion from 'uint64_t' to 'long', possible loss of data
(cherry picked from commit ad95c72)

Co-authored-by: Victor Stinner <vstinner@python.org>
@vstinner
Copy link
Copy Markdown
Member

@gpshead prefers to backport this change before Python 3.12 final: #106700 (comment)

@vstinner vstinner merged commit 497bfd5 into python:3.12 Jul 17, 2023
@miss-islington miss-islington deleted the backport-ad95c72-3.12 branch July 17, 2023 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants