You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Crash when using wrong password to open encrypted zip file #51
Hey, thanks for reaching out. I tried reproducing the crash in a test case (which I pushed to a new branch named issue51), but it passes. Can you try modifying that test case so it fails, demonstrating your crash? Also, is this happening for Mac or iOS (and, if iOS, device or simulator)?
@segunlee Thanks for letting me know! Can you help me reproduce this? If possible, could you send me the archive you're having trouble with, and your code that's causing the crash? I have test cases for incorrect passwords, and they don't crash.
@monobono@segunlee Are either of you able to help me reproduce this in a test case? You can checkout the issue51 branch, which has a test case named testValidatePassword_Issue51 I created to help get you started.
@amosavian I'm going through open issues now, so you'll see me replying to a bunch of them. Sorry to hit you all at once :)
Regarding this one, I see what you proposed in issue #56, but as I just commented there, I would like to have a test case that fails, demonstrating the bug. As I did with the other one, I created an issue51 branch in which I attempted to reproduce the issue, but the test passes. Can you, @segunlee or @monobono please modify it to reproduce this bug? If not, I'll have to close the issue out.
@abbeycode to create test case as I suggested in #56, You must have a file bigger than 4096 bytes after compressing (size of a chunk) in a password encrypted archive.
I somehow missed the 4096 byte comment back then. That's probably all I need to continue on it. Thanks! I'll let you know how it goes and if I have any other questions.
Activity
Added test case that attempts to reproduce Issue #51, but it passes
abbeycode commentedon Feb 1, 2017
Hey, thanks for reaching out. I tried reproducing the crash in a test case (which I pushed to a new branch named
issue51), but it passes. Can you try modifying that test case so it fails, demonstrating your crash? Also, is this happening for Mac or iOS (and, if iOS, device or simulator)?abbeycode commentedon Apr 24, 2017
Is this still a problem for you?
segunlee commentedon Jun 12, 2017
i got same issue,
1.png file

abbeycode commentedon Jun 12, 2017
@segunlee Thanks for letting me know! Can you help me reproduce this? If possible, could you send me the archive you're having trouble with, and your code that's causing the crash? I have test cases for incorrect passwords, and they don't crash.
abbeycode commentedon Aug 8, 2017
@monobono @segunlee Are either of you able to help me reproduce this in a test case? You can checkout the
issue51branch, which has a test case namedtestValidatePassword_Issue51I created to help get you started.amosavian commentedon Aug 27, 2017
@abbeycode I described how to resolve problem in #56. Just amend your code with my diff 😃
abbeycode commentedon Oct 7, 2017
@amosavian I'm going through open issues now, so you'll see me replying to a bunch of them. Sorry to hit you all at once :)
Regarding this one, I see what you proposed in issue #56, but as I just commented there, I would like to have a test case that fails, demonstrating the bug. As I did with the other one, I created an issue51 branch in which I attempted to reproduce the issue, but the test passes. Can you, @segunlee or @monobono please modify it to reproduce this bug? If not, I'll have to close the issue out.
amosavian commentedon Oct 7, 2017
@abbeycode to create test case as I suggested in #56, You must have a file bigger than 4096 bytes after compressing (size of a chunk) in a password encrypted archive.
abbeycode commentedon Oct 7, 2017
I somehow missed the 4096 byte comment back then. That's probably all I need to continue on it. Thanks! I'll let you know how it goes and if I have any other questions.
amosavian commentedon Oct 7, 2017
Thanks in advance ❤️❤️❤️
Added test case that attempts to reproduce Issue #51, but it passes
Updated testValidatePassword_Issue51 so it breaks now, demonstrating …
abbeycode commentedon Oct 12, 2017
I was able to reproduce, and fix. Look for it in the first v1.9 beta!