rata73 / theunarchiver

Automatically exported from code.google.com/p/theunarchiver
Other
0 stars 0 forks source link

Crash #744

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
If you try to extract a file that The Unarchiver doesn't support, it'll say it 
can't unarchive that file. Then when you click the okay button, it crashes.

Original issue reported on code.google.com by darrenmo...@gmail.com on 10 Apr 2014 at 8:18

GoogleCodeExporter commented 8 years ago

Original comment by darrenmo...@gmail.com on 10 Apr 2014 at 8:22

Attachments:

GoogleCodeExporter commented 8 years ago
It only happens if you press the return key. If you click, it's fine.

Original comment by darrenmo...@gmail.com on 10 Apr 2014 at 8:25

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
The problem was caused by TUArchiveTaskView. Upon dealloc, it releases objects 
it doesn't own. See patch.

Original comment by darrenmo...@gmail.com on 10 Apr 2014 at 9:45

Attachments:

GoogleCodeExporter commented 8 years ago
No, I am pretty sure it should be releasing those objects, or else they will 
leak. Perhaps they just end up released too early?

Original comment by paracel...@gmail.com on 11 Apr 2014 at 10:23

GoogleCodeExporter commented 8 years ago
That is true. I will try to find the cause. But for now, I've attached a patch 
to fix a memory leak.

P.S. instantiateNibWithOwner:topLevelObjects: is deprecated

Original comment by darrenmo...@gmail.com on 11 Apr 2014 at 5:35

Attachments:

GoogleCodeExporter commented 8 years ago
Finally applied the second patch. Every figure anything out about the rest?

Original comment by paracel...@gmail.com on 19 Jul 2015 at 9:02