X Tutup
Skip to content

Add sockaddr_alg to sock_addr_t#234

Merged
vstinner merged 1 commit intopython:masterfrom
tiran:socket_afalg_def
Feb 22, 2017
Merged

Add sockaddr_alg to sock_addr_t#234
vstinner merged 1 commit intopython:masterfrom
tiran:socket_afalg_def

Conversation

@tiran
Copy link
Copy Markdown
Member

@tiran tiran commented Feb 22, 2017

sock_addr_t is used to define the minimum size of any socket address on
the stack. Let's make sure that an AF_ALG address always fits. Coverity
complains because in theory, AF_ALG might be larger than any of the other
structs. In practice it already fits.

Closes Coverity CID 1398948, 1398949, 1398950

Signed-off-by: Christian Heimes christian@python.org

sock_addr_t is used to define the minimum size of any socket address on
the stack. Let's make sure that an AF_ALG address always fits. Coverity
complains because in theory, AF_ALG might be larger than any of the other
structs. In practice it already fits.

Closes Coverity CID 1398948, 1398949, 1398950

Signed-off-by: Christian Heimes <christian@python.org>
@vstinner vstinner merged commit d37c068 into python:master Feb 22, 2017
@tiran tiran deleted the socket_afalg_def branch February 24, 2017 09:12
Mariatta pushed a commit to Mariatta/cpython that referenced this pull request Mar 7, 2017
sock_addr_t is used to define the minimum size of any socket address on
the stack. Let's make sure that an AF_ALG address always fits. Coverity
complains because in theory, AF_ALG might be larger than any of the other
structs. In practice it already fits.

Closes Coverity CID 1398948, 1398949, 1398950

Signed-off-by: Christian Heimes <christian@python.org>
(cherry picked from commit d37c068)
Mariatta added a commit that referenced this pull request Mar 17, 2017
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
Avoid file descriptor refleaks in as_file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-security A security issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup