From 7aa28485c422e1dae14299ff40967b80e2606235 Mon Sep 17 00:00:00 2001
From: Francesco Paolo Severino <francescopaolo.severino@gmail.com>
Date: Tue, 10 Sep 2024 09:49:29 +0200
Subject: [PATCH] Try again removing colons

---
 Sources/Zip/Zip.swift | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/Sources/Zip/Zip.swift b/Sources/Zip/Zip.swift
index d65184a3..1fc89cd0 100644
--- a/Sources/Zip/Zip.swift
+++ b/Sources/Zip/Zip.swift
@@ -95,6 +95,11 @@ public class Zip {
             fileName[Int(fileInfo.size_filename)] = 0
 
             var pathString = String(cString: fileName)
+
+            #if os(Windows)
+            pathString = pathString.replacingOccurrences(of: ":", with: "_")
+            #endif
+
             guard !pathString.isEmpty else {
                 throw ZipError.unzipFail
             }
@@ -110,7 +115,7 @@ public class Zip {
             }
 
             let fullPath = destination.appendingPathComponent(pathString).standardizedFileURL.path
-            // `.standardized` removes any `..` to move a level up.
+            // `.standardizedFileURL` removes any `..` to move a level up.
             // If we then check that the `fullPath` starts with the destination directory we know we are not extracting "outside" the destination.
             guard fullPath.starts(with: destination.standardizedFileURL.path) else {
                 throw ZipError.unzipFail
@@ -162,8 +167,6 @@ public class Zip {
                 throw ZipError.unzipFail
             }
 
-            // TODO: Set permissions properly on Windows
-            #if !os(Windows)
             // Set file permissions from current `fileInfo`
             if fileInfo.external_fa != 0 {
                 let permissions = (fileInfo.external_fa >> 16) & 0x1FF
@@ -176,7 +179,6 @@ public class Zip {
                     }
                 }
             }
-            #endif
 
             ret = unzGoToNextFile(zip)