| Summary: | Crash after a right click on a group layer | ||
|---|---|---|---|
| Product: | [Applications] krita | Reporter: | David REVOY <info> |
| Component: | Layer Stack | Assignee: | Dmitry Kazakov <dimula73> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | dimula73, halla |
| Priority: | NOR | ||
| Version: | 5.1.0-beta1 | ||
| Target Milestone: | --- | ||
| Platform: | Appimage | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/graphics/krita/commit/c94c87e9c11c59ae15fc0e69198c0fcf49aff743 | Version Fixed In: | |
| Attachments: | ^ gdb output of the crash, compressed as zip because it was too big | ||
|
Description
David REVOY
2022-07-01 09:50:00 UTC
I cannot open the file because of the missing file layer:
krita(100321)/(default) unknown: ASSERT: "channelFlags.isEmpty() ||((quint32)channelFlags.count() == colorSpace()->channelCount())" in file /home/halla/dev/krita/libs/image/kis_layer.cc, line 342
Thread 1 "krita" received signal SIGABRT, Aborted.
__GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
50 ../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
(gdb) bt
#0 __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
#1 0x00007ffff52ef859 in __GI_abort () at abort.c:79
#2 0x00007ffff579c455 in QMessageLogger::fatal(char const*, ...) const ()
from /home/halla/dev/deps/lib/libQt5Core.so.5
#3 0x00007ffff579b827 in qt_assert(char const*, char const*, int) ()
from /home/halla/dev/deps/lib/libQt5Core.so.5
#4 0x00007ffff6c9d2f8 in KisLayer::setChannelFlags (this=<optimized out>, channelFlags=...)
at /home/halla/dev/krita/libs/image/kis_layer.cc:342
#5 0x00007fffc85efdd1 in KisKraLoader::loadNode (this=0x555557e72080, element=..., image=...)
at /home/halla/dev/krita/plugins/impex/libkra/kis_kra_loader.cpp:921
#6 0x00007fffc85f14e3 in KisKraLoader::loadNodes (this=0x555557e72080, element=..., image=..., parent=...)
at /home/halla/dev/krita/plugins/impex/libkra/kis_kra_loader.cpp:756
#7 0x00007fffc85f2663 in KisKraLoader::loadXML (this=0x555557e72080, imageElement=...)
at /home/halla/dev/krita/plugins/impex/libkra/kis_kra_loader.cpp:297
#8 0x00007fffc862b83a in KraConverter::loadXML (this=0x7fffffffc050, doc=..., store=<optimized out>)
at /home/halla/dev/krita/plugins/impex/libkra/kra_converter.cpp:374
#9 0x00007fffc862db31 in KraConverter::buildImage (this=this@entry=0x7fffffffc050, io=0x7fffffffbf50,
io@entry=0x7fffffffc110) at /home/halla/dev/krita/plugins/impex/libkra/kra_converter.cpp:87
#10 0x00007fffc864e3ce in KraImport::convert (this=<optimized out>, document=0x555559406b20, io=0x7fffffffc110)
at /home/halla/dev/krita/plugins/impex/kra/kra_import.cpp:30
#11 0x00007ffff7c4ef9c in KisImportExportManager::doImport (this=0x555559bc2940, location=..., filter=...)
at /home/halla/dev/krita/libs/ui/KisImportExportManager.cpp:665
#12 0x00007ffff7c507ba in KisImportExportManager::convert (this=0x555559bc2940, direction=<optimized out>,
location=..., realLocation=..., mimeType=..., showWarnings=<optimized out>, exportConfiguration=...,
isAsync=<optimized out>, isAdvancedExporting=<optimized out>)
at /home/halla/dev/krita/libs/ui/KisImportExportManager.cpp:376
#13 0x00007ffff7c5158b in KisImportExportManager::importDocument (this=this@entry=0x555559bc2940, location=...,
mimeType=...) at /home/halla/dev/krita/libs/global/kis_shared_ptr.h:206
#14 0x00007ffff7c36065 in KisDocument::openFile (this=0x555559406b20)
at /home/halla/dev/krita/libs/ui/KisDocument.cpp:1860
#15 0x00007ffff7c36ad9 in KisDocument::openPathInternal (this=0x555559406b20, path=...)
at /home/halla/dev/krita/libs/ui/KisDocument.cpp:2416
#16 0x00007ffff7c3c2d9 in KisDocument::openPath (this=this@entry=0x555559406b20, _path=..., flags=...,
flags@entry=...) at /home/halla/dev/krita/libs/ui/KisDocument.cpp:1796
#17 0x00007ffff7c60045 in KisMainWindow::openDocumentInternal (this=<optimized out>, path=..., flags=...)
at /home/halla/dev/krita/libs/ui/KisMainWindow.cpp:1024
#18 0x00007ffff7c6063d in KisMainWindow::openDocument (this=0x555557945270, path=..., flags=...)
at /home/halla/dev/krita/libs/ui/KisMainWindow.cpp:998
#19 0x00007ffff7c60a75 in KisMainWindow::slotFileOpen (this=0x555557945270, isImporting=false)
at /home/halla/dev/krita/libs/ui/KisMainWindow.cpp:1637
#20 0x00007ffff59cb0f0 in QMetaObject::activate(QObject*, int, int, void**) ()
from /home/halla/dev/deps/lib/libQt5Core.so.5
#21 0x00007ffff63cfa36 in QAction::triggered(bool) () from /home/halla/dev/deps/lib/libQt5Widgets.so.5
#22 0x00007ffff63d20f2 in QAction::activate(QAction::ActionEvent) ()
from /home/halla/dev/deps/lib/libQt5Widgets.so.5
#23 0x00007ffff63d29bf in QAction::event(QEvent*) () from /home/halla/dev/deps/lib/libQt5Widgets.so.5
#24 0x00007ffff63d60d6 in QApplicationPrivate::notify_helper(QObject*, QEvent*) ()
from /home/halla/dev/deps/lib/libQt5Widgets.so.5
#25 0x00007ffff63df5d0 in QApplication::notify(QObject*, QEvent*) ()
from /home/halla/dev/deps/lib/libQt5Widgets.so.5
#26 0x00007ffff7c1c7c5 in KisApplication::notify (this=0x7fffffffd570, receiver=0x555559d4d320,
event=0x7fffffffce40) at /home/halla/dev/krita/libs/ui/KisApplication.cpp:761
#27 0x00007ffff599f6ea in QCoreApplication::notifyInternal2(QObject*, QEvent*) ()
from /home/halla/dev/deps/lib/libQt5Core.so.5
#28 0x00007ffff5dbb93e in QShortcutMap::dispatchEvent(QKeyEvent*) ()
--Type <RET> for more, q to quit, c to continue without paging--
from /home/halla/dev/deps/lib/libQt5Gui.so.5
#29 0x00007ffff5dbba23 in QShortcutMap::tryShortcut(QKeyEvent*) () from /home/halla/dev/deps/lib/libQt5Gui.so.5
#30 0x00007ffff5d6fcf3 in QWindowSystemInterface::handleShortcutEvent(QWindow*, unsigned long, int, QFlags<Qt::KeyboardModifier>, unsigned int, unsigned int, unsigned int, QString const&, bool, unsigned short) ()
from /home/halla/dev/deps/lib/libQt5Gui.so.5
#31 0x00007ffff5d8d231 in QGuiApplicationPrivate::processKeyEvent(QWindowSystemInterfacePrivate::KeyEvent*) ()
from /home/halla/dev/deps/lib/libQt5Gui.so.5
#32 0x00007ffff5d928d1 in QGuiApplicationPrivate::processWindowSystemEvent(QWindowSystemInterfacePrivate::WindowSystemEvent*) () from /home/halla/dev/deps/lib/libQt5Gui.so.5
#33 0x00007ffff5d6c97c in QWindowSystemInterface::sendWindowSystemEvents(QFlags<QEventLoop::ProcessEventsFlag>)
() from /home/halla/dev/deps/lib/libQt5Gui.so.5
#34 0x00007fffefeaa54e in ?? () from /home/halla/dev/deps/lib/libQt5XcbQpa.so.5
#35 0x00007ffff2ce217d in g_main_context_dispatch () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#36 0x00007ffff2ce2400 in ?? () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#37 0x00007ffff2ce24a3 in g_main_context_iteration () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#38 0x00007ffff59f7635 in QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) ()
from /home/halla/dev/deps/lib/libQt5Core.so.5
#39 0x00007ffff599e29b in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) ()
from /home/halla/dev/deps/lib/libQt5Core.so.5
#40 0x00007ffff59a5ff6 in QCoreApplication::exec() () from /home/halla/dev/deps/lib/libQt5Core.so.5
#41 0x000055555555dd5b in main (argc=<optimized ou
After unzipping and removing the file layer, I can load the file, but right-clicking on the frame group layer doesn't lead to a crash anymore. Thank you. > [...] removing the file layer, I can load the file [...] Here I can still open the file with 5.1(beta1)appimage even after moving it to another directory (I'm prompted for the missing 'txt' layer too but I can skip it). So, if you couldn't open this one, it might be another issue about reading missing file layer. > but right-clicking on the frame group layer doesn't lead to a crash anymore. Good, maybe this right-click menu crash issue is fixed then in Git~master. I'll try to test it with a recent build too. I can reproduce the crash, but only on Linux, because it requires KRITA_DEVS build to active Q_ASSERT checks. Git commit ccfda933bbceba8dd1b9bf992c7433561fb382ba by Dmitry Kazakov. Committed on 11/07/2022 at 08:57. Pushed by dkazakov into branch 'master'. Fix an assert when a graya file layer fails to load its image The fallback color space for a file layer used to be the image's color space, which caused an qssert when trying to load graya channel flags. M +2 -2 libs/ui/kis_file_layer.cpp M +1 -1 libs/ui/kis_file_layer.h M +3 -3 plugins/impex/libkra/kis_kra_loader.cpp M +1 -1 plugins/impex/libkra/kis_kra_loader.h https://invent.kde.org/graphics/krita/commit/ccfda933bbceba8dd1b9bf992c7433561fb382ba Git commit fd924d7c03ce5409e5e0df9ae15b8991dd3cae8c by Dmitry Kazakov. Committed on 11/07/2022 at 08:59. Pushed by dkazakov into branch 'krita/5.1'. Fix an assert when a graya file layer fails to load its image The fallback color space for a file layer used to be the image's color space, which caused an qssert when trying to load graya channel flags. M +2 -2 libs/ui/kis_file_layer.cpp M +1 -1 libs/ui/kis_file_layer.h M +3 -3 plugins/impex/libkra/kis_kra_loader.cpp M +1 -1 plugins/impex/libkra/kis_kra_loader.h https://invent.kde.org/graphics/krita/commit/fd924d7c03ce5409e5e0df9ae15b8991dd3cae8c Hi, David! Please check if you can still reproduce the click-crash in the current master. I have fixed the assert that Halla experienced, but I couldn't reproduce the original click-crash. Hi Dmitry, I updated the last appimage nightly stable an hour ago, and I can still reproduce. But it is worst now: right clicking on "Frame" layer freeze my whole Plasma desktop. The only escape I have is to Ctrl+Alt+F3 and login and type 'sudo reboot now'. I tried twice and get twice the same behavior. I'll try again now with vanilla default settings. Same system freeze with vanilla default. I tried to put the terminal with gdb on the second screen, but nothing unusual appears in it before the freeze. Created attachment 150554 [details]
^ gdb output of the crash, compressed as zip because it was too big
@Dmitry: Thanks to your instruction on IRC, I could catch a hu~~~uge 6.4MB output of gdb. Very verbose.
I attached it to this bug report, but I decided to compress it on a zip (it shrank to 170KB, better disk space usage for bugs.kde.admin). I hope you'll find good quality information in it.
Git commit d20876ddb9dc872e0d82e0096c409e96e8d0d705 by Dmitry Kazakov. Committed on 12/07/2022 at 13:35. Pushed by dkazakov into branch 'krita/5.1'. Fix an infinite recursion in the color label selector The size of the spacer should be changed not in resizeEvent() (which causes an infinite recursion), but in showEvent(), right before the widget is displayed to the user M +3 -16 libs/ui/widgets/kis_color_label_selector_widget.cpp M +1 -2 libs/ui/widgets/kis_color_label_selector_widget.h https://invent.kde.org/graphics/krita/commit/d20876ddb9dc872e0d82e0096c409e96e8d0d705 Git commit c94c87e9c11c59ae15fc0e69198c0fcf49aff743 by Dmitry Kazakov. Committed on 12/07/2022 at 13:38. Pushed by dkazakov into branch 'master'. Fix an infinite recursion in the color label selector The size of the spacer should be changed not in resizeEvent() (which causes an infinite recursion), but in showEvent(), right before the widget is displayed to the user M +3 -16 libs/ui/widgets/kis_color_label_selector_widget.cpp M +1 -2 libs/ui/widgets/kis_color_label_selector_widget.h https://invent.kde.org/graphics/krita/commit/c94c87e9c11c59ae15fc0e69198c0fcf49aff743 Thank you @Dmitry, I tested the appimage from the nightly stable and I confirm: **the bug is fixed** 🎉 (I'll keep the gdb receipe somewhere to report future crashes with details). |