Release the display mode when we are done with it.

[vlc] / modules / access / sdi.cpp

diff --git a/modules/access/sdi.cpp b/modules/access/sdi.cpp
index c2a2bdaf725fc3eb8608a456823397b7c72a3d00..bce0a04624cf173019e910553b7987fcefce2ff2 100644 (file)
--- a/modules/access/sdi.cpp
+++ b/modules/access/sdi.cpp
@@ -306,13 +306,11 @@ static int Open( vlc_object_t *p_this )
     if( result != S_OK )
     {
         msg_Err( p_demux, "Could not get model name" );
-        free( psz_model_name );
         Close( p_this );
         return VLC_EGENERIC;
     }
 
     msg_Dbg( p_demux, "Opened DeckLink PCI card %d (%s)", i_card_index, psz_model_name );
-    free( psz_model_name );
 
     if( p_sys->p_card->QueryInterface( IID_IDeckLinkInput, (void**)&p_sys->p_input) != S_OK )
     {
@@ -439,9 +437,7 @@ static int Open( vlc_object_t *p_this )
         IDeckLinkDisplayMode *p_display_mode;
         result = p_display_iterator->Next( &p_display_mode );
         if( result != S_OK || !p_display_mode )
-        {
             break; 
-        }
 
         char sz_mode_id_text[5] = {0};
         BMDDisplayMode mode_id = ntohl( p_display_mode->GetDisplayMode() );
@@ -452,7 +448,7 @@ static int Open( vlc_object_t *p_this )
         if( result != S_OK )
         {
             msg_Err( p_demux, "Failed to get display mode name" );
-            free( psz_mode_name );
+            p_display_mode->Release();
             p_display_iterator->Release();
             Close( p_this );
             return VLC_EGENERIC;
@@ -463,7 +459,7 @@ static int Open( vlc_object_t *p_this )
         if( result != S_OK )
         {
             msg_Err( p_demux, "Failed to get frame rate" );
-            free( psz_mode_name );
+            p_display_mode->Release();
             p_display_iterator->Release();
             Close( p_this );
             return VLC_EGENERIC;
@@ -497,7 +493,6 @@ static int Open( vlc_object_t *p_this )
                  sz_mode_id_text, psz_mode_name,
                  p_display_mode->GetWidth(), p_display_mode->GetHeight(),
                  double(time_scale) / frame_duration, psz_field_dominance );
-        free( psz_mode_name );
 
         if( wanted_mode_id == mode_id )
         {
@@ -508,6 +503,8 @@ static int Open( vlc_object_t *p_this )
             p_sys->i_fps_den = frame_duration;
             p_sys->i_dominance_flags = i_dominance_flags;
         }
+
+        p_display_mode->Release();
     }
 
     p_display_iterator->Release();