Bug 411703 - [4.3 prealpha] Crash when using Liquify Transformation
Summary: [4.3 prealpha] Crash when using Liquify Transformation
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Tools/Transform (show other bugs)
Version: nightly build (please specify the git hash!)
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-09-08 10:41 UTC by acc4commissions
Modified: 2019-09-13 09:59 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Asserts. (57.57 KB, image/png)
2019-09-08 10:41 UTC, acc4commissions
Details

Note You need to log in before you can comment on or make changes to this bug.
Description acc4commissions 2019-09-08 10:41:18 UTC
Created attachment 122531 [details]
Asserts.

SUMMARY

git ca902a4 (krita-nightly-x64-v4.3.0-prealpha-336-gca902a4557)

Crash when using Liquify Transformation.


STEPS TO REPRODUCE
1. Press image with Liquify tool, edit it, then press the enter key to apply.
2. Repeat 1. If you do the recursive process fast enough, couple of assert messages will pop up. 
3. If you press 'abort', Krita crashes.



SOFTWARE/OS VERSIONS
Windows: Win7
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION

kritacrash.log : 

Error occurred on Sunday, September 8, 2019 at 18:43:28.

krita.exe caused a Stack Overflow at location 0000000077727D3F in module ntdll.dll.

AddrPC           Params
Comment 1 Dmitry Kazakov 2019-09-09 15:07:58 UTC
Okay, I can reproduce it here:
https://invent.kde.org/snippets/441
Comment 2 Dmitry Kazakov 2019-09-09 15:29:10 UTC
Git commit e6e9fea57bc9b57b5b07810e8bcf2b94ec65b954 by Dmitry Kazakov.
Committed on 09/09/2019 at 15:28.
Pushed by dkazakov into branch 'master'.

Fix crash when liquify transform is started too quickly

Basically, we shouldn't initiate any transform action until
the transform strategy has finished preparing tool initialization
(which is flagged by setting m_transform.rootNode()).

All user actions happened before the initialization should be
ignored (he doesn't have any feedback anyway).

M  +1    -1    plugins/tools/tool_transform2/kis_tool_transform.cc

https://invent.kde.org/kde/krita/commit/e6e9fea57bc9b57b5b07810e8bcf2b94ec65b954
Comment 3 Dmitry Kazakov 2019-09-13 09:59:56 UTC
Git commit ce8ed8014a8171a8017541d0e6dd32bc3ae9f88a by Dmitry Kazakov.
Committed on 13/09/2019 at 09:00.
Pushed by dkazakov into branch 'krita/4.2'.

Fix crash when liquify transform is started too quickly

Basically, we shouldn't initiate any transform action until
the transform strategy has finished preparing tool initialization
(which is flagged by setting m_transform.rootNode()).

All user actions happened before the initialization should be
ignored (he doesn't have any feedback anyway).

M  +1    -1    plugins/tools/tool_transform2/kis_tool_transform.cc

https://invent.kde.org/kde/krita/commit/ce8ed8014a8171a8017541d0e6dd32bc3ae9f88a