Bug 487872 - Crash when trying to delete subtitle file from interface
Summary: Crash when trying to delete subtitle file from interface
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (show other bugs)
Version: 24.05.0
Platform: Arch Linux Linux
: NOR crash
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-06-01 08:47 UTC by Ivan
Modified: 2024-06-05 06:56 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 24.05.1
Sentry Crash Report:


Attachments
Video showcasing a simple example project breaking when deleting the first subtitle file (340.88 KB, video/webm)
2024-06-04 17:39 UTC, Ivan
Details
Subtitle_crash-log (11.18 KB, text/plain)
2024-06-04 18:20 UTC, emohr
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ivan 2024-06-01 08:47:40 UTC
Cannot add a backtrace right now (will try later), but it should be straightforward to reproduce.

SUMMARY

Create a project, create subtitles through the interface. Open the subtitle track manager, and add a new one

STEPS TO REPRODUCE
1.  Create a project
2. Create subtitles through the standard interface
3. Open the subtitle track manager
4. Create another subtitle file (Subtitles 2) from the interface
5. Click on the first one, and click on "Delete"

OBSERVED RESULT

The app crashes.


EXPECTED RESULT

The first subtitle file is deleted and nothing else special happens.

SOFTWARE/OS VERSIONS
KDE Plasma Version: 6.0.5
KDE Frameworks Version: 6.2.0
Qt Version: 6.7.1
Kernel Version: 6.9.3-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 8 × AMD Ryzen 5 2500U with Radeon Vega Mobile Gfx
Memory: 7.3 GiB of RAM
Graphics Processor: AMD Radeon Vega 8 Graphics

ADITIONAL INFORMATION

Kdenlive is from the main Arch Linux repos.
Comment 1 emohr 2024-06-01 11:22:01 UTC
On Windows with Kdenlive e24.05.0. I cannot reproduce the crash following your steps. 

Please try with the current Kdenlive AppImage version 24.05.0 to see if there are any packaging issues: https://kdenlive.org/en/download/
Comment 2 Ivan 2024-06-04 13:30:50 UTC
Just tried on both Flatpak and AppImage, same crash.
Comment 3 Bernd 2024-06-04 16:53:47 UTC
(In reply to Giovanni from comment #2)
> Just tried on both Flatpak and AppImage, same crash.

I tried to reproduce with appimage 24.05.0 on Pop!_OS 22.04 and everything works just fine.

Could you share a screen recording of the steps you are taking?
Comment 4 Ivan 2024-06-04 17:39:17 UTC
Created attachment 170143 [details]
Video showcasing a simple example project breaking when deleting the first subtitle file
Comment 5 emohr 2024-06-04 18:20:36 UTC
Created attachment 170145 [details]
Subtitle_crash-log

Thank you for reporting. I can reproduce the crash, following your steps on the video.
Attached the backtrace.
Comment 6 Bernd 2024-06-04 19:45:22 UTC
(In reply to Giovanni from comment #4)
> Created attachment 170143 [details]
> Video showcasing a simple example project breaking when deleting the first
> subtitle file

Yep, crashes the appimage 24.05.0 on Pop!_OS 22.04.

Interestingly enough, if I delete the Subtitle folder instead of just the .srt file in said folder, there is no crash and I can continue working ...
Comment 7 Jean-Baptiste Mardelle 2024-06-05 06:33:28 UTC
Git commit 1b48bb0919472bd29aee7acea7b37e546ff645ff by Jean-Baptiste Mardelle.
Committed on 05/06/2024 at 06:32.
Pushed by mardelle into branch 'release/24.05'.

Fix crash deleting subtitle when the file url was selected.
FIXED-IN: 24.05.1

M  +7    -1    src/dialogs/managesubtitles.cpp

https://invent.kde.org/multimedia/kdenlive/-/commit/1b48bb0919472bd29aee7acea7b37e546ff645ff
Comment 8 Ivan 2024-06-05 06:56:59 UTC
That was incredibly fast. Thank you very much.