DotNetZip falsches Passwort beim Entpacken

Question

Szenario: Datei hochladen als versuchen, zip es mit DotNetZip mit Passwortschutz, Passwort wird mit Membership.GeneratePassword() Methode generiert. Alles funktioniert einwandfrei, außer dass der Benutzer manchmal Dateien mit dem generierten Passwort nicht entpacken kann. Wired Sache ist, dass dies passiert nur manchmal sagen wir 1 von 15 mal. Generieren Passwort:

public static String FilePassword() 
{ 
    while (_filePassword.Length < 12) 
    { 
     _filePassword += string.Concat(Membership.GeneratePassword(1, 0).Where(char.IsLetterOrDigit)); 
    } 

    return _filePassword; 
} 

Datei speichern:

if (FileUploadControl.HasFile) 
{ 

    fileName = Path.GetFileName(FileUploadControl.FileName); 
    FileUploadControl.SaveAs(FileSavePath + fileName); 

     // Archive uploaded file to zip. 
     using (ZipFile zip = new ZipFile()) 
     { 
      // File to be archived. 
      var file = FileUploadControl.PostedFile; 

      // Enable encryption of file 
      zip.Encryption = EncryptionAlgorithm.PkzipWeak; 

      // Set password. 
      zip.Password = Settings.FilePassword(); 

      // Set temporary folder path for archive process. 
      zip.TempFileFolder = tempPath; 

      // Add file to archive with its path. 
      zip.AddFile(FileSavePath + file.FileName, ""); 

      File objFile = new File(file.FileName, FileSavePath); 

      // Save zip file with the name as file ID. 
      zip.Save(FileSavePath + file.FileName); 
     } 
} 

ich Passwort angemeldet, während Verfahren zu schaffen, in und auch ZIP-Datei mit einem Passwort, während der Schutz, sie passen immer, ich kann nicht sehen, was falsch ist, warum manchmal Beim Entpacken der Datei wird ein falsches Passwort angezeigt.

Answer 1

Warum verwenden Sie die statische globale Variable _filePassword in FilePassword() statt einer im Geltungsbereich?

So könnte es von außen modifiziert werden, oder sogar noch den zuletzt verwendeten Wert enthalten. Es ist auch nicht Thread-sicher ohne lock.

Abrechnen mit einer lokalen Variablen und es sollte in Ordnung sein.

public static String FilePassword() 
{ 
    string retString = string.Empty; 
    while (retString.Length < 12) 
    { 
     retString += string.Concat(Membership.GeneratePassword(1, 0).Where(char.IsLetterOrDigit)); 
    }  
    return retString; 
} 

Sie können auch einen Rückgabewert protokollieren.

---

Beispiel für das Verständnis

if (FileUploadControl.HasFile) 
{ 

    fileName = Path.GetFileName(FileUploadControl.FileName); 
    FileUploadControl.SaveAs(FileSavePath + fileName); 
    string filePassword = Settings.FilePassword(); // Contains the Password 

    using (ZipFile zip = new ZipFile()) 
    { 
     var file = FileUploadControl.PostedFile; 
     zip.Encryption = EncryptionAlgorithm.PkzipWeak;    
     zip.Password = filePassword; // <-- Set password for ZIP 
     zip.TempFileFolder = tempPath; 
     zip.AddFile(FileSavePath + file.FileName, ""); 
     File objFile = new File(file.FileName, FileSavePath); 
     zip.Save(FileSavePath + file.FileName); 
    } 

    // Log this value! 
    Log(filePassword); 
}