Merge pull request #1 from SunPower/fix/CM2-2781/path-traversal-vulne…

…rability Fix zip traversal vulnerability
SunPower · Feb 10, 2022 · 62617d4 · 62617d4
2 parents 977b57d + 68725df
commit 62617d4
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/src/android/Zip.java b/src/android/Zip.java
@@ -126,6 +126,14 @@ private void unzipSync(CordovaArgs args, CallbackContext callbackContext) {
                    dir.mkdirs();
                 } else {
                     File file = new File(outputDirectory + compressedName);
+                    String canonicalDestinationPath = (new File(outputDirectory)).getCanonicalPath();
+                    String canonicalPath = file.getCanonicalPath();
+                     if (!canonicalPath.startsWith(canonicalDestinationPath)) {
+                        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());