Skip to content

Commit

Permalink
Fix missing null check in 'Adopt "JDK-8324646: Avoid Class.forName in…
Browse files Browse the repository at this point in the history
… SecureRandom constructor"'
  • Loading branch information
simonis committed Jun 13, 2024
1 parent 0edb942 commit dcecff7
Showing 1 changed file with 6 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -686,10 +686,13 @@ private static Function<String, Class<?>> getConstructorParameterClassAccessor(I
if (engineDescription == null) {
return null;
}
if (consParamClassFieldFinal.getName().equals("constructorParameterClassName")) {
return loader.findClass((String) consParamClassFieldFinal.get(engineDescription)).get();
if (consParamClassFieldFinal.getName().equals("constructorParameterClass")) {
return (Class<?>) consParamClassFieldFinal.get(engineDescription);
}
String constructorParameterClassName = (String) consParamClassFieldFinal.get(engineDescription);
if (constructorParameterClassName != null) {
return loader.findClass(constructorParameterClassName).get();
}
return (Class<?>) consParamClassFieldFinal.get(engineDescription);
} catch (IllegalAccessException e) {
VMError.shouldNotReachHere(e);
}
Expand Down

0 comments on commit dcecff7

Please sign in to comment.