On Sun, 11 Sep 2022 08:04:36 GMT, Markus KARG wrote:
> I think you are asking if is safe to leak a reference to the internal buffer.
> If there is no mark then it might be okay because there is no replay for an
> evil output stream to attack. However, I think it would require wider review
> to
On Mon, 26 Sep 2022 21:35:26 GMT, Brian Burkhalter wrote:
>> @bplb Fixed the issues you pointed out. Kindly requesting approval. :-)
>
>> @bplb Fixed the issues you pointed out.
>
> @mkarg You are welcome.
> @bplb I would be very glad if you would `/sponsor` my PR. :-)
Done.
-
PR
On Mon, 26 Sep 2022 21:35:26 GMT, Brian Burkhalter wrote:
>> @bplb Fixed the issues you pointed out. Kindly requesting approval. :-)
>
>> @bplb Fixed the issues you pointed out.
>
> @mkarg You are welcome.
@bplb I would be very glad if you would `/sponsor` my PR. :-)
-
PR: https:/
On Sat, 24 Sep 2022 12:14:49 GMT, Markus KARG wrote:
> @bplb Fixed the issues you pointed out.
@mkarg You are welcome.
-
PR: https://git.openjdk.org/jdk/pull/6935
On Sun, 25 Sep 2022 07:28:56 GMT, Alan Bateman wrote:
>>> I do not quite understand what would be wrong with the code below instead
>>> of falling back to the super implementation _in case of non-empty buffer_?
>>
>> I think you are asking if is safe to leak a reference to the internal
>> buff
On Sun, 25 Sep 2022 07:28:56 GMT, Alan Bateman wrote:
> > @AlanBateman Kindly requesting approval. :-)
>
> I don't have any more comments, the recent test cleanups are good. I assume
> @bplb will sponsor.
Thanks a lot for all your kind help with this PR! :-)
-
PR: https://git.ope
On Sun, 11 Sep 2022 07:44:44 GMT, Alan Bateman wrote:
>> Markus KARG has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - corrected copyright
>> - testing transferTo() after reset()
>
>> I do not quite understand what would be wrong with t
On Thu, 22 Sep 2022 22:14:02 GMT, Brian Burkhalter wrote:
>> Markus KARG has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - corrected copyright
>> - testing transferTo() after reset()
>
> test/jdk/java/io/BufferedInputStream/TransferTo.j
On Fri, 9 Sep 2022 12:19:54 GMT, Markus KARG wrote:
>> Implementation of JDK-8279283
>
> Markus KARG has updated the pull request incrementally with two additional
> commits since the last revision:
>
> - corrected copyright
> - testing transferTo() after reset()
@bplb Fixed the issues you p
On Fri, 9 Sep 2022 12:19:54 GMT, Markus KARG wrote:
>> Implementation of JDK-8279283
>
> Markus KARG has updated the pull request incrementally with two additional
> commits since the last revision:
>
> - corrected copyright
> - testing transferTo() after reset()
test/jdk/java/io/BufferedInp
On Sun, 11 Sep 2022 07:44:44 GMT, Alan Bateman wrote:
>> Markus KARG has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - corrected copyright
>> - testing transferTo() after reset()
>
>> I do not quite understand what would be wrong with t
On Sun, 11 Sep 2022 07:44:44 GMT, Alan Bateman wrote:
> > I do not quite understand what would be wrong with the code below instead
> > of falling back to the super implementation _in case of non-empty buffer_?
>
> I think you are asking if is safe to leak a reference to the internal buffer.
>
On Fri, 9 Sep 2022 12:19:54 GMT, Markus KARG wrote:
>> Implementation of JDK-8279283
>
> Markus KARG has updated the pull request incrementally with two additional
> commits since the last revision:
>
> - corrected copyright
> - testing transferTo() after reset()
> I do not quite understand
> Implementation of JDK-8279283
Markus KARG has updated the pull request incrementally with two additional
commits since the last revision:
- corrected copyright
- testing transferTo() after reset()
-
Changes:
- all: https://git.openjdk.org/jdk/pull/6935/files
- new: https://
14 matches
Mail list logo
