merge zip

MobileChromeApps#92
wnabil · Feb 11, 2021 · c54aa43 · c54aa43
1 parent 977b57d
commit c54aa43
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/android/Zip.java b/src/android/Zip.java
@@ -126,6 +126,13 @@ private void unzipSync(CordovaArgs args, CallbackContext callbackContext) {
                    dir.mkdirs();
                 } else {
                     File file = new File(outputDirectory + compressedName);
+                    String canonicalPath = file.getCanonicalPath();
+                    if (!canonicalPath.startsWith(outputDirectory)) {
+                        String errorMessage = "Zip traversal security error";
+                        callbackContext.error(errorMessage);
+                        Log.e(LOG_TAG, errorMessage);
+                        return;
+                    }
                     file.getParentFile().mkdirs();
                     if(file.exists() || file.createNewFile()){
                         Log.w("Zip", "extracting: " + file.getPath());