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

wolthera <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
           Keywords|                            |triaged

--- Comment #1 from wolthera <[email protected]> ---
Hi, thanks for your report.

This is the relevant section of the crash log:
-------------------------------------------------------------------

00007FF888AF5F90 0000000000000000 00007FF888DCB880 00000000119EBC90 
libkritaimage.dll!0x1f5f90 KisSelection::pixelSelection+0x0
00007FF8869925FF 00000000119EBBD0 00000000119EBBD0 00000000223A7FA0 
libkritaui.dll!0x3225ff
KisSelectAllActionFactory::run(KisViewManager*)::SelectAll::paint+0x2f
00007FF888983A6C 000000004C93A8F0 00007FF888A51B45 000000005E03FE70 
libkritaimage.dll!0x83a6c KisTransactionBasedCommand::redo+0x2c
00007FF888A3EB96 0000000053A9A3F0 0000000019DE6D40 000000005E03FE70 
libkritaimage.dll!0x13eb96
KisStrokeStrategyUndoCommandBased::doStrokeCallback+0x76
00007FF888C49E70 0000000000000010 0000000067C93680 0000000059C271E0 
libkritaimage.dll!0x349e70 KisUpdateJobItem::run+0x90
00007FF886068A90 0000000053A9A3F0 0000000059C27100 0000000000000000 
Qt5Core.dll!0x28a90 QThreadPool::tryStart+0x520
00007FF886061AAF 0000000000000000 0000000000000000 0000000000000000 
Qt5Core.dll!0x21aaf QThread::qt_metacall+0x69f
00007FF8B4337974 0000000000000000 0000000000000000 0000000000000000 
KERNEL32.DLL!0x17974 BaseThreadInitThunk+0x14
00007FF8B6FFA2F1 0000000000000000 0000000000000000 0000000000000000 
ntdll.dll!0x5a2f1 RtlUserThreadStart+0x21

---------------------------

Basically, what seems to be going on is that KisSelection can somehow end up
without a pixelSelection pointer, and crashes when it's accessed in
KisSelectAllActionFactory::run at line 157. My instinct is to say we should
first check if that pointer exists, but I am also wondering why that pointer is
able to be missing to begin with. Maybe we should always ensure it exists by
initializing, for example. I'll need to ask around.

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

Reply via email to