Fix #590 : Check mkdirs() result during WAR assembly

src/main/java/org/apache/maven/plugins/war/AbstractWarMojo.java

@@ -459,7 +459,10 @@ protected String[] getDependentWarIncludes() {
      * @throws MojoFailureException in case of failure
      */
     public void buildExplodedWebapp(File webapplicationDirectory) throws MojoExecutionException, MojoFailureException {
-        webapplicationDirectory.mkdirs();
+        if (!webapplicationDirectory.exists() && !webapplicationDirectory.mkdirs()) {
+            throw new MojoExecutionException(
+                    "Failed to create directory: " + webapplicationDirectory.getAbsolutePath());
+        }
 
         try {
             buildWebapp(project, webapplicationDirectory);

src/main/java/org/apache/maven/plugins/war/packaging/AbstractWarPackagingTask.java

@@ -254,9 +254,12 @@
                     encoding = context.getResourceEncoding();
                 }
                 // fix for MWAR-36, ensures that the parent dir are created first
-                targetFile.getParentFile().mkdirs();
+                File parentDir = targetFile.getParentFile();
+                if (!parentDir.exists() && !parentDir.mkdirs()) {
+                    throw new MojoExecutionException("Failed to create directory: " + parentDir.getAbsolutePath());
+                }
 
                 context.getMavenFileFilter().copyFile(file, targetFile, true, context.getFilterWrappers(), encoding);
             } catch (MavenFilteringException e) {
                 throw new MojoExecutionException(e.getMessage(), e);
             }
@@ -334,7 +337,7 @@
 
                 try {
                     JarArchiver archiver = context.getJarArchiver();
                     archiver.addDirectory(source);
                     archiver.setDestFile(destination);
                     archiver.createArchive();
                 } catch (ArchiverException e) {
@@ -360,7 +363,7 @@
      * @throws java.io.IOException if an error occurred while reading the file
      */
     protected String getEncoding(File webXml) throws IOException {
         try (XmlStreamReader xmlReader = new XmlStreamReader(webXml)) {
             return xmlReader.getEncoding();
         }
     }

src/main/java/org/apache/maven/plugins/war/packaging/ClassesPackagingTask.java

@@ -52,8 +52,8 @@ public ClassesPackagingTask(Overlay currentProjectOverlay) {
     @Override
     public void performPackaging(WarPackagingContext context) throws MojoExecutionException {
         final File webappClassesDirectory = new File(context.getWebappDirectory(), CLASSES_PATH);
-        if (!webappClassesDirectory.exists()) {
-            webappClassesDirectory.mkdirs();
+        if (!webappClassesDirectory.exists() && !webappClassesDirectory.mkdirs()) {
+            throw new MojoExecutionException("Failed to create directory: " + webappClassesDirectory.getAbsolutePath());
         }
 
         if (context.getClassesDirectory().exists()

src/main/java/org/apache/maven/plugins/war/packaging/WarProjectPackagingTask.java

@@ -79,9 +79,13 @@
 
         // Prepare the INF directories
         File webinfDir = new File(context.getWebappDirectory(), WEB_INF_PATH);
-        webinfDir.mkdirs();
+        if (!webinfDir.exists() && !webinfDir.mkdirs()) {
+            throw new MojoExecutionException("Failed to create directory: " + webinfDir.getAbsolutePath());
+        }
         File metainfDir = new File(context.getWebappDirectory(), META_INF_PATH);
-        metainfDir.mkdirs();
+        if (!metainfDir.exists() && !metainfDir.mkdirs()) {
+            throw new MojoExecutionException("Failed to create directory: " + metainfDir.getAbsolutePath());
+        }
 
         handleWebResources(context);
 
@@ -214,7 +218,7 @@
 
                 if (context.isFilteringDeploymentDescriptors()) {
                     context.getMavenFileFilter()
                             .copyFile(
                                     webXml,
                                     new File(webinfDir, "web.xml"),
                                     true,
@@ -230,7 +234,7 @@
                 if (defaultWebXml.exists() && context.isFilteringDeploymentDescriptors()) {
                     context.getWebappStructure().registerFile(id, WEB_INF_PATH + "/web.xml");
                     context.getMavenFileFilter()
                             .copyFile(
                                     defaultWebXml,
                                     new File(webinfDir, "web.xml"),
                                     true,
@@ -246,7 +250,7 @@
 
                 if (context.isFilteringDeploymentDescriptors()) {
                     context.getMavenFileFilter()
                             .copyFile(
                                     containerConfigXML,
                                     new File(metainfDir, xmlFileName),
                                     true,