Summary: | ark crashes when I open a specific rar archive | ||
---|---|---|---|
Product: | [Applications] ark | Reporter: | Rami <ramib85> |
Component: | general | Assignee: | Harald Hvaal <metellius> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | rakuco |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 4.4.5 | |
Sentry Crash Report: |
Description
Rami
2010-06-18 14:27:18 UTC
here is a link to the archive that causes the crash : http://www.mediafire.com/?zmzy4zji2hu by the way,the file opens without errors in windows. One more piece of information: the archive contains a tar archive inside. It actually looks related to some extended attributes listed in the archive, such as ":Zone.Identifier" and ":encryptable". Was it you who created the archive? no, it's not me. Can you please list all the files you get when you extract this archive on Windows? Does any of them have "Zone.Identifier" or "encryptable" in their names? folder: NLP3 that contains the following files: Automatic Evaluation of Summaries Using N-gram Co-Occurrence Statistics.ppt Duc07.doc duc07.results.data.tar NLP09 - Summarization.pdf README.txt Thumbs.db tokenizationAndSegmentation.ppt SVN commit 1140173 by rkcosta: Revamp the clirar plugin. This was prompted by bug 242071: RAR files have this concept of subheaders, which include comments, NTFS streams and other things I have no idea since the format is undocumented. Subheaders can be ignored during the listing, but they confused the old parser code because they were unexpected and broke the expected listing pattern. In order to fix that, we now call unrar with 'vt' instead of just 'v' so that we get more information. However, this means more lines are output for each entry, so our "parser" needed to be improved. The code looks much cleaner and (hopefully) easier to understand now. Furthermore, the old code probably ignored files whose names started with '*'. Finally, some TODOs have been added for some aspects we should watch out when dealing with the code. BUG: 242071 M +90 -41 cliplugin.cpp M +9 -6 cliplugin.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1140173 SVN commit 1140180 by rkcosta: Backport r1140173. Revamp the clirar plugin. This was prompted by bug 242071: RAR files have this concept of subheaders, which include comments, NTFS streams and other things I have no idea since the format is undocumented. Subheaders can be ignored during the listing, but they confused the old parser code because they were unexpected and broke the expected listing pattern. In order to fix that, we now call unrar with 'vt' instead of just 'v' so that we get more information. However, this means more lines are output for each entry, so our "parser" needed to be improved. The code looks much cleaner and (hopefully) easier to understand now. Furthermore, the old code probably ignored files whose names started with '*'. Finally, some TODOs have been added for some aspects we should watch out when dealing with the code. CCBUG: 242071 M +90 -41 cliplugin.cpp M +9 -6 cliplugin.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1140180 |