Bug #78883 [Ver]: fgets(STDIN) fails on Windows

Edit report at https://bugs.php.net/bug.php?id=78883&edit=1 ID: 78883 Updated by: cmb@php.net Reported by: php at nztm dot net Summary: fgets(STDIN) fails on Windows Status: Verified Type: Bug Package: Streams related Operating System: Windows 10 x64 PHP Version: 7.4.0 Assigned To: cmb Block user comment: N Private report: N New Comment: The behavioral change has been introduced with commit d05deaa[1], which has been squashed in to the commit mentioned above. That change is necessary to avoid that ftell() gets called on character files, which would render the return value to be undefined[2]. What has been missed in this commit, is to properly adjust self->is_pipe_blocking (which is a Windows only flag), to avoid calling PeekNamedPipe() on characters files, which is not supported[3]. [1] <https://github.com/php/php-src/pull/4684/commits/d05deaad69dfc1e578bd690923cb573c2e6583c9> [2] <https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/ftell-ftelli64?view=vs-2019#return-value> [3] <https://docs.microsoft.com/en-us/windows/win32/api/namedpipeapi/nf-namedpipeapi-peeknamedpipe#parameters> Previous Comments: ------------------------------------------------------------------------ [2019-11-29 08:49:40] julien dot boudry at gmail dot com Ok, In fact, I have tested under higer level code, so probably you are right. ( https://github.com/symfony/symfony/issues/34702 . This one, I'm sur 100% it's RC5 on two differents Windows 10 up to date installation. I will try to switch on RC4 and tell you if it's working. If I not answer, consider that you are right. ------------------------------------------------------------------------ [2019-11-29 08:34:12] cmb@php.net > This happens since 7.4 RC5, working well on RC4 ! For me, it fails with RC4 as well. Apparently, the issue has been caused by commit 9ec61e4[1], which is shipped as of 7.4.0RC2. [1] <http://git.php.net/?p=php-src.git;a=commit;h=9ec61e43d453aadab9539f6814e8694295aa1ed1> ------------------------------------------------------------------------ [2019-11-29 07:53:35] julien dot boudry at gmail dot com Hi, This happens since 7.4 RC5, working well on RC4 ! ------------------------------------------------------------------------ [2019-11-28 23:10:31] cmb@php.net Thanks for reporting! ------------------------------------------------------------------------ [2019-11-28 19:23:57] php at nztm dot net Description: ------------ On Linux, fgets(STDIN) asks the user for input and returns the results, but this fails on Windows - it immediately returns false. On PHP 7.3 it works on Linux and Windows fine. Test script: --------------- var_dump(fgets(STDIN)); // Try on Windows and Linux, PHP 7.4.0 ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=78883&edit=1