Re: RFR: 8292327: java.io.EOFException in InflaterInputStream after JDK-8281962 [v3]

2022-08-17 Thread Volker Simonis
On Tue, 16 Aug 2022 20:01:10 GMT, Lance Andersen wrote: >> Volker Simonis has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Changed the payload to a more innocent version > > This looks OK Volker. > > It would be nice if possible to add a

Re: RFR: 8292327: java.io.EOFException in InflaterInputStream after JDK-8281962 [v3]

2022-08-16 Thread Lance Andersen
On Tue, 16 Aug 2022 17:34:58 GMT, Volker Simonis wrote: >> The problem is that after >> [JDK-8281962](https://bugs.openjdk.org/browse/JDK-8281962) we call `fill()` >> unconditionally (and before calling `Inflater::inflate()`) in >> `InflaterInputStream::read()` if `Inflater::needsInput()` is t

Re: RFR: 8292327: java.io.EOFException in InflaterInputStream after JDK-8281962 [v3]

2022-08-16 Thread Alan Bateman
On Tue, 16 Aug 2022 17:34:58 GMT, Volker Simonis wrote: >> The problem is that after >> [JDK-8281962](https://bugs.openjdk.org/browse/JDK-8281962) we call `fill()` >> unconditionally (and before calling `Inflater::inflate()`) in >> `InflaterInputStream::read()` if `Inflater::needsInput()` is t

Re: RFR: 8292327: java.io.EOFException in InflaterInputStream after JDK-8281962 [v3]

2022-08-16 Thread Volker Simonis
> The problem is that after > [JDK-8281962](https://bugs.openjdk.org/browse/JDK-8281962) we call `fill()` > unconditionally (and before calling `Inflater::inflate()`) in > `InflaterInputStream::read()` if `Inflater::needsInput()` is true. This > misses the case where the native inflater has con