Bug 434754 - Crash on "Clear current cache"
Summary: Crash on "Clear current cache"
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (show other bugs)
Version: 20.12.3
Platform: Appimage Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-03-22 10:38 UTC by DrSlony
Modified: 2022-03-21 08:09 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:
fritzibaby: timeline_corruption+


Attachments
Screencast of crash (1.53 MB, video/mp4)
2021-04-21 12:56 UTC, DrSlony
Details
gdb backtrace (from flatpak) (24.47 KB, text/plain)
2021-04-21 13:25 UTC, Julius Künzel
Details
same folder listed twice (63.32 KB, image/png)
2021-05-31 09:28 UTC, DrSlony
Details

Note You need to log in before you can comment on or make changes to this bug.
Description DrSlony 2021-03-22 10:38:23 UTC
kdenlive-20.12.3a-x86_64.appimage (can't find the exact build hash in the AppImage - issue #434752)

1. Run kdenlive > Settings > Manage Cached Data.
2. Select a folder with an auto-generated numeric name, e.g. 1616409092687.
3. Click "Delete current cache"
4. In the popup warning dialog, click Continue.
5. In the second warning popup dialog (too many warning popup dialogs!), click Continue. Crash.

qt.qpa.xcb: QXcbConnection: XCB error: 3 (BadWindow), sequence: 2467, resource id: 16114544, major code: 40 (TranslateCoords), minor code: 0
/tmp/.mount_kdenliT4I9py/AppRun: line 42: 32335 Segmentation fault      (core dumped) kdenlive --config kdenlive-appimagerc $@

When I ran "gdb ./kdenlive-20.12.3a-x86_64.appimage", there was no stack.
Comment 1 Julius Künzel 2021-04-20 13:02:45 UTC
Thanks for your report! I recently made some fixes to the cache management. Can you please try if this is still happening with the 21.04 release candidate: https://files.kde.org/kdenlive/unstable/kdenlive-21.04.0-rc-x86_64.appimage

Btw I don't know if you are aware of how to use gdb correctly (I wasn't for a long time so I am going to explain)… After you did

$ gdb ./kdenlive-20.12.3a-x86_64.appimage

you will see something like

Reading symbols from /home/user/kdenlive-20.12.3a-x86_64.appimage...
(No debugging symbols found in /home/user/kdenlive-20.12.3a-x86_64.appimage)
(gdb)

then you have to enter "r" and hit enter. After kdenlive crashed you have to type "bt full" and hit enter to see the stack
Comment 2 DrSlony 2021-04-21 11:25:09 UTC
Hey

Yes I am familiar with gdb, but the stack is empty, so no backtrace.

I tested kdenlive-21.04.0-rc-x86_64.appimage and received the same 2 confirmation popups and reproduced the same crash.

(gdb) bt full
No stack.
Comment 3 Julius Künzel 2021-04-21 12:23:58 UTC
It seems you have opened "Manage Cached Data" after you opened a project or changed anything on the "untitled" project (i.e. you see two tabs "Current Project" and "All Projects"), right?

…and you haven't selected any "auto-generated numeric name" but the one of the current project. That's also the reason why you get multiple popup dialogs…

Unfortunately I can still not reproduce, for me it works fine.
What is your OS?
Comment 4 DrSlony 2021-04-21 12:56:38 UTC
> It seems you have opened "Manage Cached Data" after you opened a project or changed anything on the "untitled" project

Negative, the steps to reproduce are from once I start kdenlive.

I'm using Sabayon.

I will attach a screencast to help.
Comment 5 DrSlony 2021-04-21 12:56:53 UTC
Created attachment 137758 [details]
Screencast of crash
Comment 6 Julius Künzel 2021-04-21 13:24:49 UTC
Thank you for the screen record. That was very useful, I was able to reproduce now! It seems to happen only if you are on a empty project and select the first auto-generated numeric name entry. It happens even with the flapak and was able to get a backtrace there.

However it does not happen with builds from master / nightly build. That's likely because I made further changes in the master branch that are not going to be in 21.04. I have to check if I can port them for 21.04.1 otherwise they will be in 21.08

If you can confirm that it is fixed with the nightly that would be nice: https://binary-factory.kde.org/job/Kdenlive_Nightly_Appimage_Build/
Comment 7 Julius Künzel 2021-04-21 13:25:25 UTC
Created attachment 137759 [details]
gdb backtrace (from flatpak)
Comment 8 Julius Künzel 2021-05-16 13:32:36 UTC
Please test with the nightly build as asked above.
Comment 9 Bug Janitor Service 2021-05-31 04:33:39 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 10 DrSlony 2021-05-31 09:27:54 UTC
Tested kdenlive-21.07.70-9b89f4e-x86_64.appimage with no project open.

Crash fixed.

There are still 2 popups with the same message.

If I select all folders and click "Clear current cache", OR if I delete all folders one by one and then delete the last one, the last one appears twice.
Comment 11 DrSlony 2021-05-31 09:28:33 UTC
Created attachment 138894 [details]
same folder listed twice
Comment 12 Julius Künzel 2021-09-27 13:14:57 UTC
Yes I can confirmed the duplicated entry with today's git master
Comment 13 Jean-Baptiste Mardelle 2022-03-21 08:09:06 UTC
Git commit 73e3994f7265337fdb596148aad8f7a975786727 by Jean-Baptiste Mardelle.
Committed on 21/03/2022 at 08:08.
Pushed by mardelle into branch 'release/22.04'.

Fix double warning and duplicate folder on manage cache data deletion.

M  +12   -7    src/project/dialogs/temporarydata.cpp
M  +1    -1    src/project/dialogs/temporarydata.h

https://invent.kde.org/multimedia/kdenlive/commit/73e3994f7265337fdb596148aad8f7a975786727