Summary: | Crash when extracting content from mobipocket file | ||
---|---|---|---|
Product: | [Frameworks and Libraries] kdegraphics-mobipocket | Reporter: | Danilo Araujo <daniloaraujo1990> |
Component: | general | Assignee: | Unassigned bugs <unassigned-bugs-null> |
Status: | RESOLVED WAITINGFORINFO | ||
Severity: | crash | CC: | stefan.bruens, tagwerk19 |
Priority: | NOR | Keywords: | drkonqi |
Version First Reported In: | 2.1.0 | ||
Target Milestone: | --- | ||
Platform: | Neon | ||
OS: | Linux | ||
See Also: | https://bugs.kde.org/show_bug.cgi?id=475975 | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Danilo Araujo
2024-06-27 02:40:29 UTC
(In reply to Danilo Araujo from comment #0) > Application: baloo_file_extractor (6.3.0) > ... > #19 Mobipocket::BitReader::read (this=<optimized out>) at ./lib/decompressor.cpp:69 > #20 Mobipocket::HuffdicDecompressor::unpack (this=0x5798f11b9610, reader=..., depth=1) at ./lib/decompressor.cpp:202 > #21 0x0000777c99aabb64 in Mobipocket::HuffdicDecompressor::unpack (this=0x5798f11b9610, reader=..., depth=0) at ./lib/decompressor.cpp:225 > #22 0x0000777c99aabd0f in Mobipocket::HuffdicDecompressor::decompress (this=this@entry=0x5798f11b9610, data=...) at ./lib/decompressor.cpp:194 > #23 0x0000777c99aaeceb in Mobipocket::Document::text (this=this@entry=0x7fff19a0d008, size=size@entry=-1) at ./lib/mobipocket.cpp:261 > #24 0x000077bc9c0140f9 in KFileMetaData::MobiExtractor::extract (this=<optimized out>, result=0x7fff19a0d490) at ./src/extractors/mobiextractor.cpp:93 There have been a number of .mobi related crashes recently, see Bug 475975 This looks a little different but it still suggest the .mobi extractor needs some more love. You might try to see if Calibre can read and convert the file, https://bugs.kde.org/show_bug.cgi?id=475975#c5 Two possible explanations: - Some distributions have started to enable the QMobipocket depedency for KFileMetaData - Download & Transfer for Kindle eBooks Going Away on February 26 [1] [1] https://blog.the-ebook-reader.com/2025/02/12/download-transfer-for-kindle-ebooks-going-away-on-february-26/ Git commit a188b893654fe5f88b1ebab7e8341ceb181f6dc9 by Stefan Brüns. Committed on 23/02/2025 at 03:54. Pushed by bruns into branch 'disable_mobipocket_text'. [MobiExtractor] Disable buggy text extraction by default The text extraction in mobiextractor is extremely buggy, and causes a lot of bug reports for baloo (which then gets blamed for its "buggyness" when calling third-party code). QMobipocket lacks support for any halfway current mobipocket version (last supported: 4, current: 8), and has no testsuite. Make this opt-in ("ENABLE_MOBIPOCKET_TEXT_EXTRACTION") until the bugs in QMobiPocket gets fixed. SENTRY: BALOO-2N5 SENTRY: BALOO-426 SENTRY: BALOO-33 // use `stack.filename is mobipocket.cpp` for more Related: bug 475975, bug 482420 M +1 -0 CMakeLists.txt M +3 -0 src/extractors/CMakeLists.txt M +2 -1 src/extractors/mobiextractor.cpp https://invent.kde.org/frameworks/kfilemetadata/-/commit/a188b893654fe5f88b1ebab7e8341ceb181f6dc9 Git commit 8bd1e61cca1e07a0ffce7ff79b861e2872662e6d by Stefan Brüns. Committed on 15/03/2025 at 12:16. Pushed by bruns into branch 'master'. [MobiExtractor] Disable buggy text extraction by default The text extraction in mobiextractor is extremely buggy, and causes a lot of bug reports for baloo (which then gets blamed for its "buggyness" when calling third-party code). QMobipocket lacks support for any halfway current mobipocket version (last supported: 4, current: 8), and has no testsuite. Make this opt-in ("ENABLE_MOBIPOCKET_TEXT_EXTRACTION") until the bugs in QMobiPocket gets fixed. SENTRY: BALOO-2N5 SENTRY: BALOO-426 SENTRY: BALOO-33 // use `stack.filename is mobipocket.cpp` for more Related: bug 475975, bug 482420 M +1 -0 CMakeLists.txt M +3 -0 src/extractors/CMakeLists.txt M +2 -1 src/extractors/mobiextractor.cpp https://invent.kde.org/frameworks/kfilemetadata/-/commit/8bd1e61cca1e07a0ffce7ff79b861e2872662e6d A possibly relevant merge request was started @ https://invent.kde.org/graphics/kdegraphics-mobipocket/-/merge_requests/35 Git commit 439a01662e72102e114a46d168fbabbb4de04184 by Stefan Brüns. Committed on 07/06/2025 at 15:19. Pushed by bruns into branch 'master'. Handle trailing data entries correctly Text records may contain extra auxiliary data which should not be fed to the decompressor. The existence of such data is signalled by the `extraflags` header field, and each set bit signals the corresponding extra data which will be present in all text records. The entries can be decoded (or removed) by reading the record from the back. When an entry is present, its size will be at the very end of the record, preceded by the actual data. Related: bug 475975, bug 482420 M +0 -1 autotests/mobipockettest.cpp M +62 -3 lib/mobipocket.cpp https://invent.kde.org/graphics/kdegraphics-mobipocket/-/commit/439a01662e72102e114a46d168fbabbb4de04184 Likely fixed in 25.08. |