[gcc r12-10754] libstdc++: Handle EMLINK and EFTYPE in std::filesystem::remove_all

[Jonathan Wakely via Libstdc++-cvs]

https://gcc.gnu.org/g:135be552d134c47c6fc71b9d8c2eeb98bdd85ede

commit r12-10754-g135be552d134c47c6fc71b9d8c2eeb98bdd85ede
Author: Jonathan Wakely <jwak...@redhat.com>
Date:   Mon Apr 8 17:41:00 2024 +0100

    libstdc++: Handle EMLINK and EFTYPE in std::filesystem::remove_all
    
    Although POSIX requires ELOOP, FreeBSD documents that openat with
    O_NOFOLLOW returns EMLINK if the last component of a filename is a
    symbolic link.  Check for EMLINK as well as ELOOP, so that the TOCTTOU
    mitigation in remove_all works correctly.
    
    See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=214633 or the
    FreeBSD man page for reference.
    
    According to its man page, DragonFlyBSD also uses EMLINK for this error,
    and NetBSD uses its own EFTYPE. OpenBSD follows POSIX and uses EMLINK.
    
    This fixes these failures on FreeBSD:
    FAIL: 27_io/filesystem/operations/remove_all.cc  -std=gnu++17 execution test
    FAIL: experimental/filesystem/operations/remove_all.cc  -std=gnu++17 
execution test
    
    libstdc++-v3/ChangeLog:
    
            * src/c++17/fs_ops.cc (remove_all) [__FreeBSD__ || __DragonFly__]:
            Check for EMLINK as well as ELOOP.
            [__NetBSD__]: Check for EFTYPE as well as ELOOP.

Diff:
---
 libstdc++-v3/src/c++17/fs_ops.cc | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/libstdc++-v3/src/c++17/fs_ops.cc b/libstdc++-v3/src/c++17/fs_ops.cc
index 397c46413bba..0fb7c9e52448 100644
--- a/libstdc++-v3/src/c++17/fs_ops.cc
+++ b/libstdc++-v3/src/c++17/fs_ops.cc
@@ -1310,7 +1310,13 @@ fs::remove_all(const path& p)
     // Our work here is done.
     return 0;
   case ENOTDIR:
-  case ELOOP:
+  case ELOOP:  // POSIX says openat with O_NOFOLLOW sets ELOOP for a symlink.
+#if defined __FreeBSD__ || defined __DragonFly__
+  case EMLINK: // Used instead of ELOOP
+#endif
+#if defined __NetBSD__ && defined EFTYPE
+  case EFTYPE: // Used instead of ELOOP
+#endif
     // Not a directory, will remove below.
     break;
 #endif
@@ -1350,7 +1356,13 @@ fs::remove_all(const path& p, error_code& ec)
     ec.clear();
     return 0;
   case ENOTDIR:
-  case ELOOP:
+  case ELOOP:  // POSIX says openat with O_NOFOLLOW sets ELOOP for a symlink.
+#if defined __FreeBSD__ || defined __DragonFly__
+  case EMLINK: // Used instead of ELOOP
+#endif
+#if defined __NetBSD__ && defined EFTYPE
+  case EFTYPE: // Used instead of ELOOP
+#endif
     // Not a directory, will remove below.
     break;
 #endif