Changeset 36d1d663db8961cbb85e578f43f90ecfde11016e

Timestamp:

19/07/08 13:19:16 (5 months ago)

Author:

Laurent Aimar <fenrir@videolan.org>

git-committer:

Laurent Aimar <fenrir@videolan.org> 1216466356 +0200

git-parent:

[ab131d8bfb68e02a952567e93c5a891ce8d6322c]

git-author:

Laurent Aimar <fenrir@videolan.org> 1216466356 +0200

Message:

Fixed a segfault in aout_EnableFilter.
Fixed missing lock in aout callbacks !
Factorized duplicated code.

Files:

• src/audio_output/aout_internal.h (modified) (1 diff)
• src/audio_output/input.c (modified) (5 diffs)
• src/audio_output/intf.c (modified) (1 diff)

src/audio_output/aout_internal.h

@@ -139 +139 @@
-int aout_DecPlay( aout_instance_t *, aout_input_t *, aout_buffer_t *, int i_input_rate );
-
+/* Helpers */
+
+/**
+ * This function will safely mark aout input to be restarted as soon as
+ * possible to take configuration changes into account */
+static inline void AoutInputsMarkToRestart( aout_instance_t *p_aout )
+{
+    int i;
+    vlc_mutex_lock( &p_aout->mixer_lock );
+    for( i = 0; i < p_aout->i_nb_inputs; i++ )
+        p_aout->pp_inputs[i]->b_restart = true;
+    vlc_mutex_unlock( &p_aout->mixer_lock );
+}
+
+/* This function will add or remove a a module from a string list (comma
+ * separated). It will return true if there is a modification
+ * In case p_aout is NULL, we will use configuration instead of variable */
+static inline bool AoutChangeFilterString( vlc_object_t *p_obj, aout_instance_t * p_aout,
+                                           const char* psz_variable,
+                                           const char *psz_name, bool b_add )
+{
+    vlc_value_t val;
+    char *psz_parser;
+
+    if( *psz_name == '\0' )
+        return false;
+
+    if( p_aout )
+        var_Get( p_aout, psz_variable, &val );
+    else
+        val.psz_string = config_GetPsz( p_obj, "audio-filter" );
+
+    if( !val.psz_string )
+        val.psz_string = strdup("");
+
+    psz_parser = strstr( val.psz_string, psz_name );
+
+    if( ( b_add && psz_parser ) || ( !b_add && !psz_parser ) )
+    {
+        /* Nothing to do */
+        free( val.psz_string );
+        return false;
+    }
+
+    if( b_add )
+    {
+        char *psz_old = val.psz_string;
+        if( *psz_old )
+            asprintf( &val.psz_string, "%s:%s", psz_old, psz_name );
+        else
+            val.psz_string = strdup( psz_name );
+        free( psz_old );
+    }
+    else
+    {
+        const int i_name = strlen( psz_name );
+        const char *psz_next;
+
+        psz_next = &psz_parser[i_name];
+        if( *psz_next == ':' )
+            psz_next++;
+
+        memmove( psz_parser, psz_next, strlen(psz_next)+1 );
+    }
+
+    if( p_aout )
+        var_Set( p_aout, psz_variable, val );
+    else
+        config_PutPsz( p_obj, psz_variable, val.psz_string );
+    free( val.psz_string );
+    return true;
+}
+
-#endif /* !__LIBVLC_AOUT_INTERNAL_H */

src/audio_output/input.c

@@ -767 +767 @@
-                                 const char *psz_name, bool b_add )
-{
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
-}
-
@@ -816 +777 @@
-    char *psz_mode = newval.psz_string;
-    vlc_value_t val;
-    int i;
-    (void)psz_cmd; (void)oldval; (void)p_data;
-
@@ -852 +812 @@
-
-    /* That sucks */
-
-
-
-
+
-
-    return VLC_SUCCESS;
@@ -866 +823 @@
-    char *psz_mode = newval.psz_string;
-    vlc_value_t val;
-    int i;
-    int i_ret;
-    (void)psz_cmd; (void)oldval; (void)p_data;
@@ -887 +843 @@
-    /* That sucks */
-    if( i_ret == 1 )
-
-
-
-
-
-
-
+
-    return VLC_SUCCESS;
-}

src/audio_output/intf.c

@@ -490 +490 @@
-                        bool b_add )
-{
-
-
-
+
+
+
+
+
+
+
+
-
-    if( p_aout )
-        psz_string = var_GetNonEmptyString( p_aout, "audio-filter" );
-    else
-        psz_string = config_GetPsz( p_this, "audio-filter" );
-
-    if( !psz_string ) psz_string = strdup("");
-
-    psz_parser = strstr( psz_string, psz_name );
-
-    if( b_add )
-    {
-        if( !psz_parser )
-        {
-            psz_parser = psz_string;
-            asprintf( &psz_string, (*psz_string) ? "%s:%s" : "%s%s",
-                            psz_string, psz_name );
-            free( psz_parser );
-        }
-        else
-        {
-            vlc_object_release( p_aout );
-            return;
-        }
-    }
-    else
-    {
-        if( psz_parser )
-        {
-            memmove( psz_parser, psz_parser + strlen(psz_name) +
-                            (*(psz_parser + strlen(psz_name)) == ':' ? 1 : 0 ),
-                            strlen(psz_parser + strlen(psz_name)) + 1 );
-
-            if( *(psz_string+strlen(psz_string ) -1 ) == ':' )
-            {
-                *(psz_string+strlen(psz_string ) -1 ) = '\0';
-            }
-         }
-         else
-         {
-             free( psz_string );
-             return;
-         }
-    }
-
-    if( p_aout == NULL )
-        config_PutPsz( p_this, "audio-filter", psz_string );
-    else
-    {
-        var_SetString( p_aout, "audio-filter", psz_string );
-        for( int i = 0; i < p_aout->i_nb_inputs; i++ )
-            p_aout->pp_inputs[i]->b_restart = true;
-        vlc_object_release( p_aout );
-    }
-    free( psz_string );
-}
-