Disable clone handlers for Redis and RedisCluster.

michael-grunder · michael-grunder · commit cd05a3441fa1 · 2021-02-01T11:21:07.000-08:00
Presently PhpRedis will segfault if users attempt to clone either the
Redis or RedisCluster objects.

We plan on investigating proper clone support in future releases, but
for now just diable the handlers.
diff --git a/redis.c b/redis.c
@@ -611,6 +611,7 @@ create_redis_object(zend_class_entry *ce)
     memcpy(&redis_object_handlers, zend_get_std_object_handlers(), sizeof(redis_object_handlers));
     redis_object_handlers.offset = XtOffsetOf(redis_object, std);
     redis_object_handlers.free_obj = free_redis_object;
+    redis_object_handlers.clone_obj = NULL;
     redis->std.handlers = &redis_object_handlers;
 
     return &redis->std;
diff --git a/redis_cluster.c b/redis_cluster.c
@@ -333,6 +333,7 @@ zend_object * create_cluster_context(zend_class_entry *class_type) {
     memcpy(&RedisCluster_handlers, zend_get_std_object_handlers(), sizeof(RedisCluster_handlers));
     RedisCluster_handlers.offset = XtOffsetOf(redisCluster, std);
     RedisCluster_handlers.free_obj = free_cluster_context;
+    RedisCluster_handlers.clone_obj = NULL;
 
     cluster->std.handlers = &RedisCluster_handlers;
 
