https://bugs.kde.org/show_bug.cgi?id=410092

Elvis Angelaccio <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/util
                   |                            |ities/ark/commit/a41e69b30c
                   |                            |c07dd758849f8685d322150459e
                   |                            |4f1
   Version Fixed In|                            |20.12.2
             Status|CONFIRMED                   |RESOLVED

--- Comment #7 from Elvis Angelaccio <[email protected]> ---
Git commit a41e69b30cc07dd758849f8685d322150459e4f1 by Elvis Angelaccio.
Committed on 05/01/2021 at 21:03.
Pushed by elvisangelaccio into branch 'release/20.12'.

Fix crash after closing the window while loading a TAR archive

A first problem was in LibarchivePlugin::list() not checking
isInterruptionRequested() after the while loop. It was wrongly calling
emitCorruptArchive() instead of aborting as requested.

But according to the stacktrace, the actual crash seemed to be caused by
the for() loop over qAsConst(m_jobs):

    #0  0x00007ffff5f6a023 in QHashData::nextNode(QHashData::Node*) () from
/usr/lib/libQt5Core.so.5
    #1  0x00007fffe1dcb2ec in QHash<KJob*,
QHashDummyValue>::const_iterator::operator++ (this=0x7fffffffc4d0) at
/usr/include/qt/QtCore/qhash.h:426
    #2  0x00007fffe1dcadac in QSet<KJob*>::const_iterator::operator++
(this=0x7fffffffc4d0) at /usr/include/qt/QtCore/qset.h:174
    #3  0x00007fffe1dca17e in JobTracker::~JobTracker (this=0x555555c319c0) at
../part/jobtracker.cpp:41

Porting to QSetIterator fixes the crash.
FIXED-IN: 20.12.2

M  +3    -1    part/jobtracker.cpp
M  +4    -0    plugins/libarchive/libarchiveplugin.cpp

https://invent.kde.org/utilities/ark/commit/a41e69b30cc07dd758849f8685d322150459e4f1

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to