Bug 455244

Summary: crash clicking brush settings button when brush selection button is open if brush has been set to new smudge algorithm
Product: [Applications] krita Reporter: fivetailsfox
Component: * UnknownAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED FIXED    
Severity: crash CC: dimula73
Priority: NOR    
Version: 5.0.6   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: https://invent.kde.org/graphics/krita/commit/d1625a218353c3186e3f82e4b665a58d9c61fa77 Version Fixed In:

Description fivetailsfox 2022-06-14 11:34:26 UTC 
SUMMARY:
If I choose a smudge brush (default or custom) and enable the new smudge algorithm, opening brush settings by clicking the button on the UI while the pop-up for the brush selection button is still open will cause a crash.


STEPS TO REPRODUCE:
1. open krita and load a document, a new one will do
2. choose a smudge brush (e.g. "wet bristles rough"), and enable 'new smudge algorithm' under 'smudge length'
3. click the 'brush selection' button on the UI to open the pop-up with the list of brushes
4. while it is open, click the 'brush settings' button on the UI.

OBSERVED RESULT:
Krita freezes and crashes.

EXPECTED RESULT
The brush settings window opens and the brush selection pop-up closes

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

ADDITIONAL INFORMATION
Comment 1 Dmitry Kazakov 2022-07-13 07:32:34 UTC 
I can reproduce the crash here in master :)
Comment 2 Dmitry Kazakov 2022-07-13 08:11:56 UTC 
Git commit 5558ebc0274adee3543d80d433d4c1d298abff12 by Dmitry Kazakov.
Committed on 13/07/2022 at 08:11.
Pushed by dkazakov into branch 'master'.

Fix cycling updates in the new colorsmudge enginge config widget

When changing radius range we may accidentially change the actual
radius value. Since it happens in the 'page-changed' call, we need
to make sure this change doesn't emit any signal.

M  +2    -0    plugins/paintops/colorsmudge/kis_colorsmudgeop_settings_widget.cpp

https://invent.kde.org/graphics/krita/commit/5558ebc0274adee3543d80d433d4c1d298abff12
Comment 3 Dmitry Kazakov 2022-07-13 08:12:04 UTC 
Git commit d1625a218353c3186e3f82e4b665a58d9c61fa77 by Dmitry Kazakov.
Committed on 13/07/2022 at 08:11.
Pushed by dkazakov into branch 'krita/5.1'.

Fix cycling updates in the new colorsmudge enginge config widget

When changing radius range we may accidentially change the actual
radius value. Since it happens in the 'page-changed' call, we need
to make sure this change doesn't emit any signal.

M  +2    -0    plugins/paintops/colorsmudge/kis_colorsmudgeop_settings_widget.cpp

https://invent.kde.org/graphics/krita/commit/d1625a218353c3186e3f82e4b665a58d9c61fa77