when taking ownership of stdin and stdout in PipeStream, the stdin an…

…d stdout must be replaced by devnull as on one hand the code executed could write to it but also we will close the channels and thus cause exit code failures

rpyc/core/stream.py

@@ -326,7 +326,10 @@ def from_std(cls):
 
         :returns: a :class:`PipeStream` instance
         """
-        return cls(sys.stdin, sys.stdout)
+        pipestream = cls(sys.stdin, sys.stdout)
+        sys.stdin = os.open(os.devnull, os.O_RDWR)
+        sys.stdout = sys.stdin
+        return pipestream
 
     @classmethod
     def create_pair(cls):
@@ -405,7 +408,10 @@ def __init__(self, incoming, outgoing):
 
     @classmethod
     def from_std(cls):
-        return cls(sys.stdin, sys.stdout)
+        pipestream = cls(sys.stdin, sys.stdout)
+        sys.stdin = os.open(os.devnull, os.O_RDWR)
+        sys.stdout = sys.stdin
+        return pipestream
 
     @classmethod
     def create_pair(cls):

rpyc/lib/colls.py

@@ -37,7 +37,6 @@ def __getitem__(self, key):
         obj = self._dict[key]()
         if obj is None:
             raise KeyError(key)
-        self[key] = obj
         return obj
 
     def __setitem__(self, key, value):