-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Automated cherry pick of #61480: Use O_PATH to avoid errors on Openat #61722
Automated cherry pick of #61480: Use O_PATH to avoid errors on Openat #61722
Conversation
Also add tests for sockets and fifos Replace use of syscall for flags with unix
05b07ee
to
46a2460
Compare
pkg/util/mount/mount_linux_test.go
Outdated
@@ -1395,6 +1462,15 @@ func TestSafeOpen(t *testing.T) { | |||
if err != nil { | |||
t.Fatalf(err.Error()) | |||
} | |||
|
|||
// If base path is too long, creation of unix sockets will fail. | |||
if len(base) > 75 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you still need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No I don't. I am waiting for master PR to get merged, I will simply cherry-pick that one verbatim rather than creating new commit.
/assign |
46a2460
to
b6ab1e2
Compare
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gnufied, msau42, saad-ali The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[MILESTONENOTIFIER] Milestone Pull Request: Up-to-date for process Pull Request Labels
|
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. |
Cherry pick of #61480 on release-1.9.
#61480: Use O_PATH to avoid errors on Openat
Also cherry-pick #61842 to avoid test failures because of long socket paths.