Skip to content

Commit 9b7e08f

Browse files
authored
Fix jamulussoftware#1083: free CJamRecorder if Init() fails (jamulussoftware#3838)
1 parent 9d1eca4 commit 9b7e08f

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

src/recorder/jamcontroller.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,12 @@ void CJamController::SetRecordingDir ( QString newRecordingDir, int iServerFrame
116116
bRecorderInitialised = ( strRecorderErrMsg == QString() );
117117
bEnableRecording = bRecorderInitialised && !bDisableRecording;
118118

119+
if ( !bRecorderInitialised )
120+
{
121+
delete pJamRecorder; // Init() failed: never moved to thread / never
122+
pJamRecorder = nullptr; // wired to deleteLater, so free it here (#1083)
123+
}
124+
119125
qInfo() << qUtf8Printable ( QString ( "Recording state: %1" ).arg ( bEnableRecording ? "enabled" : "disabled" ) );
120126
}
121127
else

0 commit comments

Comments
 (0)