* 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 ef22753e9a34cfeef89adf206525cd00b947f230..b946e482334a0b8b1204503c2fde59725d67bd78 100644 (file)
--- a/emufs/emufs.c
+++ b/emufs/emufs.c
@@ -79,11 +79,12 @@ int emufs_crear_archivo_auxiliar(const char* name, const char* ext)
        return 0;
 }
 
-EMUFS *emufs_crear(const char *filename, EMUFS_TYPE tipo, EMUFS_BLOCK_SIZE tam_bloque, EMUFS_REG_SIZE tam_reg)
+EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_bloque, EMUFS_REG_SIZE tam_reg)
 {
        char name[255];
        FILE *fp;
        EMUFS *efs;
+       int err = 0;
 
        /* Si no es un tipo conocido, sale. */
        if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {
@@ -112,7 +113,7 @@ EMUFS *emufs_crear(const char *filename, EMUFS_TYPE tipo, EMUFS_BLOCK_SIZE tam_b
        }
 
        /* Guarda cabecera común. */
-       fwrite(&tipo, sizeof(EMUFS_TYPE), 1, fp);
+       fwrite(&tipo, sizeof(EMUFS_Tipo), 1, fp);
 
        /* Crea archivo de índice. */
        if (emufs_idx_crear(efs)) {
@@ -142,7 +143,13 @@ EMUFS *emufs_crear(const char *filename, EMUFS_TYPE tipo, EMUFS_BLOCK_SIZE tam_b
        switch (tipo) {
 
                case T1:
-                       emufs_tipo1_inicializar(efs);
+                       /* Asigna punteros a funciones. */
+                       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);
@@ -151,19 +158,20 @@ EMUFS *emufs_crear(const char *filename, EMUFS_TYPE tipo, EMUFS_BLOCK_SIZE tam_b
 
                case T2:
                        /* Asigna punteros a funciones. */
-                       efs->grabar_registro = emufs_tipo2_grabar_registro;           
-            efs->borrar_registro = emufs_tipo2_borrar_registro;
-                       efs->nombre = str_dup(filename);
-                   /*efs->leer_registro = emufs_tipo2_leer_registro;*/                                         
+                       emufs_tipo2_inicializar(efs);
                        break;
 
                case T3:
                        /* Asigna punteros a funciones. */
-                       efs->leer_bloque     = emufs_tipo3_leer_bloque;
-                       efs->leer_registro   = emufs_tipo3_leer_registro;
+                       efs->leer_bloque = emufs_tipo3_leer_bloque;
+                       efs->leer_registro = emufs_tipo3_leer_registro;
+                       efs->leer_registro_raw = emufs_tipo3_leer_registro_raw;
                        efs->grabar_registro = emufs_tipo3_grabar_registro;
                        efs->borrar_registro = emufs_tipo3_borrar_registro;
-
+                       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);                        
@@ -181,6 +189,7 @@ EMUFS *emufs_abrir(const char *filename)
        char name[255];
        char tipo;
        FILE *fp;
+       int err = 0;
 
        strcpy(name, filename);
        strcat(name, ".dat");
@@ -188,7 +197,7 @@ EMUFS *emufs_abrir(const char *filename)
        /* Trato de determinar el tipo de archivo */
        fp = fopen(name, "r");
        if (fp == NULL) return NULL;
-       fread(&tipo, sizeof(EMUFS_TYPE), 1, fp);
+       fread(&tipo, sizeof(EMUFS_Tipo), 1, fp);
 
        /* Si no es un tipo conocido, sale. */
        if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {
@@ -204,10 +213,9 @@ EMUFS *emufs_abrir(const char *filename)
        }
        efs->tipo = tipo;
        efs->nombre = str_dup(filename);
-
+       
        switch (tipo) {
                case T1:
-                       emufs_tipo1_inicializar(efs);
                        /* Lee cabeceras propias. */
                        if (!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) {
                                free(efs->nombre);
@@ -215,8 +223,16 @@ EMUFS *emufs_abrir(const char *filename)
                                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. */
+                       emufs_tipo2_inicializar(efs);                   
                        break;
                case T3:
                        if ((!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) ||
@@ -227,10 +243,16 @@ EMUFS *emufs_abrir(const char *filename)
                                fclose(fp);
                                return NULL;
                        }                       
+                       /* Asigna punteros a funciones. */                      
                        efs->leer_bloque = emufs_tipo3_leer_bloque;
+                       efs->leer_registro_raw = emufs_tipo3_leer_registro_raw;
                        efs->leer_registro = emufs_tipo3_leer_registro;
                        efs->grabar_registro = emufs_tipo3_grabar_registro;
-                       /*efs->borrar_registro = emufs_tipo3_borrar_registro;*/
+                       efs->borrar_registro = emufs_tipo3_borrar_registro;
+                       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;
        }
 
@@ -256,12 +278,13 @@ int ver_archivo_FS(EMUFS *emu)
        strcat(name_f_block_free,".fsc");
 
        if ( (f_block_free = fopen(name_f_block_free,"r"))==NULL ){
-               printf("no pude abrir el archivo %s\n",name_f_block_free);
+               fprintf(stderr, "no pude abrir el archivo %s\n",name_f_block_free);
                return -1;
        }
+       fprintf(stderr,"BOQUES Y ESPACIO LIBRE\n");
        fread(&reg,sizeof(reg),1,f_block_free);
        while ( !feof(f_block_free) ){
-               fprintf(stderr, " Bloque = %li   Espacio libre = %li\n",reg.n_marker, reg.n_freespace);
+               fprintf(stderr, "Bloque = %li   Espacio libre = %li\n",reg.marker, reg.freespace);
                fread(&reg,sizeof(reg),1,f_block_free);
        }
        
@@ -271,17 +294,15 @@ int ver_archivo_FS(EMUFS *emu)
        fprintf(stderr, "BLOQUES Y REGISTROS\n");
        strcpy(name_f_block_free,emu->nombre);
        strcat(name_f_block_free,".idx");
+       f_block_free = fopen(name_f_block_free, "r");
        {
                EMUFS_IDX r;
-               f_block_free = fopen(name_f_block_free, "r");
-               fread(&r, sizeof(EMUFS_IDX), 1, f_block_free);
                while (!feof(f_block_free)) {
+                       if (fread(&r, sizeof(EMUFS_IDX), 1, f_block_free) != 1) continue;
                        fprintf(stderr, "ID %li en bloque %li\n", r.id_reg, r.location);
-                       fread(&r, sizeof(EMUFS_IDX), 1, f_block_free);
                }
-               fclose(f_block_free);
        }
-       
+       fclose(f_block_free);
        
        return 0;
 }