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.

Sign up for GitHub

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

Jump to bottom

path/filepath: Clean removes ending slash for volume on Windows in Go 1.21.4 [1.21 backport] #64041

Closed
gopherbot opened this issue Nov 9, 2023 · 3 comments
Closed

path/filepath: Clean removes ending slash for volume on Windows in Go 1.21.4 [1.21 backport] #64041

gopherbot opened this issue Nov 9, 2023 · 3 comments
Labels
CherryPickApproved Used during the release process for point releases release-blocker Security
Milestone
Go1.21.5

Comments

@gopherbot
Copy link
Contributor

@bcmills requested issue #64028 to be considered for backport to the next 1.21 minor release.

@gopherbot, please backport to Go 1.21 and maybe 1.20. This appears to be a regression introduced in a security release.
@gopherbot gopherbot added the CherryPickCandidate Used during the release process for point releases label Nov 9, 2023
@gopherbot gopherbot mentioned this issue Nov 9, 2023
Closed
@gopherbot gopherbot added this to the Go1.21.5 milestone Nov 9, 2023
thaJeztah pushed a commit to thaJeztah/go that referenced this issue Nov 13, 2023
@neild @thaJeztah
[release-branch.go1.21] path/filepath: consider \\?\c: as a volume on…
6405c22 
… Windows

While fixing several bugs in path handling on Windows,
beginning with \\?\.

Prior to #540277, VolumeName considered the first path component
after the \\?\ prefix to be part of the volume name.
After, it considered only the \\? prefix to be the volume name.

Restore the previous behavior.

Fixes golang#64041
Updates golang#64028

Change-Id: I6523789e61776342800bd607fb3f29d496257e68
Reviewed-on: https://go-review.googlesource.com/c/go/+/541175
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Roland Shoemaker <roland@golang.org>
(cherry picked from commit eda42f7)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
thaJeztah pushed a commit to thaJeztah/go that referenced this issue Nov 13, 2023
@neild @thaJeztah
[release-branch.go1.21] path/filepath: consider \\?\c: as a volume on…
db1f17a 
… Windows

While fixing several bugs in path handling on Windows,
beginning with \\?\.

Prior to #540277, VolumeName considered the first path component
after the \\?\ prefix to be part of the volume name.
After, it considered only the \\? prefix to be the volume name.

Restore the previous behavior.

Fixes golang#64041
Updates golang#64028

Change-Id: I6523789e61776342800bd607fb3f29d496257e68
Reviewed-on: https://go-review.googlesource.com/c/go/+/541175
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Roland Shoemaker <roland@golang.org>
(cherry picked from commit eda42f7)
@thaJeztah thaJeztah mentioned this issue Nov 13, 2023
Closed
@gopherbot
Copy link
Contributor Author

Change https://go.dev/cl/541521 mentions this issue: [release-branch.go1.21] path/filepath: consider \\?\c: as a volume on Windows
@gopherbot

This comment was marked as duplicate.

Sign in to view
@heschi heschi added the CherryPickApproved Used during the release process for point releases label Nov 15, 2023
@gopherbot gopherbot removed the CherryPickCandidate Used during the release process for point releases label Nov 15, 2023
@gopherbot
Copy link
Contributor Author

Closed by merging 8caf4bb to release-branch.go1.21.
gopherbot pushed a commit that referenced this issue Nov 28, 2023
@neild @gopherbot
[release-branch.go1.21] path/filepath: consider \\?\c: as a volume on…
8caf4bb 
… Windows

While fixing several bugs in path handling on Windows,
beginning with \\?\.

Prior to #540277, VolumeName considered the first path component
after the \\?\ prefix to be part of the volume name.
After, it considered only the \\? prefix to be the volume name.

Restore the previous behavior.

For #64028.
Fixes #64041.

Change-Id: I6523789e61776342800bd607fb3f29d496257e68
Reviewed-on: https://go-review.googlesource.com/c/go/+/541175
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Roland Shoemaker <roland@golang.org>
(cherry picked from commit eda42f7)
Reviewed-on: https://go-review.googlesource.com/c/go/+/541521
Reviewed-by: Damien Neil <dneil@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CherryPickApproved Used during the release process for point releases release-blocker Security
3 participants
@bcmills @gopherbot @heschi