src/jdk/nashorn/internal/runtime/Context.java

Print this page
rev 1199 : 8072595: nashorn should not use obj.getClass() for null checks
Reviewed-by: hannesw, attila

*** 58,67 **** --- 58,68 ---- import java.security.ProtectionDomain; import java.util.Collection; import java.util.HashMap; import java.util.LinkedHashMap; import java.util.Map; + import java.util.Objects; import java.util.concurrent.atomic.AtomicLong; import java.util.concurrent.atomic.AtomicReference; import java.util.function.Consumer; import java.util.function.Supplier; import java.util.logging.Level;
*** 902,912 **** * @param sm current security manager instance * @param fullName fully qualified package name * @throw SecurityException if not accessible */ private static void checkPackageAccess(final SecurityManager sm, final String fullName) { ! sm.getClass(); // null check final int index = fullName.lastIndexOf('.'); if (index != -1) { final String pkgName = fullName.substring(0, index); AccessController.doPrivileged(new PrivilegedAction<Void>() { @Override --- 903,913 ---- * @param sm current security manager instance * @param fullName fully qualified package name * @throw SecurityException if not accessible */ private static void checkPackageAccess(final SecurityManager sm, final String fullName) { ! Objects.requireNonNull(sm); final int index = fullName.lastIndexOf('.'); if (index != -1) { final String pkgName = fullName.substring(0, index); AccessController.doPrivileged(new PrivilegedAction<Void>() { @Override