Bug 496391

Summary:	Kdenlive crashes when clicking "Remove all keyframes after cursor"
Product:	[Applications] kdenlive	Reporter:	Bibido <bibidiboom>
Component:	Video Effects & Transitions	Assignee:	emohr <fritzibaby>
Status:	RESOLVED FIXED
Severity:	crash	CC:	fritzibaby
Priority:	NOR	Keywords:	triaged
Version First Reported In:	24.08.3
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	https://invent.kde.org/multimedia/kdenlive/-/commit/709c41081c63eeb887733a52038e90de63251092	Version Fixed/Implemented In:	24.12.0
Sentry Crash Report:
Attachments:	crash_delete keyframe

Description Bibido 2024-11-17 19:09:25 UTC

SUMMARY
If you have a video or color clip in the timeline and apply any effect that allows you to set keyframes, clicking "Remove all keyframes after cursor" when there are no actual keyframes after the cursor causes Kdenlive to crash.

STEPS TO REPRODUCE
1. Have a color clip and apply a Transform effect
2. Without setting any keyframe, move the cursor inside the effect to the middle of the clip
3. Click "Remove all keyframes after cursor" in the effect's options

OBSERVED RESULT
Kdenlive crashes under Windows 10.

EXPECTED RESULT
The application should not crash where there are no keyframes to remove.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
(available in the Info Center app, or by running `kinfo` in a terminal window)
Linux/KDE Plasma: 
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION

Comment 1 emohr 2024-11-18 17:18:29 UTC

Created attachment 175925 [details]
crash_delete keyframe

Thank you for reporting. I can confirm the crash. Attached the crash report.

Comment 2 Jean-Baptiste Mardelle 2024-11-22 12:10:54 UTC

Git commit 709c41081c63eeb887733a52038e90de63251092 by Jean-Baptiste Mardelle.
Committed on 22/11/2024 at 12:09.
Pushed by mardelle into branch 'release/24.12'.

Fix crash trying to remove keyframes when there are none.
FIXED-IN: 24.12.0

M  +4    -0    src/assets/keyframes/model/keyframemodel.cpp

https://invent.kde.org/multimedia/kdenlive/-/commit/709c41081c63eeb887733a52038e90de63251092