RFR JDK-8170114 jimage extract to not an empty directory overwrites content of the directory
Michal Vala
mvala at redhat.com
Fri Feb 9 15:23:29 UTC 2018
Hi,
sending fix for jimage bug JDK-8170114[1]. I'm not sure whether this is correct
list. If it is not, please direct me somewhere else.
I don't have an openjdk account, so webrev is on my fedora public space. I will
need a sponsor for this.
webrev: https://michalvala.fedorapeople.org/webrevs/JDK-8170114/webrev.00/webrev/
Patch validates output directory before any jimage extracting happen. I've moved
validation to extra private method as it is few lines of code. I've also added
proper error message for case when output path is not a directory
(JImageTask.java#449).
I've implemented beforeTest method, that cleans cwd. This must be done because
files were extracted to cwd and constantly overwriting and tests were
non-deterministic (order did matter). Plus some tests added.
I want to fix JDK-8170120 with IOException when file is not an jimage. It is
blocked by this. This will fix 3 now problematic and excluded tests.
[1] - https://bugs.openjdk.java.net/browse/JDK-8170114
--
Michal Vala
OpenJDK QE
Red Hat Czech
More information about the jigsaw-dev
mailing list