Add class cycle detection during unmarshaling (#946)

Add support for detection of class cycles during unmarshaling in
languages which do no have garbage collection: C++, Swift, and Objective-C.
A `MarshalException` is thrown when a cycle is detected.

The property `Ice.AcceptClassCycles` can be set to a value greater than `0`
to change this behavior.

2 files changed, 16 insertions, 0 deletions

diff --git a/php/src/php5/Communicator.cpp b/php/src/php5/Communicator.cpp
index a7d185e9ccc..41b91656238 100644
--- a/php/src/php5/Communicator.cpp
+++ b/php/src/php5/Communicator.cpp
@@ -1234,6 +1234,14 @@ ZEND_FUNCTION(Ice_initialize)
     initData.compactIdResolver = new IdResolver(TSRMLS_C);
     initData.valueFactoryManager = new ValueFactoryManager;
 
+    if(!initData.properties)
+    {
+        initData.properties = Ice::createProperties();
+    }
+
+    // Always accept cycles in PHP
+    initData.properties->setProperty("Ice.AcceptClassCycles", "1");
+
     CommunicatorInfoIPtr info = initializeCommunicator(return_value, seq, zvargs != 0, initData TSRMLS_CC);
     if(!info)
     {
diff --git a/php/src/php7/Communicator.cpp b/php/src/php7/Communicator.cpp
index 3a0f53ca70b..4143ec65a4c 100644
--- a/php/src/php7/Communicator.cpp
+++ b/php/src/php7/Communicator.cpp
@@ -1245,6 +1245,14 @@ ZEND_FUNCTION(Ice_initialize)
     initData.compactIdResolver = new IdResolver();
     initData.valueFactoryManager = new ValueFactoryManager;
 
+    if(!initData.properties)
+    {
+        initData.properties = Ice::createProperties();
+    }
+
+    // Always accept cycles in PHP
+    initData.properties->setProperty("Ice.AcceptClassCycles", "1");
+
     CommunicatorInfoIPtr info = initializeCommunicator(return_value, seq, zvargs != 0, initData);
     if(!info)
     {