Fixed Python bug related to invalid return parameters, fixes #550

4 files changed, 33 insertions, 0 deletions

diff --git a/python/test/Ice/exceptions/AllTests.py b/python/test/Ice/exceptions/AllTests.py
index 3aa59ab16f4..762058d61ef 100644
--- a/python/test/Ice/exceptions/AllTests.py
+++ b/python/test/Ice/exceptions/AllTests.py
@@ -813,6 +813,25 @@ def allTests(helper, communicator):
 
     print("ok")
 
+    sys.stdout.write("catching unknown non-Ice exception with futures... ")
+    sys.stdout.flush()
+    try:
+        try:
+            thrower.throwMarshalException(context={"response":""})
+        except Ice.UnknownLocalException as ex:
+            test("::Ice::MarshalException" in str(ex))
+        try:
+            thrower.throwMarshalException(context={"param":""})
+        except Ice.UnknownLocalException as ex:
+            test("::Ice::MarshalException" in str(ex))
+        try:
+            thrower.throwMarshalException()
+        except Ice.UnknownLocalException as ex:
+            test("::Ice::MarshalException" in str(ex))
+    except Ice.OperationNotExistException:
+        pass
+    print("ok")
+
     sys.stdout.write("catching exact types with AMI mapping... ")
     sys.stdout.flush()
 
diff --git a/python/test/Ice/exceptions/ServerAMD.py b/python/test/Ice/exceptions/ServerAMD.py
index 8bd9a6f35a8..b9418e7e29d 100755
--- a/python/test/Ice/exceptions/ServerAMD.py
+++ b/python/test/Ice/exceptions/ServerAMD.py
@@ -136,6 +136,12 @@ class ThrowerI(Test.Thrower):
         f.set_exception(Test.A())
         return f
 
+    def throwMarshalException(self, current):
+        if "return" in current.ctx:
+            return Ice.Future.completed(("", 0))
+        if "param" in current.ctx:
+            return Ice.Future.completed((0, ""))
+        return Ice.Future.completed(None)
 
 class ServerAMD(TestHelper):
 
diff --git a/python/test/Ice/exceptions/Test.ice b/python/test/Ice/exceptions/Test.ice
index adbcdcf3c9c..c9513cfa3ec 100644
--- a/python/test/Ice/exceptions/Test.ice
+++ b/python/test/Ice/exceptions/Test.ice
@@ -71,6 +71,8 @@ interface Thrower
 
     void throwAfterResponse();
     void throwAfterException() throws A;
+
+    int throwMarshalException(out int p);
 }
 
 interface WrongOperation
diff --git a/python/test/Ice/exceptions/TestI.py b/python/test/Ice/exceptions/TestI.py
index d4ea54c5df1..65860b6cad6 100644
--- a/python/test/Ice/exceptions/TestI.py
+++ b/python/test/Ice/exceptions/TestI.py
@@ -101,3 +101,9 @@ class ThrowerI(Test.Thrower):
         # Only relevant for AMD.
         #
         raise Test.A()
+
+    def throwMarshalException(self, current):
+        if "return" in current.ctx:
+            return ("", 0)
+        if "param" in current.ctx:
+            return (0, "")