Fix error handling in recursive_directory_iterator::increment

	* src/filesystem/dir.cc (recursive_directory_iterator::increment):
	Reset state on error.
	* testsuite/experimental/filesystem/iterators/
	recursive_directory_iterator.cc: Check state after increment error.

From-SVN: r241552

diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index 7d02cf4815f9601d5735353d750c75266ea5f08d..fadd349bce637251f18f5c4cbc92393a8ea7a679 100644 (file)
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,5 +1,10 @@
 2016-10-26  Jonathan Wakely  <jwakely@redhat.com>
 
+       * src/filesystem/dir.cc (recursive_directory_iterator::increment):
+       Reset state on error.
+       * testsuite/experimental/filesystem/iterators/
+       recursive_directory_iterator.cc: Check state after increment error.
+
        PR libstdc++/78111
        * src/filesystem/ops.cc (canonical): Set error for non-existent path.
 

diff --git a/libstdc++-v3/src/filesystem/dir.cc b/libstdc++-v3/src/filesystem/dir.cc
index 4640d751127130713ce70052653abc1623091654..bcd7dd062ece1a41e3a9bbdbcdd855c2d36f1245 100644 (file)
--- a/libstdc++-v3/src/filesystem/dir.cc
+++ b/libstdc++-v3/src/filesystem/dir.cc
@@ -343,7 +343,10 @@ fs::recursive_directory_iterator::increment(error_code& ec) noexcept
     {
       _Dir dir = open_dir(top.entry.path(), _M_options, &ec);
       if (ec)
-       return *this;
+       {
+         _M_dirs.reset();
+         return *this;
+       }
       if (dir.dirp)
          _M_dirs->push(std::move(dir));
     }

diff --git a/libstdc++-v3/testsuite/experimental/filesystem/iterators/recursive_directory_iterator.cc b/libstdc++-v3/testsuite/experimental/filesystem/iterators/recursive_directory_iterator.cc
index b41c394b41a07a505c90336f682c89d44d741403..79aa178f15264904f36227b9d77b4838956f2645 100644 (file)
--- a/libstdc++-v3/testsuite/experimental/filesystem/iterators/recursive_directory_iterator.cc
+++ b/libstdc++-v3/testsuite/experimental/filesystem/iterators/recursive_directory_iterator.cc
@@ -81,6 +81,7 @@ test01()
   VERIFY( iter->path() == p/"d1/d2" );
   iter.increment(ec);  // should fail to recurse into p/d1/d2
   VERIFY( ec );
+  VERIFY( iter == fs::recursive_directory_iterator() );
 
   // Test inaccessible sub-directory, skipping permission denied.
   iter = fs::recursive_directory_iterator(p, opts, ec);