Re: [Lost] [PATCH] + cdi/ext2: Loeschen von Dateien und Verzeichnissen

[Kevin Wolf <mail@xxxxxxxxxxxxx>]

Am Sonntag, 5. Oktober 2008 15:58:53 schrieb Antoine Kaufmann:
> [Lost] [PATCH] + cdi/ext2: Loeschen von Dateien und Verzeichnissen
> Von:
> Antoine Kaufmann <toni@xxxxxxxxxxxxxxxx>
>   An:
> "LOST-Patches" <lost@xxxxxxxxxx>
>   Datum:
> Heute 15:58:53
>   Anhänge: 
>  23b329356f4c3c45e6964be61e754324228a26a8.diff
>    
> ---
>  src/modules/cdi/ext2/res.c       |   64
> ++++++++++++++++++++++++++++++++++++++ src/modules/cdi/ext2/resources.c |  
>  4 ++-
>  2 files changed, 67 insertions(+), 1 deletions(-)
> 23b329356f4c3c45e6964be61e754324228a26a8.diff
>   diff --git a/src/modules/cdi/ext2/res.c b/src/modules/cdi/ext2/res.c
> index 21af41d..cc7c331 100644
> --- a/src/modules/cdi/ext2/res.c
> +++ b/src/modules/cdi/ext2/res.c
> @@ -271,3 +271,67 @@ error_out:
>      return 0;
>  }
>  
> +int ext2_fs_res_remove_class(struct cdi_fs_stream* stream,
> +    cdi_fs_res_class_t class)
> +{
> +    struct ext2_fs_res* res = (struct ext2_fs_res*) stream->res;
> +    struct ext2_fs_res* parent_res = (struct ext2_fs_res*)
> res->res.parent; +
> +    // Bei Verzeichnissen muessen noch ein paar Interne Daten freigegeben
> werden +    if (class == CDI_FS_CLASS_DIR) {
> +        dir_clear(stream);
> +    }
> +
> +    if (!ext2_dir_unlink(parent_res->inode, res->res.name)) {
> +        stream->error = CDI_FS_ERROR_IO;
> +        goto error_out;
> +    }

Hm, wäre erst unlinken und dann freigeben nicht schlauer, falls es schiefgeht? 
Gibt es in der CDI-"Doku" eigentlich irgendwelche Beschreibungen, was im 
Fehlerfall garantiert wird? Falls nein, sollte man das mal nachholen.

Und was passiert, wenn ich eine Klasse freigebe, die die Ressource gar nicht 
hat? Mit dieser Implementierung würde sie trotzdem gelöscht werden, oder? 
Halbwegs korrekt ist die Sache sowieso nur, weil eine Datei in ext2 nicht 
mehrere Klassen gleichzeitig haben kann.

> +
> +    free(res->inode);
> +    res->inode = NULL;
> +
> +    switch (class) {
> +        case CDI_FS_CLASS_FILE:
> +            res->res.file = NULL;
> +            break;
> +
> +        case CDI_FS_CLASS_DIR:
> +            res->res.dir = NULL;
> +            break;
> +
> +        case CDI_FS_CLASS_LINK:
> +            res->res.link = NULL;
> +            break;
> +
> +        case CDI_FS_CLASS_SPECIAL:
> +            res->res.special = NULL;
> +            break;
> +
> +    };
> +
> +    return 1;
> +error_out:
> +    return 0;
> +}
> +
> +int ext2_fs_res_remove(struct cdi_fs_stream* stream)
> +{
> +    struct ext2_fs_res* res = (struct ext2_fs_res*) stream->res;
> +    struct ext2_fs_res* parent_res = (struct ext2_fs_res*)
> res->res.parent; +    struct ext2_fs_res* child_res;
> +    int i;
> +
> +    // Eintrag aus dem Elternverzeichnis loeschen
> +    for (i = 0; parent_res->res.children &&
> +        (child_res = cdi_list_get(parent_res->res.children, i)); i++)
> +    {
> +        if (!strcmp(child_res->res.name, res->res.name)) {
> +            cdi_list_remove(parent_res->res.children, i);
> +            break;
> +        }
> +    }

Hm, bin ich jetzt blind oder fehlt da irgendwo ein libext2-Aufruf, um die 
Datei auch wirklich physisch und nicht nur im Speicher zu löschen?

> +
> +    free(res->res.name);
> +    free(res);
> +    return 1;
> +}
> diff --git a/src/modules/cdi/ext2/resources.c
> b/src/modules/cdi/ext2/resources.c index 560783e..8b31a42 100644
> --- a/src/modules/cdi/ext2/resources.c
> +++ b/src/modules/cdi/ext2/resources.c
> @@ -38,11 +38,13 @@
>  struct cdi_fs_res_res   ext2_fs_res = {
>      .load = ext2_fs_res_load,
>      .unload = ext2_fs_res_unload,
> +    .remove = ext2_fs_res_remove,
>  
>      .meta_read = ext2_fs_res_meta_read,
>      .meta_write = ext2_fs_res_meta_write,
>  
> -    .assign_class = ext2_fs_res_assign_class
> +    .assign_class = ext2_fs_res_assign_class,
> +    .remove_class = ext2_fs_res_remove_class

Mach gleich ein Komma hinten dran.

>  };
>  
>  struct cdi_fs_res_file  ext2_fs_file = {
>   _______________________________________________
> Lost mailing list
> Lost@xxxxxxxxxx
> http://list.tyndur.org/mailman/listinfo/lost