[PATCH] PR libstdc++/88881 fix filesystem::symlink_status for Windows
Jonathan Wakely
jwakely@redhat.com
Wed May 29 22:02:00 GMT 2019
The fix for PR 88881 only added a workaround to filesystem::status, but
filesystem::symlink_status is also affected by the _wstat bug and needs
the same workaround.
The recent change to optimize path::parent_path() means that the
workaround can be simplified to just use parent_path().
PR libstdc++/88881
* src/c++17/fs_ops.cc [_GLIBCXX_FILESYSTEM_IS_WINDOWS]
(status(const path&, error_code&)): Use parent_path() to remove
trailing slash.
(symlink_status(const path&, error_code&)): Duplicate workaround for
bug in _wstat for paths with trailing slash.
* testsuite/27_io/filesystem/operations/remove_all.cc: Check path
with trailing slash.
* testsuite/27_io/filesystem/operations/status.cc: Likewise.
* testsuite/27_io/filesystem/operations/symlink_status.cc: Likewise.
Tested powerpc64le-linux and x86_64-w64-mingw32, committed to trunk.
I'll backport this to gcc-9-branch too.
-------------- next part --------------
commit eb7855646dfce7bec919977ec57e1e413b7890a6
Author: Jonathan Wakely <jwakely@redhat.com>
Date: Wed May 29 22:02:55 2019 +0100
PR libstdc++/88881 fix filesystem::symlink_status for Windows
The fix for PR 88881 only added a workaround to filesystem::status, but
filesystem::symlink_status is also affected by the _wstat bug and needs
the same workaround.
The recent change to optimize path::parent_path() means that the
workaround can be simplified to just use parent_path().
PR libstdc++/88881
* src/c++17/fs_ops.cc [_GLIBCXX_FILESYSTEM_IS_WINDOWS]
(status(const path&, error_code&)): Use parent_path() to remove
trailing slash.
(symlink_status(const path&, error_code&)): Duplicate workaround for
bug in _wstat for paths with trailing slash.
* testsuite/27_io/filesystem/operations/remove_all.cc: Check path
with trailing slash.
* testsuite/27_io/filesystem/operations/status.cc: Likewise.
* testsuite/27_io/filesystem/operations/symlink_status.cc: Likewise.
diff --git a/libstdc++-v3/src/c++17/fs_ops.cc b/libstdc++-v3/src/c++17/fs_ops.cc
index 274ee7f0834..d8064819d36 100644
--- a/libstdc++-v3/src/c++17/fs_ops.cc
+++ b/libstdc++-v3/src/c++17/fs_ops.cc
@@ -1395,23 +1395,19 @@ fs::status(const fs::path& p, error_code& ec) noexcept
#if ! defined __MINGW64_VERSION_MAJOR || __MINGW64_VERSION_MAJOR < 6
// stat() fails if there's a trailing slash (PR 88881)
path p2;
- if (p.has_relative_path())
+ if (p.has_relative_path() && !p.has_filename())
{
- wstring_view s = p.native();
- const auto len = s.find_last_not_of(L"/\\") + wstring_view::size_type(1);
- if (len != 0 && len != s.length())
+ __try
{
- __try
- {
- p2.assign(s.substr(0, len));
- }
- __catch(const bad_alloc&)
- {
- ec = std::make_error_code(std::errc::not_enough_memory);
- return status;
- }
+ p2 = p.parent_path();
str = p2.c_str();
}
+ __catch(const bad_alloc&)
+ {
+ ec = std::make_error_code(std::errc::not_enough_memory);
+ return status;
+ }
+ str = p2.c_str();
}
#endif
#endif
@@ -1440,8 +1436,31 @@ fs::file_status
fs::symlink_status(const fs::path& p, std::error_code& ec) noexcept
{
file_status status;
+ auto str = p.c_str();
+
+#if _GLIBCXX_FILESYSTEM_IS_WINDOWS
+#if ! defined __MINGW64_VERSION_MAJOR || __MINGW64_VERSION_MAJOR < 6
+ // stat() fails if there's a trailing slash (PR 88881)
+ path p2;
+ if (p.has_relative_path() && !p.has_filename())
+ {
+ __try
+ {
+ p2 = p.parent_path();
+ str = p2.c_str();
+ }
+ __catch(const bad_alloc&)
+ {
+ ec = std::make_error_code(std::errc::not_enough_memory);
+ return status;
+ }
+ str = p2.c_str();
+ }
+#endif
+#endif
+
stat_type st;
- if (posix::lstat(p.c_str(), &st))
+ if (posix::lstat(str, &st))
{
int err = errno;
ec.assign(err, std::generic_category());
diff --git a/libstdc++-v3/testsuite/27_io/filesystem/operations/remove_all.cc b/libstdc++-v3/testsuite/27_io/filesystem/operations/remove_all.cc
index 119dd3dc783..a19bac9c5f6 100644
--- a/libstdc++-v3/testsuite/27_io/filesystem/operations/remove_all.cc
+++ b/libstdc++-v3/testsuite/27_io/filesystem/operations/remove_all.cc
@@ -108,9 +108,42 @@ test02()
VERIFY( !exists(dir) );
}
+void
+test03()
+{
+ // PR libstdc++/88881 symlink_status confused by trailing slash on Windows
+ const std::error_code bad_ec = make_error_code(std::errc::invalid_argument);
+ unsigned removed;
+ std::error_code ec = bad_ec;
+ const auto p = __gnu_test::nonexistent_path() / ""; // with trailing slash
+
+ create_directories(p);
+ removed = remove_all(p, ec);
+ VERIFY( !ec );
+ VERIFY( removed == 1 );
+ VERIFY( !exists(p) );
+ create_directories(p);
+ removed = remove_all(p);
+ VERIFY( removed == 1 );
+ VERIFY( !exists(p) );
+
+ const auto p_subs = p/"foo/bar";
+ ec = bad_ec;
+ create_directories(p_subs);
+ removed = remove_all(p, ec);
+ VERIFY( !ec );
+ VERIFY( removed == 3 );
+ VERIFY( !exists(p) );
+ create_directories(p_subs);
+ remove_all(p);
+ VERIFY( removed == 3 );
+ VERIFY( !exists(p) );
+}
+
int
main()
{
test01();
test02();
+ test03();
}
diff --git a/libstdc++-v3/testsuite/27_io/filesystem/operations/status.cc b/libstdc++-v3/testsuite/27_io/filesystem/operations/status.cc
index 38c0f65370c..b65a2f20906 100644
--- a/libstdc++-v3/testsuite/27_io/filesystem/operations/status.cc
+++ b/libstdc++-v3/testsuite/27_io/filesystem/operations/status.cc
@@ -93,10 +93,20 @@ test03()
fs::permissions(dir, fs::perms::owner_all, ec);
}
+void
+test04()
+{
+ // PR libstdc++/88881
+ fs::path p = "./";
+ auto st = status(p);
+ VERIFY( is_directory(st) );
+}
+
int
main()
{
test01();
test02();
test03();
+ test04();
}
diff --git a/libstdc++-v3/testsuite/27_io/filesystem/operations/symlink_status.cc b/libstdc++-v3/testsuite/27_io/filesystem/operations/symlink_status.cc
index 6f01419da3e..c097e4f2bf0 100644
--- a/libstdc++-v3/testsuite/27_io/filesystem/operations/symlink_status.cc
+++ b/libstdc++-v3/testsuite/27_io/filesystem/operations/symlink_status.cc
@@ -111,10 +111,20 @@ test03()
fs::permissions(dir, fs::perms::owner_all, ec);
}
+void
+test04()
+{
+ // PR libstdc++/88881
+ fs::path p = "./";
+ auto st = symlink_status(p);
+ VERIFY( is_directory(st) );
+}
+
int
main()
{
test01();
test02();
test03();
+ test04();
}
More information about the Libstdc++
mailing list