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

vendor: github.com/gofrs/flock v0.12.1 #48234

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Jul 24, 2024

vendor: golang.org/x/sys v0.22.0

full diff: golang/sys@v0.21.0...v0.22.0

vendor: github.com/gofrs/flock v0.12.1

  • fix: missing read-write flag in reopenFDOnError
    fixes a regression that could result in a ERROR: bad file descriptor.

gofrs/flock@b659e1e
introduced a regression where f.flag would not be in read-write mode
1 but read-only 2 which breaks people using NFS protocol.

full diff: gofrs/flock@v0.12.0...v0.12.1

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

full diff: golang/sys@v0.21.0...v0.22.0

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- fix: missing read-write flag in reopenFDOnError
  fixes a regression that could result in a `ERROR: bad file descriptor`.

gofrs/flock@b659e1e
introduced a regression where `f.flag` would not be in read-write mode
[1]  but read-only [2] which breaks people using NFS protocol.

[1]: gofrs/flock@b659e1e#diff-87c2c4fe0fb43f4b38b4bee45c1b54cfb694c61e311f93b369caa44f6c1323ffR192
[2]: gofrs/flock@b659e1e#diff-22145325dded38eb5288ed3321a113d8260ccc70747ee04d4551bfd2fba975fdR69

full diff: gofrs/flock@v0.12.0...v0.12.1

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@AkihiroSuda AkihiroSuda merged commit d25a728 into moby:master Jul 25, 2024
153 checks passed
@thaJeztah thaJeztah deleted the vendor_flock branch July 25, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants