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

Dmitry Kazakov <dimul...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://commits.kde.org/kri |https://commits.kde.org/kri
                   |ta/3dffd58b401a04699d24521e |ta/a4bf4835af64a4ac01953125
                   |53492f3214f1caab            |7b62646479a90226

--- Comment #11 from Dmitry Kazakov <dimul...@gmail.com> ---
Git commit a4bf4835af64a4ac019531257b62646479a90226 by Dmitry Kazakov.
Committed on 28/11/2016 at 12:58.
Pushed by dkazakov into branch 'krita/3.1'.

Fix for a crash when saving a document after adding a text shape

Summary:
This patch should fix a crash that happens after
KisShapeLayerCanvas::repaint() is called on a cloned image. Actually,
KisShapeLayer::setImage() was implemented incorrectly. It deleted
m_d->converter and m_d->paintDevice, although canvas still owned a link
to them. Therefore the next coming update crashed the application.

Test Plan:
I could never reproduce the original bug, I only used the results of
Boud's valgrind output. So please test the original bug with the patch
before I push it into 3.1 and impex.

Reviewers: #krita, rempt

Differential Revision: https://phabricator.kde.org/D3452

M  +9    -15   libs/ui/flake/kis_shape_layer.cc
M  +10   -4    libs/ui/flake/kis_shape_layer_canvas.cpp
M  +6    -3    libs/ui/flake/kis_shape_layer_canvas.h
M  +5    -0    libs/ui/kis_image_view_converter.cpp
M  +3    -1    libs/ui/kis_image_view_converter.h

https://commits.kde.org/krita/a4bf4835af64a4ac019531257b62646479a90226

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

Reply via email to