Anatol,
except for the bug that mysqlnd converts internally the int to uint when passing back through an out desc_num parameter of _mysqlnd_poll() everything else is more or less copied from another part of the distribution. The code exists in 2 places : streamfuncs.c and sockets.c (search for "Solaris + BSD"). Is the code in this places also not correct?

Best,
Andrey

Hi Andrey,

that's true, i also can see some similar code in the cli server. Regarding windows, especially 64 bit, i'd call it "fail ready". In linux there's no disctinction between file descriptor and socket descriptor, both are some integer and 32 bit on x86 and x64 platform. On windows there's ditinction between SOCKET and file descriptor, whereby SOCKET is unsigned and additionally 64 bit on x64 platform. I can't tell why it works in other cases :) ... just haven't time to debug deep. For instance I can imagine this flow could be breaking the max_fd variable in _mysqlnd_poll()

php5ts_debug.dll!php_sockop_cast(_php_stream * stream, int castas, void * * ret, void * * * tsrm_ls) Line 429	C
php5ts_debug.dll!_php_stream_cast(_php_stream * stream, int castas, void * * ret, int show_err, void * * * tsrm_ls) Line 309	C
php5ts_debug.dll!mysqlnd_stream_array_to_fd_set(st_mysqlnd_connection * * conn_array, fd_set * fds, unsigned __int64 * max_fd, void * * * tsrm_ls) Line 1280	C
php5ts_debug.dll!_mysqlnd_poll(st_mysqlnd_connection * * r_array, st_mysqlnd_connection * * e_array, st_mysqlnd_connection * * * dont_poll, long sec, long usec, unsigned int * desc_num, void * * * tsrm_ls) Line 1361	C
php_mysqli.dll!zif_mysqli_poll(int ht, _zval_struct * return_value, _zval_struct * * return_value_ptr, _zval_struct * this_ptr, int return_value_used, void * * * tsrm_ls) Line 800	C

As there in xp_socket.c around line 429 is

*(int*)ret = sock->socket

On x64 windows int is 32 bit, pointer and SOCKET (so php_socket_t) are 64 bit. As one can see, the code is very tricky in that case :)

But the big joke is, that at the other 3-4 places similar code works. Unfortunately earlier i didn't realize the similarity you've pointed to. Maybe that thing should be fixed globally then, as the basic issue on windows is casting int (descritors) vs. SOCKET, which are different things using different APIs. Using SOCKET as descriptor might work but is rather an exception, windows needs two different select()s for those. 

Best regards

Anatol

fixed with da62fd5ed824bafc4dc3e90278c3d57d8e74cbe1