Ensure path is set for rmdir to prevent accidents (Fixes #756)

Signed-off-by: Jeroen van Erp <[email protected]>
hierynomus · May 8, 2023 · 350c73e · 350c73e
1 parent 0b926eb
commit 350c73e
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/main/java/com/hierynomus/smbj/share/DiskShare.java b/src/main/java/com/hierynomus/smbj/share/DiskShare.java
@@ -379,6 +379,10 @@ public boolean isSuccess(long statusCode) {
      * Remove the directory at the given path.
      */
     public void rmdir(String path, boolean recursive) throws SMBApiException {
+        if (path == null || path.isEmpty()) {
+            throw new IllegalArgumentException("rmdir: path should be non-null and non-empty");
+        }
+
         if (recursive) {
             List<FileIdBothDirectoryInformation> list = list(path);
             for (FileIdBothDirectoryInformation fi : list) {