Unzip AES support intentionally restricted to a specific vendor id? #11
Labels
1.2
Related to older version of minizip 1.2
encryption
Encryption/decryption issue
fixed
Issue or bug has been fixed
Nice work putting all this together. I encountered only one small issue trying to use this library. My 7zip AES encrypted zip file would not unzip properly. I traced the problem down to unz64local_GetCurrentFileInfoInternal.
Lines:
The code tests if vendor id != 1 and if so it returns an error. Just my opinion, but I think the test should be removed entirely. Otherwise it should be noted that 7zip is compatible and uses 2 so the test could be changed to be uL != 1 && ul != 2.
If safety is the concern, one is probably better off making sure that file_info_internal.aes_encryption_mode and file_info_internal.aes_compression_method are being set to valid values rather than checking the vendor id. Just my opinion though.
The text was updated successfully, but these errors were encountered: