8000 Possible use of unitialized socked address · Issue #132075 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

Possible use of unitialized socked address #132075

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

Closed
serhiy-storchaka opened this issue Apr 4, 2025 · 0 comments
Closed

Possible use of unitialized socked address #132075

serhiy-storchaka opened this issue Apr 4, 2025 · 0 comments
Assignees
Labels
3.12 only security fixes 3.13 bugs and security fixes 3.14 bugs and security fixes type-bug An unexpected behavior, bug, or error

Comments

@serhiy-storchaka
Copy link
Member
serhiy-storchaka commented Apr 4, 2025

Bug report

New Bluetooth tests added in #132023 fail on Fedora based platforms. It looks that the hci_channel member of the sockaddr_hci structure is not initialized. This may be a cause of the failure.

On NetBSD and DragonFly sockaddr_hci is an alias of sockaddr_bt which contain other fields. This also may be an issue.

If sockaddr_hci contains additional optional members, not initializing them with zeros also may cause troubles.

It is safer to always initialize sockaddr_hci and all other sockaddr_* structures with zeroes.

Linked PRs

@serhiy-storchaka serhiy-storchaka added 3.12 only security fixes 3.13 bugs and security fixes 3.14 bugs and security fixes type-bug An unexpected behavior, bug, or error labels Apr 4, 2025
@serhiy-storchaka serhiy-storchaka self-assigned this Apr 4, 2025
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this issue Apr 4, 2025
…alized members

Now all structure members are initialized with zeroes by default.
serhiy-storchaka added a commit that referenced this issue Apr 4, 2025
… members (GH-132076)

Now all structure members are initialized with zeroes by default.
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Apr 4, 2025
…alized members (pythonGH-132076)

Now all structure members are initialized with zeroes by default.
(cherry picked from commit 345baa7)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Apr 4, 2025
…alized members (pythonGH-132076)

Now all structure members are initialized with zeroes by default.
(cherry picked from commit 345baa7)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
serhiy-storchaka added a commit that referenced this issue Apr 4, 2025
…ialized members (GH-132076) (GH-132087)

Now all structure members are initialized with zeroes by default.
(cherry picked from commit 345baa7)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
serhiy-storchaka added a commit that referenced this issue Apr 4, 2025
…ialized members (GH-132076) (GH-132086)

Now all structure members are initialized with zeroes by default.
(cherry picked from commit 345baa7)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
seehwan pushed a commit to seehwan/cpython that referenced this issue Apr 16, 2025
…alized members (pythonGH-132076)

Now all structure members are initialized with zeroes by default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.12 only security fixes 3.13 bugs and security fixes 3.14 bugs and security fixes type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

No branches or pull requests

1 participant
0