Skip to content

Fix AF_INET6 accept() address handling#568

Closed
mrexodia wants to merge 2 commits intomachinezone:masterfrom
mrexodia:ipv6-accept-fix
Closed

Fix AF_INET6 accept() address handling#568
mrexodia wants to merge 2 commits intomachinezone:masterfrom
mrexodia:ipv6-accept-fix

Conversation

@mrexodia
Copy link
Copy Markdown
Contributor

6 year old FIXME mrexodia@b213063

@mrexodia
Copy link
Copy Markdown
Contributor Author

mrexodia commented Feb 20, 2026

CI failure was fixed by pinning mbedtls to v3 on brew.

@mrexodia mrexodia changed the title Fix AF_INET6 accept(0 address handling Fix AF_INET6 accept() address handling Feb 20, 2026
@bsergean
Copy link
Copy Markdown
Contributor

Is this similar to #571 ?

@cosminpolifronie
Copy link
Copy Markdown
Contributor

The fix is the same as in #571, so I would say yes.

@mrexodia
Copy link
Copy Markdown
Contributor Author

mrexodia commented Apr 25, 2026

It’s the same but CI is actually passing 😉 (and the my code is cleaner)

@cosminpolifronie
Copy link
Copy Markdown
Contributor

Ah, my bad, my PR is missing this PR's 4eec026 commit.

@cosminpolifronie
Copy link
Copy Markdown
Contributor

It’s the same but CI is actually passing 😉 (and the my code is cleaner)

No need for the snarky reply. The other PR is fixing a different issue and I happened to need an additional fix that you have already done but didn't see. I can revert my commit and they can use your PR for that specific part, I don't care.

@mrexodia mrexodia closed this Apr 25, 2026
@mrexodia
Copy link
Copy Markdown
Contributor Author

As long as the issue gets fixed I'm happy 🤷🏻‍♂️

@andymillermz
Copy link
Copy Markdown
Collaborator

Thanks, all!

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.

4 participants