Godzil / theunarchiver

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

Crash upon extracting UTF-8 named files/directories #697

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
% unar arhiva.zip 
arhiva.zip: Zip
unar: Uncaught exception NSCharacterConversionException, reason: Can't get 
cString from Unicode string.

The example archive contains a file with UTF-8 characters in it's name (any of 
these: "ć", "č", "š", "ž", "đ", as well as Serbian Cyrilic characters).

lsar is OK.

The problem is also extracting to a directory with UTF-8 characters in its name.

Original issue reported on code.google.com by pej...@gmail.com on 29 Aug 2013 at 7:33

GoogleCodeExporter commented 8 years ago
What platform? Sounds like a Foundation implementation bug rather than a bug in 
unar, really.

Original comment by paracel...@gmail.com on 29 Aug 2013 at 7:36

GoogleCodeExporter commented 8 years ago
Arch Linux, 64 bit. Yes, it occured to me to it could be a bug in GNUStep (or 
something).

There's also another bug report for the same matter here: 
https://bugzilla.redhat.com/show_bug.cgi?id=989150

I'll check when did the GNU Step got upgraded...

Original comment by pej...@gmail.com on 29 Aug 2013 at 7:40

GoogleCodeExporter commented 8 years ago
Lots of Linux distros seem to ship terribly outdated GNUSteps. If you have the 
time, try building a new one and see if it still breaks.

Original comment by paracel...@gmail.com on 29 Aug 2013 at 7:44

GoogleCodeExporter commented 8 years ago
The last gnustep update (gnustep-base-1.24.4) for Arch Linux was in May. 
Probably the issues started then, but I didn't notice until now.
However, I've upgraded to the latest version (1.24.5) and the problems remain. 
Don't know what to do next. :/

Original comment by pej...@gmail.com on 29 Aug 2013 at 8:15

GoogleCodeExporter commented 8 years ago
Perhaps GNUstep is failing to figure out that your system should use UTF-8 
filenames? I seem to vaguely recall that there is some option that can be used 
to override that, but I forgot what it is.

It might be worth reporting it to GNUStep, too.

Original comment by paracel...@gmail.com on 29 Aug 2013 at 8:17

GoogleCodeExporter commented 8 years ago
Don't know, it used to work fine before. I'm gonna try with a previous versions 
of gnustep.

Original comment by pej...@gmail.com on 29 Aug 2013 at 8:29

GoogleCodeExporter commented 8 years ago
I've tried version 1.24.3 - 1.24.5, and the same error. I could not compile 
version 1.24.0. So, I've reported the bug to the gnustep bug tracker:

https://savannah.gnu.org/bugs/index.php?39900

Original comment by pej...@gmail.com on 30 Aug 2013 at 1:03

GoogleCodeExporter commented 8 years ago
Well, it seems it is an issue in unar, after all. I've got helped by people 
(https://savannah.gnu.org/bugs/index.php?39900) and came up with a patch 
(attached) that solved my problems.

So, any coments?

Original comment by pej...@gmail.com on 31 Aug 2013 at 12:34

Attachments:

GoogleCodeExporter commented 8 years ago
Seems like that would fix it, yes, but I am not entirely convinced it should be 
necessary. I'll add it in the next round of updates, but I'll try to talk to 
the GNUstep people about it.

Original comment by paracel...@gmail.com on 31 Aug 2013 at 12:55

GoogleCodeExporter commented 8 years ago
Applied.

Original comment by paracel...@gmail.com on 7 Sep 2013 at 4:48