[11u] RFR 8224487: outputStream should not be copyable
Thomas Stüfe
thomas.stuefe at gmail.com
Thu Jun 13 16:29:02 UTC 2019
Hi all,
I would like to backport to 11u
https://bugs.openjdk.java.net/browse/JDK-8224487.
It is a precondition to backport three other fixes surrounding
stringStream:
- https://bugs.openjdk.java.net/browse/JDK-8224193 (stringStream should
not use Resource Area)
- https://bugs.openjdk.java.net/browse/JDK-8220394 (bufferedStream does
not honor size limit)
- https://bugs.openjdk.java.net/browse/JDK-8225225 (stringStream internal
buffer should always be zero terminated)
Original RFR discussion:
https://mail.openjdk.java.net/pipermail/hotspot-dev/2019-May/038208.html
Original patch: http://hg.openjdk.java.net/jdk/jdk/rev/0927d8c7296f
Full patch (with 11u corrections):
http://cr.openjdk.java.net/~stuefe/webrevs/backports/8224487-make-streams-not-copyable.patch
Delta to original patch:
http://cr.openjdk.java.net/~stuefe/webrevs/backports/8224487-make-streams-not-copyable-11u-changes.patch
The patch disables copy and assignment on outputStream child classes, since
this has been a source of errors (unintended sharing of the stream backing
buffer between two instances of stringStream, for instance). It fixes all
resulting build errors - which mostly indicate real errors.
Patch did not apply cleanly since 11u misses some work in the event log
Coleen did in 12, and a small change Lutz Schmidt did for the code heap
printer.
Thanks for the review.
Cheers, Thomas
More information about the jdk-updates-dev
mailing list