Very simple code review: 7196677: diff compares same file to itself in PaddingTest regression test.
Weijun Wang
weijun.wang at oracle.com
Fri Sep 7 01:46:17 UTC 2012
http://cr.openjdk.java.net/~weijun/7196677/webrev.00/
Thanks
Max
-------- Original Message --------
7196677: diff compares same file to itself in PaddingTest regression test.
=== *Description*
============================================================
Introduced due to 6330275
Changeset http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c8dbb9e19355
+ private static void diff(String fname1, String fname2) throws Exception {
+ if (!Arrays.equals(Files.readAllBytes(Paths.get(fname1)),
+ Files.readAllBytes(Paths.get(fname1)))) {
+ throw new Exception(
+ "files " + fname1 + " and " + fname2 + " differ");
+ }
+ }
There is a error in the diff, reads n compares fname1 with itself,
instead of fname2
More information about the security-dev
mailing list