Added permission error handling

Guichaguri · Jun 22, 2018 · 361f4ce · 361f4ce
1 parent a7b20ce
commit 361f4ce
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/main/java/com/guichaguri/minimalftp/impl/NativeFileSystem.java b/src/main/java/com/guichaguri/minimalftp/impl/NativeFileSystem.java
@@ -200,9 +200,13 @@ public void rename(File from, File to) throws IOException {
 
     @Override
     public void chmod(File file, int perms) throws IOException {
-        file.setReadable(Utils.hasPermission(perms, Utils.CAT_OWNER + Utils.TYPE_READ), true);
-        file.setWritable(Utils.hasPermission(perms, Utils.CAT_OWNER + Utils.TYPE_WRITE), true);
-        file.setExecutable(Utils.hasPermission(perms, Utils.CAT_OWNER + Utils.TYPE_EXECUTE), true);
+        boolean read = Utils.hasPermission(perms, Utils.CAT_OWNER + Utils.TYPE_READ);
+        boolean write = Utils.hasPermission(perms, Utils.CAT_OWNER + Utils.TYPE_WRITE);
+        boolean execute = Utils.hasPermission(perms, Utils.CAT_OWNER + Utils.TYPE_EXECUTE);
+
+        if(!file.setReadable(read, true)) throw new IOException("Couldn't update the readable permission");
+        if(!file.setWritable(write, true)) throw new IOException("Couldn't update the writable permission");
+        if(!file.setExecutable(execute, true)) throw new IOException("Couldn't update the executable permission");
     }
 
     @Override