Program: 35795
Project: 35795
Buggy source:
/** 
 * Creates a new layer by defining the modules in the given   {@code Configuration} to the Java virtual machine.Each module is mapped, by name, to its class loader by means of the given function. The class loader delegation implemented by these class loaders must respect module readability. The class loaders should be {@link ClassLoader#registerAsParallelCapable parallel-capable} so as toavoid deadlocks during class loading. In addition, the entity creating a new layer with this method should arrange that the class loaders are ready to load from these modules before there are any attempts to load classes or resources. <p> Creating a  {@code Layer} can fail for the following reasons: </p><ul> <li><p> Two or more modules with the same package are mapped to the same class loader. </p></li> <li><p> A module is mapped to a class loader that already has a module of the same name defined to it. </p></li> <li><p> A module is mapped to a class loader that has already defined types in any of the packages in the module. </p></li> </ul> <p> If the function to map a module name to class loader throws an error or runtime exception then it is propagated to the caller of this method. </p>
 * @apiNote It is implementation specific as to whether creating a Layerwith this method is an atomic operation or not. Consequentially it is possible for this method to fail with some modules, but not all, defined to Java virtual machine.
 * @param cf The configuration for the layer
 * @param parentLayers The list parent layers in search order
 * @param clf The function to map a module name to a class loader
 * @return A controller that controls the newly created layer
 * @throws IllegalArgumentException If the parent configurations do not match the configuration of the parent layers, including order
 * @throws LayerInstantiationException If creating the  {@code Layer} fails for any of the reasonslisted above, the layer cannot be created because the configuration contains a module named " {@code java.base}", a module with a package name starting with "  {@code java.}" is mapped to a class loader other than the   {@link ClassLoader#getPlatformClassLoader() platform class loader}, or the function to map a module name to a class loader returns  {@code null}
 * @throws SecurityException If  {@code RuntimePermission("getClassLoader")} is denied bythe security manager
 */
public static Controller defineModules(Configuration cf,List<Layer> parentLayers,Function<String,ClassLoader> clf){
  List<Layer> parents=new ArrayList<>(parentLayers);
  checkConfiguration(cf,parents);
  Objects.requireNonNull(clf);
  checkGetClassLoaderPermission();
  if (boot() != null) {
    checkForDuplicatePkgs(cf,clf);
  }
  try {
    Layer layer=new Layer(cf,parents,clf);
    return new Controller(layer);
  }
 catch (  IllegalArgumentException iae) {
    throw new LayerInstantiationException(iae.getMessage());
  }
}

Fixed version (ground truth):
/** 
 * Creates a new layer by defining the modules in the given   {@code Configuration} to the Java virtual machine.Each module is mapped, by name, to its class loader by means of the given function. The class loader delegation implemented by these class loaders must respect module readability. The class loaders should be {@link ClassLoader#registerAsParallelCapable parallel-capable} so as toavoid deadlocks during class loading. In addition, the entity creating a new layer with this method should arrange that the class loaders are ready to load from these modules before there are any attempts to load classes or resources. <p> Creating a  {@code Layer} can fail for the following reasons: </p><ul> <li><p> Two or more modules with the same package are mapped to the same class loader. </p></li> <li><p> A module is mapped to a class loader that already has a module of the same name defined to it. </p></li> <li><p> A module is mapped to a class loader that has already defined types in any of the packages in the module. </p></li> </ul> <p> If the function to map a module name to class loader throws an error or runtime exception then it is propagated to the caller of this method. </p>
 * @apiNote It is implementation specific as to whether creating a Layerwith this method is an atomic operation or not. Consequentially it is possible for this method to fail with some modules, but not all, defined to Java virtual machine.
 * @param cf The configuration for the layer
 * @param parentLayers The list parent layers in search order
 * @param clf The function to map a module name to a class loader
 * @return A controller that controls the newly created layer
 * @throws IllegalArgumentException If the parent configurations do not match the configuration of the parent layers, including order
 * @throws LayerInstantiationException If creating the  {@code Layer} fails for any of the reasonslisted above, the layer cannot be created because the configuration contains a module named " {@code java.base}", a module with a package name starting with "  {@code java.}" is mapped to a class loader other than the   {@link ClassLoader#getPlatformClassLoader() platform class loader}, or the function to map a module name to a class loader returns  {@code null}
 * @throws SecurityException If  {@code RuntimePermission("getClassLoader")} is denied bythe security manager
 */
public static Controller defineModules(Configuration cf,List<Layer> parentLayers,Function<String,ClassLoader> clf){
  List<Layer> parents=new ArrayList<>(parentLayers);
  checkConfiguration(cf,parents);
  Objects.requireNonNull(clf);
  checkGetClassLoaderPermission();
  if (boot() != null) {
    checkForDuplicatePkgs(cf,clf);
  }
  try {
    Layer layer=new Layer(cf,parents,clf);
    return new Controller(layer);
  }
 catch (  IllegalArgumentException|IllegalStateException e) {
    throw new LayerInstantiationException(e.getMessage());
  }
}