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

uhttp: force empty string to random addr #98

Merged

Conversation

aesteve-rh
Copy link
Member

A recent change in cpython broke autobind of abstract
unix socket in linux, causing bind calls with an
empty string to be bound to '\0' instead of a
random address (expected). This results in an
'address already in use' error when consecutive empty
string binding is used.

While we wait for this issue to be solved, we need
to enforce a random address to be used for empty
strings in the abstract space.

Related: python/cpython#94821
Signed-off-by: Albert Esteve aesteve@redhat.com

A recent change in cpython broke autobind of abstract
unix socket in linux, causing bind calls with an
empty string to be bound to '\0' instead of a
random address (expected). This results in an
'address already in use' error when consecutive empty
string binding is used.

While we wait for this issue to be solved, we need
to enforce a random address to be used for empty
strings in the abstract space.

Related: python/cpython#94821
Signed-off-by: Albert Esteve <aesteve@redhat.com>
@aesteve-rh aesteve-rh requested a review from nirs as a code owner July 14, 2022 13:06
@aesteve-rh aesteve-rh self-assigned this Jul 14, 2022
ovirt_imageio/_internal/uhttp.py Show resolved Hide resolved
@nirs nirs merged commit 2d0a8aa into oVirt:master Jul 14, 2022
@aesteve-rh aesteve-rh added this to the ovirt-4.5.2 milestone Jul 15, 2022
@aesteve-rh aesteve-rh deleted the aesteve/uhttp-force-random-address-socket branch August 23, 2022 07:44
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.

2 participants