Skip to content

Commit cfdfee7

Browse files
committed
8241353: NPE in ToolProvider.getSystemJavaCompiler
Backport-of: 6b4b676
1 parent 0605504 commit cfdfee7

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/java.compiler/share/classes/javax/tools/ToolProvider.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727

2828
import java.security.AccessController;
2929
import java.security.PrivilegedAction;
30-
import java.util.Iterator;
30+
import java.util.Objects;
3131
import java.util.ServiceConfigurationError;
3232
import java.util.ServiceLoader;
3333

@@ -116,8 +116,7 @@ private static <T> T getSystemTool(Class<T> clazz, String moduleName, String cla
116116

117117
try {
118118
ServiceLoader<T> sl = ServiceLoader.load(clazz, ClassLoader.getSystemClassLoader());
119-
for (Iterator<T> iter = sl.iterator(); iter.hasNext(); ) {
120-
T tool = iter.next();
119+
for (T tool : sl) {
121120
if (matches(tool, moduleName))
122121
return tool;
123122
}
@@ -138,7 +137,7 @@ private static <T> boolean matches(T tool, String moduleName) {
138137
PrivilegedAction<Boolean> pa = () -> {
139138
Module toolModule = tool.getClass().getModule();
140139
String toolModuleName = toolModule.getName();
141-
return toolModuleName.equals(moduleName);
140+
return Objects.equals(toolModuleName, moduleName);
142141
};
143142
return AccessController.doPrivileged(pa);
144143
}

0 commit comments

Comments
 (0)