[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-23 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

Dmitry Kazakov  changed:

   What|Removed |Added

 CC||dimul...@gmail.com
   Assignee|krita-bugs-n...@kde.org |dimul...@gmail.com

--- Comment #1 from Dmitry Kazakov  ---
Hi, Franck!

Is it possible to share the entire original file with me? If it is private, you
can sent it me to an email: dimula73 at gmail dot com.

This bug seem to be ralated to: https://bugs.kde.org/show_bug.cgi?id=420322

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-23 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #2 from Dmitry Kazakov  ---
Btw, the layer you removed doesn't seem to be a "Fill Layer". Perhaps it was a
Paint Layer with Colorize Mask attached?

At least the contents of Layer26 you attached looks as if it was filled with
colorize mask and then rasterized...

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-23 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #3 from Dmitry Kazakov  ---
Okay, the file is not needed, I can reproduce the crash in a unittest

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-23 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

Dmitry Kazakov  changed:

   What|Removed |Added

 CC||triggerpigking@protonmail.c
   ||om

--- Comment #4 from Dmitry Kazakov  ---
*** Bug 420322 has been marked as a duplicate of this bug. ***

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-23 Thread Franck Rabeson
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #5 from Franck Rabeson  ---
Hello. Sorry for the late response. For the sake of clarification, the layer’s
entry in maindoc.xml was:



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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #6 from Dmitry Kazakov  ---
Hi, Franck!

This entry is for layer stored in folder "layer18" :) You should check
attribute 'filename="layer18"'. 

Do you still have this file and can you reproduce the crash every time you try
to open this file?

Btw, how many cores your CPU has?

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread wolthera
https://bugs.kde.org/show_bug.cgi?id=419301

wolthera  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 CC||griffinval...@gmail.com
 Status|REPORTED|CONFIRMED

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #7 from Dmitry Kazakov  ---
This bug seem to be related, the same assert is caught when opening the
attached file:
https://bugs.kde.org/show_bug.cgi?id=412488

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #8 from Dmitry Kazakov  ---
No, that file doesn't help reproducing the issue...

I have found another threading issue in the extents manager, but it is rather
minor and cannot cause any crashes... I still cannot reproduce the original
crash.

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #9 from Dmitry Kazakov  ---
Hi, Franck!

Did you have any transformation masks in your document that caused a crash?

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #10 from Dmitry Kazakov  ---
Hi, Franck!

Can you please test this package? Does it fix the crash for you?

https://yadi.sk/d/K0CN26KoDcGa4g

It fixes one transoformation masks threading issue, which might be the reason
for your bug.

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Franck Rabeson
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #11 from Franck Rabeson  ---
I’m sorry, I just checked but unfortunately it seems I do not have the version
of the file that caused the bug anymore. I have the version from which I
deleted the relevant layer, and that one loads without an issue. I still get
crashes when loading other files, but most of the time they tend to open fine
on the second or even the third try.

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Franck Rabeson
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #12 from Franck Rabeson  ---
I don’t use any transform masks at all, but I’ll test that package and will
report.

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #13 from Dmitry Kazakov  ---
> I still get crashes when loading other files

Can you reproduce these crashes with the package I linked? 

If yes, could you please get a DebugView log using this link:
https://docs.microsoft.com/en-us/sysinternals/downloads/debugview

You should start DebugView before starting Krita with the faulty file :)

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Franck Rabeson
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #14 from Franck Rabeson  ---
Sorry it took so long, downloading that package took ages with my internet.

Anyway, I tested it with as many as my files as possible. Loading was
noticeably much slower, but I experienced no crash on loading any of the files.

There were a couple of crashes when closing Krita, but I don’t think those were
related at all.

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #15 from Dmitry Kazakov  ---
Hi, Franck!

If it is possible, could you also test your crashing files with this build (you
need a .zip version):

https://binary-factory.kde.org/job/Krita_Stable_Windows_Build/

I just want to know what exactly fixed the issue... This build is from the
upcoming 4.3 release and it doesn't have my transformation mask fix.

Don't feel in hurry about it, I can wait a bit :)

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-24 Thread Franck Rabeson
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #16 from Franck Rabeson  ---
I tested several files and have found two that causes issues while opening.

The first one:
- 4.2.9 stable: loads fine;
- 4.3.0 prealpha 1523: crash;
- 4.3.0 prealpha with transform mask fix: loads fine.

The second one:
- 4.2.9 stable: crash;
- 4.3.0 prealpha 1523: crash;
- 4.3.0 prealpha with transform mask fix: loads fine.

Note that neither file actually uses a transform mask, since that is a feature
I’ve never used before.

Both are private NSFW art though, so if you’re okay with that, I can email you
one or both tomorrow.

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-27 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

--- Comment #17 from Dmitry Kazakov  ---
Hi, Franck!

Could you please send me the files you used for testing to dimula73 at gmail
dot com? I will keep them in private.

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-27 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

Dmitry Kazakov  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://invent.kde.org/kde/
   ||krita/commit/7b147c93b51bb4
   ||c7e2d0ed2148fd828c54226022

--- Comment #18 from Dmitry Kazakov  ---
Git commit 7b147c93b51bb4c7e2d0ed2148fd828c54226022 by Dmitry Kazakov.
Committed on 27/04/2020 at 21:40.
Pushed by dkazakov into branch 'master'.

Fix threading issue when loading KisGeneratorLayer

We shouldn't call the same function from both, GUI thread and a worker
thread. It may lead to concurrent writes to the same image area, which
is not something we are allowed to do
Related: bug 420322

M  +1-1libs/image/generator/kis_generator_layer.cpp

https://invent.kde.org/kde/krita/commit/7b147c93b51bb4c7e2d0ed2148fd828c54226022

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

[krita] [Bug 419301] Krita crashes when opening a file containing a fill layer

2020-04-27 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=419301

Dmitry Kazakov  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/kde/ |https://invent.kde.org/kde/
   |krita/commit/7b147c93b51bb4 |krita/commit/6ba81a096f29cb
   |c7e2d0ed2148fd828c54226022  |1eaf5a100615e15f671a4326d8

--- Comment #19 from Dmitry Kazakov  ---
Git commit 6ba81a096f29cb1eaf5a100615e15f671a4326d8 by Dmitry Kazakov.
Committed on 27/04/2020 at 21:44.
Pushed by dkazakov into branch 'krita/4.3'.

Fix threading issue when loading KisGeneratorLayer

We shouldn't call the same function from both, GUI thread and a worker
thread. It may lead to concurrent writes to the same image area, which
is not something we are allowed to do
Related: bug 420322

M  +1-1libs/image/generator/kis_generator_layer.cpp

https://invent.kde.org/kde/krita/commit/6ba81a096f29cb1eaf5a100615e15f671a4326d8

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