#21197 [Fbk->Ver]: [PATCH] socket_read() outputs error with PHP_NORMAL_READ

ID: 21197 Updated by: nlopess@php.net Reported By: bool at boolsite dot net -Status: Feedback +Status: Verified Bug Type: Sockets related Operating System: * PHP Version: 5.*, 4.* (2005-11-17) (cvs) New Comment: nothing has changed in the code: it still needs to be patched, because of fcntl() on windows. Previous Comments: ------------------------------------------------------------------------ [2005-12-05 19:04:07] sniper@php.net Please try using this CVS snapshot: http://snaps.php.net/php5.1-latest.tar.gz For Windows: http://snaps.php.net/win32/php5.1-win32-latest.zip ------------------------------------------------------------------------ [2005-11-05 22:10:22] nlopess@php.net I've made a patch that implements the idea of my last comment. The test case now works properly. I haven't tested bug #35062 but it seems to be a different bug. http://mega.ist.utl.pt/~ncpl/php_sockets_win.txt ------------------------------------------------------------------------ [2005-11-05 01:36:39] nlopess@php.net I was walking through the MSDN docs and I didn't also find anything. But the best way (IMHO) is to store in the php_socket struct if the socket is blocking or not (and update that field in the socket_set_(non)block() functions). It also saves the fcntl syscall on nix systems. ------------------------------------------------------------------------ [2005-11-04 19:30:44] mike@php.net Here's a possible patch, but Wez probably knows better if there's a way to tell if a windows socket is in blocking mode... Index: sockets.c =================================================================== RCS file: /repository/php-src/ext/sockets/sockets.c,v retrieving revision 1.171.2.2 diff -u -p -d -r1.171.2.2 sockets.c --- sockets.c 3 Nov 2005 15:00:51 -0000 1.171.2.2 +++ sockets.c 4 Nov 2005 18:28:45 -0000 @@ -257,6 +257,12 @@ static int php_read(int bsd_socket, void int nonblock = 0; char *t = (char *) buf; +/* + * fcntl(s, F_GETFL) will always fail for windows, and there's no way to + * determine if a socket is in blocking mode to my current knowledge, so we + * just omit this check; though that means we're always blocking on win32... + */ +#ifndef PHP_WIN32 m = fcntl(bsd_socket, F_GETFL); if (m < 0) { return m; @@ -264,6 +270,7 @@ static int php_read(int bsd_socket, void nonblock = (m & O_NONBLOCK); m = 0; +#endif set_errno(0); ------------------------------------------------------------------------ [2005-11-04 16:24:47] sniper@php.net See also bug #35062 ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/21197 -- Edit this bug report at http://bugs.php.net/?id=21197&edit=1