ID: 50312
Updated by: ras...@php.net
Reported By: kvr at centrum dot cz
-Status: Assigned
+Status: Open
Bug Type: HTTP related
Operating System: Linux, Debian
PHP Version: 5.3SVN-2009-11-27 (snap)
Assigned To: rasmus
New Comment:
kvr, you mean you are worried about nested selects if you are doing a
stream_select() in userspace and internally the SSL reading is
selecting as well?
Previous Comments:
------------------------------------------------------------------------
[2009-11-27 18:56:49] paj...@php.net
btw, if it happens to be the same issue, can you bogus (duplicate) this
one pls?
------------------------------------------------------------------------
[2009-11-27 18:54:58] ras...@php.net
It's not so much a matter of reproducing it as it is reading the
openssl docs and looking at our code. If you read about SSL_read and
SSL_get_error it seems pretty clear that we are not handling errors
efficiently here. We are simply busy-looping on an SSL_ERROR_WANT_READ
which is something you are going to have a hard time reproducing
consistently, but when it happens it sucks.
------------------------------------------------------------------------
[2009-11-27 18:42:36] paj...@php.net
IIRC I did not blew it, I was simply not able to reproduce this
problem, same as here. I did not blindly apply the patch as every
attempt to fix smtg that was not reproducable always had bad side
effect.
I'm still not able to reproduce, Rasmus, feel free to take the hand on
both bugs if you can reproduce these problems and can fix them.
------------------------------------------------------------------------
[2009-11-27 18:37:23] kvr at centrum dot cz
Yes, it looks related. The patch looks quite logical but I'm not sure
if it would work with stream_select() functions.
Anyway, thank you and sorry the bug report was not clear from the
first description.
------------------------------------------------------------------------
[2009-11-27 18:24:29] ras...@php.net
This was reported before and a patch supplied but Pierre blew it off.
See bug #46685
The patch there looks sane. I'll try to get some time this weekend to
play with it.
------------------------------------------------------------------------
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/50312
--
Edit this bug report at http://bugs.php.net/?id=50312&edit=1
