Skip to content
Snippets Groups Projects
Commit ae642f47 authored by luyikei's avatar luyikei
Browse files

Fix possible memory leak in MainWorkflow::addEffect

parent 9323dde6
No related branches found
No related tags found
No related merge requests found
......@@ -339,11 +339,11 @@ MainWorkflow::linkClips( const QString& uuidA, const QString& uuidB )
QString
MainWorkflow::addEffect( const QString &clipUuid, const QString &effectId )
{
EffectHelper* newEffect = nullptr;
std::shared_ptr<EffectHelper> newEffect;
try
{
newEffect = new EffectHelper( effectId );
newEffect.reset( new EffectHelper( effectId ) );
}
catch( Backend::InvalidServiceException& e )
{
......@@ -353,9 +353,7 @@ MainWorkflow::addEffect( const QString &clipUuid, const QString &effectId )
for ( auto clip : m_clips )
if ( clip->uuid().toString() == clipUuid )
{
trigger( new Commands::Effect::Add(
std::shared_ptr<EffectHelper>( newEffect ), clip->input() )
);
trigger( new Commands::Effect::Add( newEffect, clip->input() ) );
return newEffect->uuid().toString();
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment