Skip to content

Fix2826 #4676

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

Draft
wants to merge 3 commits into
base: v1.x
Choose a base branch
from
Draft

Fix2826 #4676

wants to merge 3 commits into from

Conversation

geeksilva97
Copy link
Contributor

@geeksilva97 geeksilva97 commented Jan 21, 2025

Continues the work started in #2917

The BSDs let you pass path names longer than sizeof(s.sun_path), they
read past the end when socklen > sizeof(struct sockaddr_un) up to a
limit of at least 255 characters.

This commit refactors src/unix/pipe.c to make it easy to support long
paths. The actual support will be done in a follow-up commit.

Refs: libuv#2826
See the previous commit. This commit allows paths up to 253 bytes
instead of the current restriction of 104 bytes.

Fixes: libuv#2826
@geeksilva97 geeksilva97 marked this pull request as draft January 21, 2025 18:30
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