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

--- Comment #4 from wolthera <griffinval...@gmail.com> ---
(In reply to Tiar from comment #2)
> Created attachment 143220 [details]
> Potential patch for tasksets
> 
> @Wolthera, how did you make `sketch.png~` to be visible for you when you
> search for patterns? I can't see it when the ending is wrong. I think I have
> the Qt dialogs, I tried to enable native dialogs but even after restart it
> still shows the same dialog.
> 
> The pattern crash probably happens because:
> - it needs md5 sum. But it isn't calculated yet
> - to calculate md5 sum, it tries to save the pattern to memory (why? it's
> not embedded in anything, it's a file)
> - saving pattern to memory doesn't work, because it uses:
> QImage.save(buffer, "PNG~") function. So it doesn't save any bytes
> - md5 doesn't work on empty buffers (no bytes saved)
> Since we're going to talk about md5 situation with Dmitry, so I'll mention
> that.
> 
> For tasksets, the backtrace is different (note that it doesn't go into
> `md5Sum()` function but already crashes on load()). However I have the same
> problem, reproducing your issue :/
> However I read the code and I think the problem might be that it simply
> doesn't create a loader (I can't be sure because the backtrace is cut, I
> think - was it segmentation fault because of empty loader?  I hope so, at
> least). In that case, I provided a patch file, which might or might not
> work, because I couldn't test it. It would be awesome if you could please
> test it, if it's possible.

I am using the KDE native dialogs. This patch fixes the crash with the task
set, but not the crash with the backup file.

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

Reply via email to