]> git.sesse.net Git - vlc/blobdiff - modules/access/directory.c
Don't use someone's else domain in our default values
[vlc] / modules / access / directory.c
index 44f52853d0c92faea1755e96beed7ce6c3818023..5f36d4399aa463b039813ec27a4f13a2e0cecf5f 100644 (file)
@@ -18,7 +18,7 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 /*****************************************************************************
@@ -28,6 +28,7 @@
 #include <vlc/vlc.h>
 #include <vlc/input.h>
 #include <vlc_playlist.h>
+#include <vlc_input.h>
 
 #include <stdlib.h>
 #include <string.h>
@@ -77,11 +78,12 @@ static char *psz_recursive_list[] = { "none", "collapse", "expand" };
 static char *psz_recursive_list_text[] = { N_("none"), N_("collapse"),
                                            N_("expand") };
 
-#define IGNORE_TEXT N_("Ignore files with these extensions")
+#define IGNORE_TEXT N_("Ignored extensions")
 #define IGNORE_LONGTEXT N_( \
-        "Specify a comma seperated list of file extensions. " \
-        "Files with these extensions will not be added to playlist when opening a directory. " \
-        "This is useful if you add directories that contain mp3 albums for instance." )
+        "Files with these extensions will not be added to playlist when " \
+        "opening a directory.\n" \
+        "This is useful if you add directories that contain playlist files " \
+        "for instance. Use a comma-separated list of extensions." )
 
 vlc_module_begin();
     set_category( CAT_INPUT );
@@ -122,8 +124,8 @@ static int Demux( demux_t *p_demux );
 static int DemuxControl( demux_t *p_demux, int i_query, va_list args );
 
 
-static int ReadDir( playlist_t *, char *psz_name, int i_mode, int *pi_pos,
-                    playlist_item_t * );
+static int ReadDir( playlist_t *, const char *psz_name, int i_mode,
+                    playlist_item_t *, playlist_item_t * );
 
 /*****************************************************************************
  * Open: open the directory
@@ -197,9 +199,10 @@ static int Read( access_t *p_access, uint8_t *p_buffer, int i_len)
 {
     char *psz_name = NULL;
     char *psz;
-    int  i_mode, i_pos;
+    char *ptr;
+    int  i_mode, i_activity;
 
-    playlist_item_t *p_item;
+    playlist_item_t *p_item, *p_root_category;
     vlc_bool_t b_play = VLC_FALSE;
 
     playlist_t *p_playlist =
@@ -211,54 +214,40 @@ static int Read( access_t *p_access, uint8_t *p_buffer, int i_len)
         msg_Err( p_access, "can't find playlist" );
         goto end;
     }
-    else
-    {
-        char *ptr;
 
-        psz_name = ToLocale( p_access->psz_path );
-        ptr = strdup( psz_name );
-        LocaleFree( psz_name );
-        if( ptr == NULL )
-            goto end;
+    psz_name = ToLocale( p_access->psz_path );
+    ptr = strdup( psz_name );
+    LocaleFree( psz_name );
+    if( ptr == NULL )
+        goto end;
 
-        psz_name = ptr;
+    psz_name = ptr;
 
-        /* Remove the ending '/' char */
-        ptr += strlen( ptr );
-        if( ( ptr > psz_name ) )
+    /* Remove the ending '/' char */
+    ptr += strlen( ptr );
+    if( ( ptr > psz_name ) )
+    {
+        switch( *--ptr )
         {
-            switch( *--ptr )
-            {
-                case '/':
-                case '\\':
-                    *ptr = '\0';
-            }
+            case '/':
+            case '\\':
+                *ptr = '\0';
         }
     }
 
-    /* Initialize structure */
+    /* Handle mode */
     psz = var_CreateGetString( p_access, "recursive" );
     if( *psz == '\0' || !strncmp( psz, "none" , 4 )  )
-    {
         i_mode = MODE_NONE;
-    }
     else if( !strncmp( psz, "collapse", 8 )  )
-    {
         i_mode = MODE_COLLAPSE;
-    }
     else
-    {
         i_mode = MODE_EXPAND;
-    }
     free( psz );
 
-    /* Make sure we are deleted when we are done */
-    /* The playlist position we will use for the add */
-    i_pos = p_playlist->i_index + 1;
-
     msg_Dbg( p_access, "opening directory `%s'", p_access->psz_path );
 
-    if( &p_playlist->status.p_item->input ==
+    if( p_playlist->status.p_item && p_playlist->status.p_item->p_input ==
         ((input_thread_t *)p_access->p_parent)->input.p_item )
     {
         p_item = p_playlist->status.p_item;
@@ -278,20 +267,29 @@ static int Read( access_t *p_access, uint8_t *p_buffer, int i_len)
         }
         b_play = VLC_FALSE;
     }
+    p_item->p_input->i_type = ITEM_TYPE_DIRECTORY;
 
-    p_item->input.i_type = ITEM_TYPE_DIRECTORY;
-    if( ReadDir( p_playlist, psz_name , i_mode, &i_pos,
-                 p_item ) != VLC_SUCCESS )
-    {
-    }
+    p_root_category = playlist_LockItemToNode( p_playlist, p_item );
+
+    i_activity = var_GetInteger( p_playlist, "activity" );
+    var_SetInteger( p_playlist, "activity", i_activity +
+                    DIRECTORY_ACTIVITY );
+
+    ReadDir( p_playlist, psz_name , i_mode, p_item, p_root_category );
+
+    i_activity = var_GetInteger( p_playlist, "activity" );
+    var_SetInteger( p_playlist, "activity", i_activity -
+                    DIRECTORY_ACTIVITY );
 end:
 
     /* Begin to read the directory */
     if( b_play )
     {
-        playlist_Control( p_playlist, PLAYLIST_VIEWPLAY,
-                          p_playlist->status.i_view,
+#if 0
+       /// \bug we can start playing an already deleted item. Fix ?*/
+       playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, 1242,
                           p_playlist->status.p_item, NULL );
+#endif
     }
     if( psz_name ) free( psz_name );
     vlc_object_release( p_playlist );
@@ -369,6 +367,7 @@ static int Demux( demux_t *p_demux )
 {
     return 0;
 }
+
 /*****************************************************************************
  * DemuxControl:
  *****************************************************************************/
@@ -381,21 +380,38 @@ static int Filter( const struct dirent *foo )
 {
     return VLC_TRUE;
 }
+
 /*****************************************************************************
  * ReadDir: read a directory and add its content to the list
  *****************************************************************************/
-static int ReadDir( playlist_t *p_playlist,
-                    char *psz_name, int i_mode, int *pi_position,
-                    playlist_item_t *p_parent )
+static int ReadDir( playlist_t *p_playlist, const char *psz_name,
+                    int i_mode, playlist_item_t *p_parent,
+                    playlist_item_t *p_parent_category )
 {
-    struct dirent   **pp_dir_content;
-    int             i_dir_content, i;
+    struct dirent   **pp_dir_content = 0;
+    int             i_dir_content, i, i_return = VLC_SUCCESS;
     playlist_item_t *p_node;
 
-    /* Build array with ignores */
     char **ppsz_extensions = 0;
     int i_extensions = 0;
-    char *psz_ignore = var_CreateGetString( p_playlist, "ignore-filetypes" );
+    char *psz_ignore;
+
+    /* Get the first directory entry */
+    i_dir_content = scandir( psz_name, &pp_dir_content, Filter, alphasort );
+    if( i_dir_content == -1 )
+    {
+        msg_Warn( p_playlist, "failed to read directory" );
+        return VLC_EGENERIC;
+    }
+    else if( i_dir_content <= 0 )
+    {
+        /* directory is empty */
+        if( pp_dir_content ) free( pp_dir_content );
+        return VLC_SUCCESS;
+    }
+
+    /* Build array with ignores */
+    psz_ignore = var_CreateGetString( p_playlist, "ignore-filetypes" );
     if( psz_ignore && *psz_ignore )
     {
         char *psz_parser = psz_ignore;
@@ -410,41 +426,19 @@ static int ReadDir( playlist_t *p_playlist,
 
         for( a = 0; a < i_extensions; a++ )
         {
-            int b;
-            char *tmp;
-            
+            char *tmp, *ptr;
+
             while( psz_parser[0] != '\0' && psz_parser[0] == ' ' ) psz_parser++;
-            for( b = 0; psz_parser[b] != '\0'; b++ )
-            {
-                if( psz_parser[b] == ',' ) break;
-            }
-            tmp = malloc( b + 1 );
-            strncpy( tmp, psz_parser, b );
-            tmp[b] = 0;
+            ptr = strchr( psz_parser, ',');
+            tmp = ( ptr == NULL )
+                 ? strdup( psz_parser )
+                 : strndup( psz_parser, ptr - psz_parser );
+
             ppsz_extensions[a] = tmp;
-            psz_parser += b+1;
+            psz_parser = ptr + 1;
         }
     }
 
-    /* Change the item to a node */
-    if( p_parent->i_children == -1 )
-    {
-        playlist_LockItemToNode( p_playlist,p_parent );
-    }
-
-    /* get the first directory entry */
-    i_dir_content = scandir( psz_name, &pp_dir_content, Filter, alphasort );
-    if( i_dir_content == -1 )
-    {
-        msg_Warn( p_playlist, "Failed to read directory" );
-        return VLC_EGENERIC;
-    }
-    else if( i_dir_content <= 0 )
-    {
-        /* directory is empty */
-        return VLC_SUCCESS;
-    }
-
     /* While we still have entries in the directory */
     for( i = 0; i < i_dir_content; i++ )
     {
@@ -469,43 +463,65 @@ static int ReadDir( playlist_t *p_playlist,
             if( 0 )
 #endif
             {
+#if defined( S_ISLNK )
+/*
+ * FIXME: there is a ToCToU race condition here; but it is rather tricky
+ * impossible to fix while keeping some kind of portable code, and maybe even
+ * in a non-portable way.
+ */
+                if( lstat( psz_uri, &stat_data )
+                 || S_ISLNK(stat_data.st_mode) )
+                {
+                    msg_Dbg( p_playlist, "skipping directory symlink %s",
+                             psz_uri );
+                    free( psz_uri );
+                    continue;
+                }
+#endif
                 if( i_mode == MODE_NONE )
                 {
-                    msg_Dbg( p_playlist, "Skipping subdirectory %s", psz_uri );
+                    msg_Dbg( p_playlist, "skipping subdirectory %s", psz_uri );
                     free( psz_uri );
                     continue;
                 }
                 else if( i_mode == MODE_EXPAND )
                 {
                     char *psz_newname, *psz_tmp;
-                    msg_Dbg(p_playlist, "Reading subdirectory %s", psz_uri );
+                    msg_Dbg(p_playlist, "reading subdirectory %s", psz_uri );
 
                     psz_tmp = FromLocale( p_dir_content->d_name );
                     psz_newname = vlc_fix_readdir_charset(
                                                 p_playlist, psz_tmp );
                     LocaleFree( psz_tmp );
 
-                    p_node = playlist_NodeCreate( p_playlist,
-                                       p_parent->pp_parents[0]->i_view,
-                                       psz_newname, p_parent );
-
-                    playlist_CopyParents(  p_parent, p_node );
-
-                    p_node->input.i_type = ITEM_TYPE_DIRECTORY;
-
-                    if( ReadDir( p_playlist, psz_uri , MODE_EXPAND,
-                                 pi_position, p_node ) != VLC_SUCCESS )
+                    if( p_parent_category )
+                    {
+                        p_node = playlist_NodeCreate( p_playlist, psz_newname,
+                                                      p_parent_category );
+                    }
+                    else
                     {
-                        return VLC_EGENERIC;
+                        p_node = playlist_NodeCreate( p_playlist, psz_newname,
+                                                      p_parent_category );
                     }
 
                     /* an strdup() just because of Mac OS X */
                     free( psz_newname );
+
+                    /* If we had the parent in category, the it is now node.
+                     * Else, we still don't have  */
+                    if( ReadDir( p_playlist, psz_uri , MODE_EXPAND,
+                                 p_node, p_parent_category ? p_node : NULL )
+                          != VLC_SUCCESS )
+                    {
+                        i_return = VLC_EGENERIC;
+                        break;
+                    }
                 }
             }
             else
             {
-                playlist_item_t *p_item;
+                input_item_t *p_input;
                 char *psz_tmp1, *psz_tmp2, *psz_loc;
 
                 if( i_extensions > 0 )
@@ -521,7 +537,7 @@ static int ReadDir( playlist_t *p_playlist,
                         }
                         if( a < i_extensions )
                         {
-                            msg_Dbg( p_playlist, "Ignoring file %s", psz_uri );
+                            msg_Dbg( p_playlist, "ignoring file %s", psz_uri );
                             free( psz_uri );
                             continue;
                         }
@@ -538,27 +554,27 @@ static int ReadDir( playlist_t *p_playlist,
                                                     psz_loc );
                 LocaleFree( psz_loc );
 
-                p_item = playlist_ItemNewWithType( VLC_OBJECT(p_playlist),
-                        psz_tmp1, psz_tmp2, ITEM_TYPE_VFILE );
-                playlist_NodeAddItem( p_playlist,p_item,
-                                      p_parent->pp_parents[0]->i_view,
-                                      p_parent,
-                                      PLAYLIST_APPEND, PLAYLIST_END );
+                p_input = input_ItemNewWithType( VLC_OBJECT(p_playlist),
+                                                 psz_tmp1, psz_tmp2, 0, NULL,
+                                                 -1, ITEM_TYPE_VFILE );
 
-                playlist_CopyParents( p_parent, p_item );
+                playlist_AddWhereverNeeded( p_playlist, p_input, p_parent,
+                                            p_parent_category, VLC_FALSE,
+                                            PLAYLIST_APPEND|PLAYLIST_PREPARSE);
             }
         }
         free( psz_uri );
     }
 
     for( i = 0; i < i_extensions; i++ )
-    {
-        if( ppsz_extensions[i] )
-            free( ppsz_extensions[i] );
-    }
+        if( ppsz_extensions[i] ) free( ppsz_extensions[i] );
     if( ppsz_extensions ) free( ppsz_extensions );
+
     if( psz_ignore ) free( psz_ignore );
 
-    free( pp_dir_content );
-    return VLC_SUCCESS;
+    for( i = 0; i < i_dir_content; i++ )
+        if( pp_dir_content[i] ) free( pp_dir_content[i] );
+    if( pp_dir_content ) free( pp_dir_content );
+
+    return i_return;
 }