Commit 5c745329 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

directory: small simplification

parent 70cf9406
...@@ -81,10 +81,9 @@ struct directory_t ...@@ -81,10 +81,9 @@ struct directory_t
struct access_sys_t struct access_sys_t
{ {
directory_t *current; directory_t *current;
DIR *handle;
char *uri;
char *ignored_exts; char *ignored_exts;
int mode; char mode;
bool header;
int i_item_count; int i_item_count;
char *psz_xspf_extension; char *psz_xspf_extension;
}; };
...@@ -123,11 +122,30 @@ int DirInit (access_t *p_access, DIR *handle) ...@@ -123,11 +122,30 @@ int DirInit (access_t *p_access, DIR *handle)
if (unlikely(uri == NULL)) if (unlikely(uri == NULL))
goto error; goto error;
/* "Open" the base directory */
directory_t *root = malloc (sizeof (*root));
if (unlikely(root == NULL))
{
free (uri);
goto error;
}
root->parent = NULL;
root->handle = handle;
root->uri = uri;
#ifndef HAVE_OPENAT
root->path = strdup (p_access->psz_filepath);
#endif
if (fstat (dirfd (handle), &root->st))
{
free (root);
free (uri);
goto error;
}
p_access->p_sys = p_sys; p_access->p_sys = p_sys;
p_sys->current = NULL; p_sys->current = root;
p_sys->handle = handle;
p_sys->uri = uri;
p_sys->ignored_exts = var_InheritString (p_access, "ignore-filetypes"); p_sys->ignored_exts = var_InheritString (p_access, "ignore-filetypes");
p_sys->header = true;
p_sys->i_item_count = 0; p_sys->i_item_count = 0;
p_sys->psz_xspf_extension = strdup( "" ); p_sys->psz_xspf_extension = strdup( "" );
...@@ -178,11 +196,6 @@ void DirClose( vlc_object_t * p_this ) ...@@ -178,11 +196,6 @@ void DirClose( vlc_object_t * p_this )
free (current); free (current);
} }
/* corner case: Block() not called ever */
if (p_sys->handle != NULL)
closedir (p_sys->handle);
free (p_sys->uri);
free (p_sys->psz_xspf_extension); free (p_sys->psz_xspf_extension);
free (p_sys->ignored_exts); free (p_sys->ignored_exts);
free (p_sys); free (p_sys);
...@@ -214,7 +227,7 @@ block_t *DirBlock (access_t *p_access) ...@@ -214,7 +227,7 @@ block_t *DirBlock (access_t *p_access)
if (p_access->info.b_eof) if (p_access->info.b_eof)
return NULL; return NULL;
if (current == NULL) if (p_sys->header)
{ /* Startup: send the XSPF header */ { /* Startup: send the XSPF header */
static const char header[] = static const char header[] =
"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n" "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n"
...@@ -224,30 +237,7 @@ block_t *DirBlock (access_t *p_access) ...@@ -224,30 +237,7 @@ block_t *DirBlock (access_t *p_access)
if (!block) if (!block)
goto fatal; goto fatal;
memcpy (block->p_buffer, header, sizeof (header) - 1); memcpy (block->p_buffer, header, sizeof (header) - 1);
p_sys->header = false;
/* "Open" the base directory */
current = malloc (sizeof (*current));
if (current == NULL)
{
block_Release (block);
goto fatal;
}
current->parent = NULL;
current->handle = p_sys->handle;
#ifndef HAVE_OPENAT
current->path = strdup (p_access->psz_filepath);
#endif
current->uri = p_sys->uri;
if (fstat (dirfd (current->handle), &current->st))
{
free (current);
block_Release (block);
goto fatal;
}
p_sys->handle = NULL;
p_sys->uri = NULL;
p_sys->current = current;
return block; return block;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment