diff --git a/ext/sockets/conversions.c b/ext/sockets/conversions.c index a419a20c873c..eb4e58b3988e 100644 --- a/ext/sockets/conversions.c +++ b/ext/sockets/conversions.c @@ -1385,7 +1385,7 @@ static void from_zval_write_fd_array_aux(zval *elem, unsigned i, void **args, se return; } - iarr[i] = sock->bsd_socket; + iarr[i - 1] = sock->bsd_socket; return; } diff --git a/ext/sockets/tests/socket_cmsg_rights.phpt b/ext/sockets/tests/socket_cmsg_rights.phpt index 1794eaf767ac..8bfbcd57157f 100644 --- a/ext/sockets/tests/socket_cmsg_rights.phpt +++ b/ext/sockets/tests/socket_cmsg_rights.phpt @@ -58,7 +58,7 @@ if ($data["control"]) { if ($control["level"] == SOL_SOCKET && $control["type"] == SCM_RIGHTS) { foreach ($control["data"] as $resource) { - if (!is_resource($resource)) { + if (!is_resource($resource) && !($resource instanceof Socket)) { echo "FAIL RES\n"; var_dump($data); exit; diff --git a/ext/sockets/tests/socket_sendmsg_scm_rights_object.phpt b/ext/sockets/tests/socket_sendmsg_scm_rights_object.phpt new file mode 100644 index 000000000000..6f9868038ebd --- /dev/null +++ b/ext/sockets/tests/socket_sendmsg_scm_rights_object.phpt @@ -0,0 +1,65 @@ +--TEST-- +socket_sendmsg(): SCM_RIGHTS transfers a Socket object with the correct fd +--EXTENSIONS-- +sockets +--SKIPIF-- + +--FILE-- + ['x'], + 'control' => [[ + 'level' => SOL_SOCKET, + 'type' => SCM_RIGHTS, + 'data' => [$payload], + ]], +], 0); + +$data = [ + 'name' => [], + 'buffer_size' => 64, + 'controllen' => socket_cmsg_space(SOL_SOCKET, SCM_RIGHTS, 1), +]; +socket_recvmsg($recv, $data, 0); + +$got = $data['control'][0]['data'][0]; +var_dump($got instanceof Socket); +socket_getsockname($got, $addr); +var_dump($addr === $ppath); +?> +--CLEAN-- + +--EXPECT-- +bool(true) +bool(true)