#include <assert.h>
#include <stdio.h>
+#include <limits.h> /* NAME_MAX */
#include <errno.h>
#include <sys/types.h>
-#ifdef HAVE_DIRENT_H
-# include <dirent.h>
-#endif
+#include <dirent.h>
#ifdef HAVE_SYS_STAT_H
# include <sys/stat.h>
#endif
return -1;
}
-#ifdef HAVE_FDOPENDIR
+#ifdef HAVE_OPENAT
int fd = openat (dir, local_name, flags, mode);
# ifdef HAVE_FCNTL
if (fd != -1)
*
* @param dir The directory that is being read
*
- * @return a UTF-8 string of the directory entry.
- * Use free() to free this memory.
+ * @return a UTF-8 string of the directory entry. Use free() to release it.
+ * If there are no more entries in the directory, NULL is returned.
+ * If an error occurs, errno is set and NULL is returned.
*/
char *vlc_readdir( DIR *dir )
{
return FromWide (ent->d_name);
#else
+ /* Beware that readdir_r() assumes <buf> is large enough to hold the result
+ * dirent including the file name. A buffer overflow could occur otherwise.
+ * In particular, pathconf() and _POSIX_NAME_MAX cannot be used here. */
struct dirent *ent;
+ char *path = NULL;
+
+ long len = fpathconf (dirfd (dir), _PC_NAME_MAX);
+ if (len == -1)
+ {
+#ifdef NAME_MAX
+ len = NAME_MAX;
+#else
+ errno = ENOMEM;
+ return NULL; // OS is broken. There is no sane way to fix this.
+#endif
+ }
+ len += offsetof (struct dirent, d_name) + 1;
- ent = readdir( (DIR *)dir );
- if( ent == NULL )
+ struct dirent *buf = malloc (len);
+ if (unlikely(buf == NULL))
return NULL;
- return vlc_fix_readdir( ent->d_name );
+ int val = readdir_r (dir, buf, &ent);
+ if (val != 0)
+ errno = val;
+ else if (ent != NULL)
+ path = vlc_fix_readdir (ent->d_name);
+ free (buf);
+ return path;
#endif
}
int (*select)( const char * ),
int (*compar)( const char **, const char ** ) )
{
- if( select == NULL )
+ assert (dir);
+
+ if (select == NULL)
select = dummy_select;
- if( dir == NULL )
- return -1;
- else
- {
- char **tab = NULL;
- char *entry;
- unsigned num = 0;
+ char **tab = NULL;
+ unsigned num = 0;
- rewinddir( dir );
+ rewinddir (dir);
- while( ( entry = vlc_readdir( dir ) ) != NULL )
+ for (unsigned size = 0;;)
+ {
+ errno = 0;
+ char *entry = vlc_readdir (dir);
+ if (entry == NULL)
{
- char **newtab;
+ if (errno)
+ goto error;
+ break;
+ }
- if( !select( entry ) )
- {
- free( entry );
- continue;
- }
+ if (!select (entry))
+ {
+ free (entry);
+ continue;
+ }
+
+ if (num >= size)
+ {
+ size = size ? (2 * size) : 16;
+ char **newtab = realloc (tab, sizeof (*tab) * (size));
- newtab = realloc( tab, sizeof( char * ) * (num + 1) );
- if( newtab == NULL )
+ if (unlikely(newtab == NULL))
{
- free( entry );
+ free (entry);
goto error;
}
tab = newtab;
- tab[num++] = entry;
}
- if( compar != NULL )
- qsort( tab, num, sizeof( tab[0] ),
- (int (*)( const void *, const void *))compar );
-
- *namelist = tab;
- return num;
+ tab[num++] = entry;
+ }
- error:{
- unsigned i;
+ if (compar != NULL)
+ qsort (tab, num, sizeof (*tab),
+ (int (*)( const void *, const void *))compar);
+ *namelist = tab;
+ return num;
- for( i = 0; i < num; i++ )
- free( tab[i] );
- free( tab );
- }
- }
+error:
+ for (unsigned i = 0; i < num; i++)
+ free (tab[i]);
+ free (tab);
return -1;
}
* Selects file entries from a directory, as GNU C scandir().
*
* @param dirname UTF-8 diretory path
- * @param pointer [OUT] pointer set, on succesful completion, to the address
+ * @param pointer [OUT] pointer set, on successful completion, to the address
* of a table of UTF-8 filenames. All filenames must be freed with free().
* The table itself must be freed with free() as well.
*
}
/**
- * Finds file/inode informations, as stat().
+ * Finds file/inode information, as stat().
* Consider using fstat() instead, if possible.
*
* @param filename UTF-8 file path
}
/**
- * Finds file/inode informations, as lstat().
+ * Finds file/inode information, as lstat().
* Consider using fstat() instead, if possible.
*
* @param filename UTF-8 file path
else
return -1;
#else
- return _wrename (wold, wnew);
+ if (_wrename (wold, wnew) && errno == EACCES)
+ { /* Windows does not allow atomic file replacement */
+ if (_wremove (wnew))
+ {
+ errno = EACCES; /* restore errno */
+ return -1;
+ }
+ if (_wrename (wold, wnew))
+ return -1;
+ }
+ return 0;
#endif
#endif