From af74ff626ee077ea5142467ca6d1f5ed09ce1855 Mon Sep 17 00:00:00 2001 From: David Bailes Date: Fri, 28 Jun 2019 10:09:11 +0100 Subject: [PATCH] Track menu: opening track menu by pressing shift+M causes crash Fix: Call TrackPanel::OnTrackMenu(Track *t) with a track pointer. Query: Is there any reason why this function's parameter has a default value of NULL? --- src/menus/TrackMenus.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/menus/TrackMenus.cpp b/src/menus/TrackMenus.cpp index e1cd080c0..20109d8a8 100644 --- a/src/menus/TrackMenus.cpp +++ b/src/menus/TrackMenus.cpp @@ -1145,8 +1145,9 @@ void OnTrackMenu(const CommandContext &context) { auto &project = context.project; auto &trackPanel = TrackPanel::Get( project ); - - trackPanel.OnTrackMenu(); + auto track = trackPanel.GetFocusedTrack(); + if (track) + trackPanel.OnTrackMenu(track); } void OnTrackMute(const CommandContext &context)