RFR: 8376479: Http3 test server thread deadlock in ThrowingPublishersInRequest [v2]
Daniel Fuchs
dfuchs at openjdk.org
Wed Jan 28 11:10:50 UTC 2026
On Wed, 28 Jan 2026 08:18:33 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/http3/Http3ServerStreamImpl.java line 337:
>>
>>> 335: ByteBuffer buffer = current();
>>> 336: if (buffer == QuicStreamReader.EOF) {
>>> 337: return -1;
>>
>> I don't think that's correct. If we get an error we set the error flag AND add EOF to the buffer list to unblock the reader. Therefore, you must check whether error == null before returning -1 - I believe you should restore lines 334-336 here.
>
> I'm checking that in the `current()` method now.
OK.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29448#discussion_r2736128219
More information about the net-dev
mailing list