diff options
| author | Bryan C. Mills <bcmills@google.com> | 2020-05-26 16:11:22 -0400 |
|---|---|---|
| committer | Bryan C. Mills <bcmills@google.com> | 2020-05-28 21:41:10 +0000 |
| commit | 86ed0955bf58ecb738b87892b4377e556e2cc88a (patch) | |
| tree | 065c746910d00fe8527d7e479398dd279873302f /src/encoding/xml | |
| parent | 5a00adf1ea4320bb7f6c2d0b5ae7cf540275c0a4 (diff) | |
| download | go-86ed0955bf58ecb738b87892b4377e556e2cc88a.tar.xz | |
os: in Symlink, stat the correct target path for drive-relative targets on Windows
Previously, when the target (“old”) path passed to os.Symlink was a
“root-relative” Windows path,¹ we would erroneously prepend
destination (“new”) path when determining which path to Stat,
resulting in an invalid path which was then masked by the lack of
error propagation for the Stat call (#39183).
If the link target is a directory (rather than a file), that would
result in the symlink being created without the
SYMBOLIC_LINK_FLAG_DIRECTORY flag, which then fails in os.Open.
¹https://docs.microsoft.com/en-us/windows/win32/fileio/creating-symbolic-links
Updates #39183
Change-Id: I04f179cd2b0c44f984f34ec330acad2408aa3a20
Reviewed-on: https://go-review.googlesource.com/c/go/+/235317
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Diffstat (limited to 'src/encoding/xml')
0 files changed, 0 insertions, 0 deletions
