fix: translate RecursionError to StackError in fallback Unpacker.skip() (#727)

Finding #6 of #683.

Author: hdimer

msgpack/fallback.py

@@ -582,7 +582,10 @@ def __next__(self):
     next = __next__
 
     def skip(self):
-        self._unpack(EX_SKIP)
+        try:
+            self._unpack(EX_SKIP)
+        except RecursionError:
+            raise StackError
         self._consume()
 
     def unpack(self):

test/test_except.py

@@ -97,6 +97,11 @@ def test_invalidvalue():
     with raises(StackError):
         unpackb(b"\x91" * 3000)  # nested fixarray(len=1)
 
+    with raises(StackError):
+        unpacker = Unpacker()
+        unpacker.feed(b"\x91" * 3000)
+        unpacker.skip()
+
 
 def test_no_memory_leak_on_nested_invalid_tag() -> None:
     """Regression test: unpacking nested arrays containing an invalid tag must not leak objects."""