Added exception message about which file contains corrupt mode bits

Author: krosenvold

src/main/java/org/codehaus/plexus/archiver/util/ArchiveEntryUtils.java

@@ -146,7 +146,14 @@ public static void chmod( final File file, final int mode, final Logger logger )
     private static void applyPermissionsWithJvm( final File file, final String mode, final Logger logger )
         throws ArchiverException
     {
-        final FilePermission filePermission = FilePermissionUtils.getFilePermissionFromMode( mode, logger );
+        final FilePermission filePermission;
+        try
+        {
+             filePermission = FilePermissionUtils.getFilePermissionFromMode(mode, logger);
+        } catch (IllegalArgumentException e)
+        {
+            throw new ArchiverException("Problem getting permission from mode for " + file.getPath(), e);
+        }
 
         Method method;
         try

src/main/java/org/codehaus/plexus/archiver/util/FilePermissionUtils.java

@@ -57,7 +57,7 @@ public static FilePermission getFilePermissionFromMode( String mode, Logger logg
 
 
 
-        List modes = new ArrayList(mode.length());
+        List<String> modes = new ArrayList<String>(mode.length());
         for (int i = 0,size = mode.length();i<size;i++)
         {
             modes.add( String.valueOf( mode.charAt( i ) ) );
@@ -69,7 +69,7 @@ public static FilePermission getFilePermissionFromMode( String mode, Logger logg
         // handle user perm
         try
         {
-            int userMode = Integer.valueOf((String) modes.get(mode.length() == 4 ? 1 : 0));
+            int userMode = Integer.valueOf(modes.get(mode.length() == 4 ? 1 : 0));
             switch ( userMode )
             {
                 case 0 :
@@ -113,7 +113,7 @@ public static FilePermission getFilePermissionFromMode( String mode, Logger logg
         // handle all perm
         try
         {
-            int allMode = Integer.valueOf((String) modes.get(mode.length() == 4 ? 3 : 2));
+            int allMode = Integer.valueOf(modes.get(mode.length() == 4 ? 3 : 2));
             switch ( allMode )
             {
                 case 0 :