Thanks you Chris for reviewing this!
On 4/18/18 9:06 AM, Chris Hegarty wrote:
Ivan,
On 16/04/18 17:29, Ivan Gerasimov wrote:
...
WEBREV: http://cr.openjdk.java.net/~igerasim/8201510/00/webrev/
I think this is mostly good. Just one comment.
I'm not sure that this is correct.
--- OLD ---
60 String exclBindProp = AccessController.doPrivileged(
61 new GetPropertyAction("sun.net.useExclusiveBind",
""));
62 exclusiveBind = (exclBindProp.isEmpty())
63 ? true
64 : Boolean.parseBoolean(exclBindProp);
--- NEW ---
private static final boolean useExclusiveBind =
55 Boolean.parseBoolean(AccessController.doPrivileged(
56 new GetPropertyAction("sun.net.useExclusiveBind",
"true")));
Exclusive bind should be true iif:
1) it is defined and has no value, or
2) if is defined and has a value of `true`.
Oh. Good catch! Thanks!
I restored the logic here in the updated webrev.
I thought we had tests for this, but maybe not if you are not
seeing test failures.
I found only two tests that set useExclusiveBind. They seem to ignore
false positive results, that's why they didn't fail.
I updated them to test with useExclusiveBind set either to 'true' or to
the empty value.
Here's the updated webrev:
WEBREV: http://cr.openjdk.java.net/~igerasim/8201510/01/webrev/
With kind regards,
Ivan
-Chris.
--
With kind regards,
Ivan Gerasimov