Bug 487872

Summary:	Crash when trying to delete subtitle file from interface
Product:	[Applications] kdenlive	Reporter:	Ivan <3847b8fe-a74d-4b44-a19e-3b68d5e31ed7>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	crash	CC:	berndmj, fritzibaby
Priority:	NOR
Version First Reported In:	24.05.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://invent.kde.org/multimedia/kdenlive/-/commit/1b48bb0919472bd29aee7acea7b37e546ff645ff	Version Fixed In:	24.05.1
Sentry Crash Report:
Attachments:	Video showcasing a simple example project breaking when deleting the first subtitle file Subtitle_crash-log

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.