]> git.sesse.net Git - vlc/blobdiff - modules/access/file.c
Don't print a message a malloc failed.
[vlc] / modules / access / file.c
index 3454e9dfbf2daef380099fb92466fb7df75dcab7..cdf51accde8e2e351015778adc852e98fe84a228 100644 (file)
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <vlc/vlc.h>
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <vlc_common.h>
+#include <vlc_plugin.h>
 #include <vlc_input.h>
 #include <vlc_access.h>
 #include <vlc_interface.h>
 
 #if defined( WIN32 ) && !defined( UNDER_CE )
 #   include <io.h>
+#   include <ctype.h>
 #else
 #   include <unistd.h>
 #   include <poll.h>
 #endif
-#ifdef HAVE_MMAP
-#   include <sys/mman.h>
-#endif
 
 #if defined( WIN32 ) && !defined( UNDER_CE )
 #   ifdef lseek
@@ -82,22 +85,17 @@ static void Close( vlc_object_t * );
 #define CACHING_LONGTEXT N_( \
     "Caching value for files. This " \
     "value should be set in milliseconds." )
-#define CAT_TEXT N_("Concatenate with additional files")
-#define CAT_LONGTEXT N_( \
-    "Play split files as if they were part of a unique file. " \
-    "You need to specify a comma-separated list of files." )
 
 vlc_module_begin();
-    set_description( _("File input") );
-    set_shortname( _("File") );
+    set_description( N_("File input") );
+    set_shortname( N_("File") );
     set_category( CAT_INPUT );
     set_subcategory( SUBCAT_INPUT_ACCESS );
-    add_integer( "file-caching", DEFAULT_PTS_DELAY / 1000, NULL, CACHING_TEXT, CACHING_LONGTEXT, VLC_TRUE );
+    add_integer( "file-caching", DEFAULT_PTS_DELAY / 1000, NULL, CACHING_TEXT, CACHING_LONGTEXT, true );
     add_obsolete_string( "file-cat" );
-    set_capability( "access2", 50 );
+    set_capability( "access", 50 );
     add_shortcut( "file" );
     add_shortcut( "stream" );
-    add_shortcut( "kfir" );
     set_callbacks( Open, Close );
 vlc_module_end();
 
@@ -108,20 +106,18 @@ vlc_module_end();
 static int  Seek( access_t *, int64_t );
 static ssize_t Read( access_t *, uint8_t *, size_t );
 static int  Control( access_t *, int, va_list );
-static block_t *mmapBlock( access_t * );
 
 static int  open_file( access_t *, const char * );
 
 struct access_sys_t
 {
     unsigned int i_nb_reads;
-    vlc_bool_t   b_kfir;
 
     int fd;
 
     /* */
-    vlc_bool_t b_seekable;
-    vlc_bool_t b_pace_control;
+    bool b_seekable;
+    bool b_pace_control;
 };
 
 /*****************************************************************************
@@ -132,31 +128,24 @@ static int Open( vlc_object_t *p_this )
     access_t     *p_access = (access_t*)p_this;
     access_sys_t *p_sys;
 
-    vlc_bool_t    b_stdin = !strcmp (p_access->psz_path, "-");
+    bool    b_stdin = !strcmp (p_access->psz_path, "-");
 
     /* Update default_pts to a suitable value for file access */
     var_Create( p_access, "file-caching", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT );
 
     STANDARD_READ_ACCESS_INIT;
     p_sys->i_nb_reads = 0;
-    p_sys->b_kfir = VLC_FALSE;
     int fd = p_sys->fd = -1;
 
     if (!strcasecmp (p_access->psz_access, "stream"))
     {
-        p_sys->b_seekable = VLC_FALSE;
-        p_sys->b_pace_control = VLC_FALSE;
-    }
-    else if (!strcasecmp (p_access->psz_access, "kfir"))
-    {
-        p_sys->b_seekable = VLC_FALSE;
-        p_sys->b_pace_control = VLC_FALSE;
-        p_sys->b_kfir = VLC_TRUE;
+        p_sys->b_seekable = false;
+        p_sys->b_pace_control = false;
     }
     else
     {
-        p_sys->b_seekable = VLC_TRUE;
-        p_sys->b_pace_control = VLC_TRUE;
+        p_sys->b_seekable = true;
+        p_sys->b_pace_control = true;
     }
 
     /* Open file */
@@ -195,42 +184,13 @@ static int Open( vlc_object_t *p_this )
 
 #ifdef HAVE_SYS_STAT_H
     p_access->info.i_size = st.st_size;
-    if (!S_ISREG (st.st_mode) && !S_ISBLK (st.st_mode)
-     && (!S_ISCHR (st.st_mode) || (st.st_size == 0)))
-        p_sys->b_seekable = VLC_FALSE;
-
-# ifdef HAVE_MMAP
-    /* Autodetect mmap() support */
-    if (p_sys->b_pace_control && S_ISREG (st.st_mode) && (st.st_size > 0))
-    {
-        void *addr = mmap (NULL, 1, PROT_READ, MAP_PRIVATE, fd, 0);
-        if (addr != MAP_FAILED)
-        {
-            /* Does the file system support mmap? */
-            munmap (addr, 1);
-            p_access->pf_read = NULL;
-            p_access->pf_block = mmapBlock;
-            msg_Dbg (p_this, "mmap enabled");
-        }
-        else
-            msg_Dbg (p_this, "mmap disabled (%m)");
-    }
-    else
-        msg_Dbg (p_this, "mmap disabled (non regular file)");
-# endif
+    if (!S_ISREG (st.st_mode))
+        p_sys->b_seekable = false;
 #else
     p_sys->b_seekable = !b_stdin;
 # warning File size not known!
 #endif
 
-    if (p_sys->b_seekable && (p_access->info.i_size == 0))
-    {
-        /* FIXME that's bad because all others access will be probed */
-        msg_Err (p_access, "file is empty, aborting");
-        Close (p_this);
-        return VLC_EGENERIC;
-    }
-
     return VLC_SUCCESS;
 }
 
@@ -256,43 +216,23 @@ static ssize_t Read( access_t *p_access, uint8_t *p_buffer, size_t i_len )
     int fd = p_sys->fd;
 
 #if !defined(WIN32) && !defined(UNDER_CE)
-    if( !p_sys->b_pace_control )
+    if( !p_sys->b_seekable )
     {
-        if( !p_sys->b_kfir )
-        {
-            /* Find if some data is available. This won't work under Windows. */
-            do
-            {
-                struct pollfd ufd;
-
-                if( p_access->b_die )
-                    return 0;
-
-                memset (&ufd, 0, sizeof (ufd));
-                ufd.fd = fd;
-                ufd.events = POLLIN;
-
-                i_ret = poll (&ufd, 1, 500);
-            }
-            while (i_ret <= 0);
-
-            i_ret = read (fd, p_buffer, i_len);
-        }
-        else
-        {
-            /* b_kfir ; work around a buggy poll() driver implementation */
-            while (((i_ret = read (fd, p_buffer, i_len)) == 0)
-                && !p_access->b_die)
-            {
-                msleep( INPUT_ERROR_SLEEP );
-            }
-        }
+        /* Note that POSIX regular files (b_seekable) opened for read are
+         * guaranteed to always set POLLIN immediately, so we can spare
+         * poll()ing them. */
+        /* Wait until some data is available. Impossible on Windows. */
+        struct pollfd ufd[2] = {
+            { .fd = fd, .events = POLLIN, },
+            { .fd = vlc_object_waitpipe (p_access), .events = POLLIN, },
+        };
+
+        if (poll (ufd, 2, -1) < 0 || ufd[1].revents)
+            return -1;
     }
-    else
 #endif /* WIN32 || UNDER_CE */
-        /* b_pace_control || WIN32 */
-        i_ret = read( fd, p_buffer, i_len );
 
+    i_ret = read (fd, p_buffer, i_len);
     if( i_ret < 0 )
     {
         switch (errno)
@@ -303,14 +243,14 @@ static ssize_t Read( access_t *p_access, uint8_t *p_buffer, size_t i_len )
 
             default:
                 msg_Err (p_access, "read failed (%m)");
-                intf_UserFatal (p_access, VLC_FALSE, _("File reading failed"),
+                intf_UserFatal (p_access, false, _("File reading failed"),
                                 _("VLC could not read the file."));
         }
-
-        /* Delay a bit to avoid consuming all the CPU. This is particularly
-         * useful when reading from an unconnected FIFO. */
-        msleep( INPUT_ERROR_SLEEP );
     }
+    else if( i_ret > 0 )
+        p_access->info.i_pos += i_ret;
+    else if( i_ret == 0 )
+        p_access->info.b_eof = true;
 
     p_sys->i_nb_reads++;
 
@@ -328,130 +268,19 @@ static ssize_t Read( access_t *p_access, uint8_t *p_buffer, size_t i_len )
         }
     }
 #endif
-
-    if( i_ret > 0 )
-        p_access->info.i_pos += i_ret;
-    else if( i_ret == 0 )
-        p_access->info.b_eof = VLC_TRUE;
-
     return i_ret;
 }
 
-#ifdef HAVE_MMAP
-# define MMAP_SIZE (1 << 18)
-
-struct block_sys_t
-{
-    block_t self;
-    //vlc_object_t *owner;
-    void *base_addr;
-    size_t length;
-};
-
-static void mmapRelease (block_t *block)
-{
-    block_sys_t *p_sys = (block_sys_t *)block;
-
-    munmap (p_sys->base_addr, p_sys->length);
-    //vlc_object_release (p_sys->owner);
-    free (p_sys);
-}
-
-static block_t *mmapBlock (access_t *p_access)
-{
-    access_sys_t *p_sys = p_access->p_sys;
-
-    const int flags = MAP_SHARED;
-    const size_t pagesize = sysconf (_SC_PAGE_SIZE);
-    off_t offset = p_access->info.i_pos & ~(pagesize - 1);
-    size_t align = p_access->info.i_pos & (pagesize - 1);
-    size_t length = (MMAP_SIZE > pagesize) ? MMAP_SIZE : pagesize;
-    void *addr;
-
-    if (p_access->info.i_pos >= p_access->info.i_size)
-    {
-        /* End of file - check that file size hasn't change... */
-        struct stat st;
-
-        if ((fstat (p_sys->fd, &st) == 0)
-         && (st.st_size != p_access->info.i_size))
-        {
-            p_access->info.i_size = st.st_size;
-            p_access->info.i_update |= INPUT_UPDATE_SIZE;
-        }
-
-        /* Really at end of file then */
-        if (p_access->info.i_pos >= p_access->info.i_size)
-        {
-            p_access->info.b_eof = VLC_TRUE;
-            msg_Dbg (p_access, "at end of memory mapped file");
-            return NULL;
-        }
-    }
-
-    if (offset + length > p_access->info.i_size)
-        /* Don't mmap paste end of file */
-        length = p_access->info.i_size - offset;
-
-    assert (offset <= p_access->info.i_pos);               /* and */
-    assert (p_access->info.i_pos < p_access->info.i_size); /* imply */
-    assert (offset < p_access->info.i_size);               /* imply */
-    assert (length > 0);
-
-    addr = mmap (NULL, length, PROT_READ, flags, p_sys->fd, offset);
-    if (addr == MAP_FAILED)
-    {
-        msg_Err (p_access, "memory mapping failed (%m)");
-        intf_UserFatal (p_access, VLC_FALSE, _("File reading failed"),
-                        _("VLC could not read the file."));
-        msleep( INPUT_ERROR_SLEEP );
-        return NULL;
-    }
-
-    p_access->info.i_pos = offset + length;
-
-    msg_Dbg (p_access, "mapped %lu bytes at %p from offset %lu",
-             (unsigned long)length, addr, (unsigned long)offset);
-    block_sys_t *block = malloc (sizeof (*block));
-    if (block == NULL)
-    {
-        munmap (addr, length);
-        return NULL;
-    }
-
-    block_Init (&block->self, ((uint8_t *)addr) + align, length - align);
-    block->self.pf_release = mmapRelease;
-    block->base_addr = addr;
-    block->length = length;
-    //vlc_object_yield (block->owner = VLC_OBJECT (p_access));
-
-    return &block->self;
-}
-#endif
 
 /*****************************************************************************
  * Seek: seek to a specific location in a file
  *****************************************************************************/
 static int Seek (access_t *p_access, int64_t i_pos)
 {
-    if (i_pos > p_access->info.i_size)
-    {
-        msg_Err (p_access, "seeking too far");
-        i_pos = p_access->info.i_size;
-    }
-    else if (i_pos < 0)
-    {
-        msg_Err (p_access, "seeking too early");
-        i_pos = 0;
-    }
-
     p_access->info.i_pos = i_pos;
-    p_access->info.b_eof = VLC_FALSE;
+    p_access->info.b_eof = false;
 
-#ifdef HAVE_MMAP
-    if (p_access->pf_block == NULL)
-        lseek (p_access->p_sys->fd, i_pos, SEEK_SET);
-#endif
+    lseek (p_access->p_sys->fd, i_pos, SEEK_SET);
     return VLC_SUCCESS;
 }
 
@@ -461,7 +290,7 @@ static int Seek (access_t *p_access, int64_t i_pos)
 static int Control( access_t *p_access, int i_query, va_list args )
 {
     access_sys_t *p_sys = p_access->p_sys;
-    vlc_bool_t   *pb_bool;
+    bool   *pb_bool;
     int          *pi_int;
     int64_t      *pi_64;
 
@@ -470,13 +299,13 @@ static int Control( access_t *p_access, int i_query, va_list args )
         /* */
         case ACCESS_CAN_SEEK:
         case ACCESS_CAN_FASTSEEK:
-            pb_bool = (vlc_bool_t*)va_arg( args, vlc_bool_t* );
+            pb_bool = (bool*)va_arg( args, bool* );
             *pb_bool = p_sys->b_seekable;
             break;
 
         case ACCESS_CAN_PAUSE:
         case ACCESS_CAN_CONTROL_PACE:
-            pb_bool = (vlc_bool_t*)va_arg( args, vlc_bool_t* );
+            pb_bool = (bool*)va_arg( args, bool* );
             *pb_bool = p_sys->b_pace_control;
             break;
 
@@ -488,7 +317,7 @@ static int Control( access_t *p_access, int i_query, va_list args )
 
         case ACCESS_GET_PTS_DELAY:
             pi_64 = (int64_t*)va_arg( args, int64_t * );
-            *pi_64 = var_GetInteger( p_access, "file-caching" ) * I64C(1000);
+            *pi_64 = var_GetInteger( p_access, "file-caching" ) * INT64_C(1000);
             break;
 
         /* */
@@ -501,57 +330,39 @@ static int Control( access_t *p_access, int i_query, va_list args )
         case ACCESS_SET_SEEKPOINT:
         case ACCESS_SET_PRIVATE_ID_STATE:
         case ACCESS_GET_META:
+        case ACCESS_GET_PRIVATE_ID_STATE:
         case ACCESS_GET_CONTENT_TYPE:
             return VLC_EGENERIC;
 
         default:
-            msg_Warn( p_access, "unimplemented query in control" );
+            msg_Warn( p_access, "unimplemented query %d in control", i_query );
             return VLC_EGENERIC;
 
     }
     return VLC_SUCCESS;
 }
 
-
-static char *expand_path (const access_t *p_access, const char *path)
-{
-    if (strncmp (path, "~/", 2) == 0)
-    {
-        char *res;
-
-         // TODO: we should also support the ~cmassiot/ syntax
-         if (asprintf (&res, "%s/%s", p_access->p_libvlc->psz_homedir, path + 2) == -1)
-             return NULL;
-         return res;
-    }
-
-#if defined(WIN32)
-    if (!strcasecmp (p_access->psz_access, "file")
-      && ('/' == path[0]) && path[1] && (':' == path[2]) && ('/' == path[3]))
-        // Explorer can open path such as file:/C:/ or file:///C:/
-        // hence remove leading / if found
-        return strdup (path + 1);
-#endif
-
-    return strdup (path);
-}
-
-
 /*****************************************************************************
  * open_file: Opens a specific file
  *****************************************************************************/
-static int open_file (access_t *p_access, const char *psz_name)
+static int open_file (access_t *p_access, const char *path)
 {
-    char *path = expand_path (p_access, psz_name);
+#if defined(WIN32)
+    if (!strcasecmp (p_access->psz_access, "file")
+      && ('/' == path[0]) && isalpha (path[1])
+      && (':' == path[2]) && ('/' == path[3]))
+        /* Explorer can open path such as file:/C:/ or file:///C:/
+         * hence remove leading / if found */
+        path++;
+#endif
 
 #ifdef UNDER_CE
     p_sys->fd = utf8_fopen( path, "rb" );
     if ( !p_sys->fd )
     {
-        msg_Err( p_access, "cannot open file %s", psz_name );
-        intf_UserFatal( p_access, VLC_FALSE, _("File reading failed"),
-                        _("VLC could not open the file \"%s\"."), psz_name );
-        free (path);
+        msg_Err( p_access, "cannot open file %s", path );
+        intf_UserFatal( p_access, false, _("File reading failed"),
+                        _("VLC could not open the file \"%s\"."), path );
         return VLC_EGENERIC;
     }
 
@@ -561,12 +372,11 @@ static int open_file (access_t *p_access, const char *psz_name)
     fseek( p_sys->fd, 0, SEEK_SET );
 #else
     int fd = utf8_open (path, O_RDONLY | O_NONBLOCK /* O_LARGEFILE*/, 0666);
-    free (path);
     if (fd == -1)
     {
-        msg_Err (p_access, "cannot open file %s (%m)", psz_name);
-        intf_UserFatal (p_access, VLC_FALSE, _("File reading failed"),
-                        _("VLC could not open the file \"%s\"."), psz_name);
+        msg_Err (p_access, "cannot open file %s (%m)", path);
+        intf_UserFatal (p_access, false, _("File reading failed"),
+                        _("VLC could not open the file \"%s\"."), path);
         return -1;
     }