file.c: skip invalid byte

* file.c (rb_str_normalize_ospath): skip invalid byte sequence not
  to loop infinitely.  this case usually does not happen as the
  input name should come from real file systems.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@51180 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 7 insertions, 4 deletions

diff --git a/test/-ext-/string/test_normalize.rb b/test/-ext-/string/test_normalize.rb
index 283ca93db7..83bbfd3011 100644
--- a/test/-ext-/string/test_normalize.rb
+++ b/test/-ext-/string/test_normalize.rb
@@ -59,7 +59,6 @@ class Test_StringNormalize < Test::Unit::TestCase
       assert_equal expected, result,
         "#{expected.dump} is expected but #{src.dump}"
     end
-  rescue NotImplementedError
   end
 
   def test_not_normalize_kc
@@ -79,7 +78,6 @@ class Test_StringNormalize < Test::Unit::TestCase
       assert_equal src, result,
         "#{src.dump} is expected not to be normalized, but #{result.dump}"
     end
-  rescue NotImplementedError
   end
 
   def test_dont_normalize_hfsplus
@@ -101,6 +99,11 @@ class Test_StringNormalize < Test::Unit::TestCase
       assert_equal src, result,
         "#{src.dump} is expected not to be normalized, but #{result.dump}"
     end
-  rescue NotImplementedError
   end
-end
+
+  def test_invalid_sequence
+    assert_separately(%w[-r-test-/string/string], <<-'end;')
+      assert_equal("\u{fffd}", Bug::String.new("\xff").normalize_ospath)
+    end;
+  end
+end if Bug::String.method_defined?(:normalize_ospath)