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

ipv6_link_local: Ignore ifaddrs with NULL ifa_addr #4218

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

sgallagher
Copy link
Contributor

Passing this to uv__is_ipv6_link_local() is causing a segmentation fault. Note that the documentation for getifaddrs() explicitly states that this value may be NULL.

src/unix/tcp.c Outdated Show resolved Hide resolved
Passing this to uv__is_ipv6_link_local() is causing a segmentation
fault. Note that the documentation for getifaddrs() explicitly states
that this value may be NULL.

Signed-off-by: Stephen Gallagher <sgallagh@redhat.com>
Copy link
Member

@bnoordhuis bnoordhuis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@bnoordhuis
Copy link
Member

Test failure seems to be a flaky test. I've opened #4219 to track that.

@bnoordhuis bnoordhuis merged commit 31e4b90 into libuv:v1.x Nov 14, 2023
25 of 26 checks passed
sgallagher added a commit to sgallagher/libuv-1 that referenced this pull request Nov 16, 2023
Passing this to uv__is_ipv6_link_local() is causing a segmentation
fault. Note that the documentation for getifaddrs() explicitly states
that this value may be NULL.

Signed-off-by: Stephen Gallagher <sgallagh@redhat.com>
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.

None yet

3 participants