Skip to content

[3.10] bpo-44493: Add missing terminated NUL in sockaddr_un's length (GH-26866) #32140

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

Merged
merged 2 commits into from
Mar 28, 2022

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Mar 27, 2022

Add missing terminated NUL in sockaddr_un's length

Co-authored-by: ty zonyitoo@users.noreply.github.com

https://bugs.python.org/issue44493

Automerge-Triggered-By: GH:gpshead

…GH-26866)

Add missing terminated NUL in sockaddr_un's length

- Linux: https://man7.org/linux/man-pages/man7/unix.7.html
- *BSD: SUN_LEN
(cherry picked from commit f6b3a07)

Co-authored-by: ty <zonyitoo@users.noreply.github.com>
@miss-islington
Copy link
Contributor Author

@zonyitoo and @gpshead: Status check is done, and it's a success ✅ .

@bedevere-bot bedevere-bot added the type-bug An unexpected behavior, bug, or error label Mar 27, 2022
@miss-islington
Copy link
Contributor Author

@zonyitoo and @gpshead: Status check is done, and it's a failure ❌ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@zonyitoo and @gpshead: Status check is done, and it's a failure ❌ .

@miss-islington
Copy link
Contributor Author

@zonyitoo and @gpshead: Status check is done, and it's a failure ❌ .

@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: 3 of 6 required status checks are expected..

@gpshead gpshead added the needs backport to 3.9 only security fixes label Mar 28, 2022
@gpshead gpshead self-assigned this Mar 28, 2022
@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 5944807 into python:3.10 Mar 28, 2022
@miss-islington
Copy link
Contributor Author

Thanks @miss-islington for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@miss-islington miss-islington deleted the backport-f6b3a07-3.10 branch March 28, 2022 18:26
miss-islington added a commit to miss-islington/cpython that referenced this pull request Mar 28, 2022
…ythonGH-26866) (pythonGH-32140)

Add missing terminated NUL in sockaddr_un's length

- Linux: https://man7.org/linux/man-pages/man7/unix.7.html
- *BSD: SUN_LEN
(cherry picked from commit f6b3a07)

Co-authored-by: ty <zonyitoo@users.noreply.github.com>

Automerge-Triggered-By: GH:gpshead
(cherry picked from commit 5944807)

Co-authored-by: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
@bedevere-bot
Copy link

GH-32156 is a backport of this pull request to the 3.9 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Mar 28, 2022
gpshead pushed a commit that referenced this pull request Mar 28, 2022
…H-26866) (GH-32140) (GH-32156)

Add missing terminated NUL in sockaddr_un's length

- Linux: https://man7.org/linux/man-pages/man7/unix.7.html
- *BSD: SUN_LEN
(cherry picked from commit f6b3a07)

Co-authored-by: ty <zonyitoo@users.noreply.github.com>

Automerge-Triggered-By: GH:gpshead
(cherry picked from commit 5944807)

Co-authored-by: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
…ythonGH-26866) (pythonGH-32140) (pythonGH-32156)

Add missing terminated NUL in sockaddr_un's length

- Linux: https://man7.org/linux/man-pages/man7/unix.7.html
- *BSD: SUN_LEN
(cherry picked from commit f6b3a07)

Co-authored-by: ty <zonyitoo@users.noreply.github.com>

Automerge-Triggered-By: GH:gpshead
(cherry picked from commit 5944807)

Co-authored-by: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants