From 716475641ea40e26cbeff877e3986b1febd90b5a Mon Sep 17 00:00:00 2001 From: igerasim Date: Fri, 22 Jun 2018 23:35:22 -0700 Subject: [PATCH] 8205491: adjust reflective access checks Reviewed-by: alanb, mchung, igerasim, rhalade, ahgross --- src/share/classes/java/io/ObjectStreamClass.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/share/classes/java/io/ObjectStreamClass.java b/src/share/classes/java/io/ObjectStreamClass.java index f09d17799..bcbbea261 100644 --- a/src/share/classes/java/io/ObjectStreamClass.java +++ b/src/share/classes/java/io/ObjectStreamClass.java @@ -1534,11 +1534,11 @@ public class ObjectStreamClass implements Serializable { } return false; } else { - // accessible if the parent is public and any constructor - // is protected or public - if ((superCl.getModifiers() & Modifier.PUBLIC) == 0) { + // sanity check to ensure the parent is protected or public + if ((superCl.getModifiers() & (Modifier.PROTECTED | Modifier.PUBLIC)) == 0) { return false; } + // accessible if any constructor is protected or public for (Constructor ctor : superCl.getDeclaredConstructors()) { if ((ctor.getModifiers() & (Modifier.PROTECTED | Modifier.PUBLIC)) != 0) { return true; -- GitLab