Hi Roger,

Looks good! Just one nit with the test:

 194             registry.unbind(name);
195 Assert.fail("Registry filter should have rejected depth: " + depthOverride + 1);

Technically, the test will also pass if bind() succeed but
unbind() fails - for some unforeseen reason.

best regards,

-- daniel

On 25/05/17 16:31, Roger Riggs wrote:
Please review an update to the RMI Registry built-in serial filter implementation limit on the depth of a instance that can be bound in the registry. The initial depth limit was 5 and it was too limiting for some existing applications. It limit is updated to depth = 20 and should be more than adequate.

Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-depth-max-8180582/index.html

Issue:
    https://bugs.openjdk.java.net/browse/JDK-8180582

Thanks, Roger

Reply via email to