* BUGFIX : Estaba siendo olvidado verificar de no sobrepasar

[z.facultad/75.06/emufs.git] / emufs / emufs.c

diff --git a/emufs/emufs.c b/emufs/emufs.c
index 9a0344176e83b62669a5cdb0afb4ed5b01ca88fa..b946e482334a0b8b1204503c2fde59725d67bd78 100644 (file)
--- a/emufs/emufs.c
+++ b/emufs/emufs.c
@@ -84,6 +84,7 @@ EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_b
        char name[255];
        FILE *fp;
        EMUFS *efs;
        char name[255];
        FILE *fp;
        EMUFS *efs;

+       int err = 0;

 
        /* Si no es un tipo conocido, sale. */
        if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {
 
        /* Si no es un tipo conocido, sale. */
        if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {


@@ -143,9 +144,12 @@ EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_b
 
                case T1:
                        /* Asigna punteros a funciones. */
 
                case T1:
                        /* Asigna punteros a funciones. */

-                       /* TODO verificar que el tamaño de bloque sea como mínimo del
-                        * tamaño de la cabecera de un registro + N */
-                       emufs_tipo1_inicializar(efs);
+                       if ((err = emufs_tipo1_inicializar(efs))) {
+                               PERR("No se pudo inicializar el EMUFS de tipo1");
+                               free(efs->nombre);
+                               free(efs);
+                               return NULL;
+                       }

 
                        /* Guarda cabeceras propias. */
                        fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);
 
                        /* Guarda cabeceras propias. */
                        fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);


@@ -167,6 +171,7 @@ EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_b
                        efs->leer_estadisticas = emufs_tipo3_leer_estadisticas;
                        efs->modificar_registro = emufs_tipo3_modificar_registro;
                        efs->compactar = emufs_tipo3_compactar;
                        efs->leer_estadisticas = emufs_tipo3_leer_estadisticas;
                        efs->modificar_registro = emufs_tipo3_modificar_registro;
                        efs->compactar = emufs_tipo3_compactar;

+                       efs->leer_bloque_raw = emufs_tipo3_leer_bloque_raw;

                        /* Guarda cabeceras propias. */
                        fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);
                        fwrite(&tam_reg, sizeof(EMUFS_REG_SIZE), 1, fp);                        
                        /* Guarda cabeceras propias. */
                        fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);
                        fwrite(&tam_reg, sizeof(EMUFS_REG_SIZE), 1, fp);                        


@@ -184,6 +189,7 @@ EMUFS *emufs_abrir(const char *filename)
        char name[255];
        char tipo;
        FILE *fp;
        char name[255];
        char tipo;
        FILE *fp;

+       int err = 0;

 
        strcpy(name, filename);
        strcat(name, ".dat");
 
        strcpy(name, filename);
        strcat(name, ".dat");


@@ -207,11 +213,9 @@ EMUFS *emufs_abrir(const char *filename)
        }
        efs->tipo = tipo;
        efs->nombre = str_dup(filename);
        }
        efs->tipo = tipo;
        efs->nombre = str_dup(filename);

-
+       

        switch (tipo) {
                case T1:
        switch (tipo) {
                case T1:

-                       /* Asigna punteros a funciones. */
-                       emufs_tipo1_inicializar(efs);

                        /* Lee cabeceras propias. */
                        if (!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) {
                                free(efs->nombre);
                        /* Lee cabeceras propias. */
                        if (!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) {
                                free(efs->nombre);


@@ -219,6 +223,12 @@ EMUFS *emufs_abrir(const char *filename)
                                fclose(fp);
                                return NULL;
                        }
                                fclose(fp);
                                return NULL;
                        }

+                       /* Asigna punteros a funciones. */
+                       if ((err = emufs_tipo1_inicializar(efs))) {
+                               PERR("No se pudo inicializar el EMUFS de tipo1");
+                               fclose(fp);
+                               return NULL;
+                       }

                        break;
                case T2:
                        /* Asigna punteros a funciones. */
                        break;
                case T2:
                        /* Asigna punteros a funciones. */


@@ -242,6 +252,7 @@ EMUFS *emufs_abrir(const char *filename)
                        efs->leer_estadisticas = emufs_tipo3_leer_estadisticas;
                        efs->modificar_registro = emufs_tipo3_modificar_registro;
                        efs->compactar = emufs_tipo3_compactar;
                        efs->leer_estadisticas = emufs_tipo3_leer_estadisticas;
                        efs->modificar_registro = emufs_tipo3_modificar_registro;
                        efs->compactar = emufs_tipo3_compactar;

+                       efs->leer_bloque_raw = emufs_tipo3_leer_bloque_raw;

                        break;
        }
 
                        break;
        }