RFR : 8038491: Improve synchronization in ZipFile.read()

Alan Bateman Alan.Bateman at oracle.com
Wed Apr 9 18:36:15 UTC 2014


On 09/04/2014 18:39, Seán Coffey wrote:
> On re-read, I believe extending the sync block in read(..) to cover 
> the reading and setting of the rem variable works to resolve this fix. 
> It should preserve behaviour as well.
>
> http://cr.openjdk.java.net/~coffeys/webrev.8038491.v2/webrev/
This is probably okay for a test case that attempts concurrent reads but 
as it's not synchronized with skip then I don't think you can trust the 
value of rem or pos without taking copies and validating.

-Alan.



More information about the core-libs-dev mailing list