Una bobada, pongo un FIXME para acordarnos de limpiar algo que mejor no tocar ahora...

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

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index 224c09665878699ddc50191c2b374f2b00a2a8d2..c287cb4295ddc3c839c82beed50b45492ad202c0 100644 (file)
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -144,7 +144,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
        FILE *file;
        char name_f[255];
        char* bloque;
-       int cant_bloques, resto, i;
+       int cant_bloques, resto, i=0;
        
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");
@@ -153,7 +153,6 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
        resto = emu->tam_bloque - sizeof(EMUFS_REG_ID);
        /* me devuelve el ID del bloque donde quepa un registro y el espacio libre en "fs"*/
        num_bloque = emufs_fsc_buscar_lugar(emu, emu->tam_reg+sizeof(EMUFS_REG_ID), &fs);
-       printf("encontre lugar en %d\n", num_bloque);
        /*si no hay bloques con suficiente espacio creo un bloque nuevo */
        if (num_bloque == -1) {
                if ( (file = fopen(name_f,"a+"))==NULL ) return -1; /*ERROR*/
@@ -191,7 +190,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                                if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
                                        free(bloque);
                                        return -1;
-                                       }
+                               }
                        }
 
                        /* grabo el nuevo registro en el archivo de espacios libres */
@@ -205,6 +204,8 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                }
                fclose(file);
        } else {
+               /*tengo que buscar un ID valido para el nuevo registro*/
+               ID_aux = emufs_idx_get_new_id(emu, err);
                for (i=0; i<cant_bloques; i++){
                        /*cargo el bloque en "bloque"*/
                        if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque+i, err))) {
@@ -214,14 +215,11 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                        }
                        /*El error puede haberse producido porque la funcion leer_bloque devolvio -1, el cual es un bloque invalido*/
                        /*insertar el registro en el bloque*/
-                       /*tengo que buscar un ID valido para el nuevo registro*/
-                       ID_aux = emufs_idx_get_new_id(emu, err);
                        /*grabo el id en el bloque*/
                        /*veo el espacio libre que queda*/ 
                        fs = emufs_fsc_get_fs(emu, num_bloque+i);
-                       printf("el espacio libre del bloque %d es: %d\n", num_bloque+i, fs);
                        if (emu->tam_bloque-sizeof(EMUFS_REG_ID) < emu->tam_reg)
-                               memcpy(bloque+sizeof(EMUFS_REG_ID),&ID_aux,sizeof(EMUFS_REG_ID));
+                               memcpy(bloque,&ID_aux,sizeof(EMUFS_REG_ID));
                        else
                                memcpy(bloque+emu->tam_bloque-fs,&ID_aux,sizeof(EMUFS_REG_ID));
                        /*grabo el registro en el bloque*/
@@ -247,29 +245,21 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                                        if (bloque != NULL) free(bloque);
                                        return -1;
                                }
-                               /*agrego al idx*/
-                               if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
-                                               if (bloque != NULL) free(bloque);
-                                               return -1;
-                                       }
                        } else {        
                                if ( cant_bloques-1 == i )
                                        resto = emu->tam_reg - i*(emu->tam_bloque - sizeof(EMUFS_REG_ID));
-                               fs = emufs_fsc_get_fs(emu,num_bloque+i);
                                if ( emufs_fsc_agregar(emu, num_bloque+i, fs-resto) !=0 ){
                                        fclose(file);
                                        if (bloque != NULL) free(bloque);
                                        return -1;
                                }
-                               /*actualizo el archivo de bloques y registros*/
-                               if (i == 0) {
-                                       /* Tengo que agregar el primer bloque en IDX */
-                                       if( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
-                                               if (bloque != NULL) free(bloque);
-                                               return -1;
-                                       }
+                       }
+                       if ( i == 0 ){
+                               if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
+                                       if (bloque != NULL) free(bloque);
+                                       return -1;
                                }
-                       }                               
+                       }
                }
        }
        free(bloque);
@@ -345,10 +335,10 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
        /*actualizo archivo .fsc*/
        if ( emu->tam_bloque < emu->tam_reg ) {
                for (i=0; i<emu->tam_reg/(emu->tam_bloque-sizeof(EMUFS_REG_ID))+1; i++)
-                       if ( emufs_fsc_actualizar(emu, num_bloque+i, emu->tam_bloque) != 0 ) return -1;
+                       if ( emufs_fsc_agregar(emu, num_bloque+i, emu->tam_bloque) != 0 ) return -1;
        } else { 
                fs = emufs_fsc_get_fs(emu, num_bloque);
-               if ( emufs_fsc_actualizar(emu, num_bloque, fs + emu->tam_reg + sizeof(EMUFS_REG_ID)) != 0 ) return -1;
+               if ( emufs_fsc_agregar(emu, num_bloque, fs + emu->tam_reg + sizeof(EMUFS_REG_ID)) != 0 ) return -1;
        }
        /*actualizo archivo .did*/
        if ( emufs_did_agregar(emu, ID) != 0 ) return -1;
@@ -367,6 +357,7 @@ EMUFS_Estadisticas emufs_tipo3_leer_estadisticas(EMUFS *emu)
        EMUFS_REG_ID *tmp;
        char name_f[255];
 
+       memset(&stats,0,sizeof(EMUFS_Estadisticas));
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");
        if ( (f = fopen(name_f,"r")) == NULL){
@@ -374,16 +365,19 @@ EMUFS_Estadisticas emufs_tipo3_leer_estadisticas(EMUFS *emu)
                        return stats;   
        }
        
-       /* No hace falta el fseek ¿? */
        fseek(f,0,SEEK_END);
        stats.tam_archivo_bytes = ftell(f);
-       stats.cant_bloques = ( ftell(f) - sizeof(EMUFS_Tipo) - sizeof(EMUFS_BLOCK_SIZE) - sizeof(EMUFS_REG_SIZE) )/ emu->tam_bloque;
+       stats.cant_bloques =(stats.tam_archivo_bytes-sizeof(EMUFS_Tipo)-sizeof(EMUFS_BLOCK_SIZE)-sizeof(EMUFS_REG_SIZE))/
+                                                emu->tam_bloque;
        tmp = emufs_idx_get(emu, &stats.tam_archivo);
        if (tmp) free(tmp);
+               stats.info_control=stats.tam_archivo*sizeof(EMUFS_REG_ID)+sizeof(EMUFS_Tipo)+
+                                               sizeof(EMUFS_BLOCK_SIZE)+sizeof(EMUFS_REG_SIZE);
+       /* Obtengo las stats de FSC */
        stats.total_fs = emufs_fsc_get_total_fs(emu);
-       /*verificar el segentado*/
-       stats.info_control = stats.tam_archivo*sizeof(EMUFS_REG_ID) + sizeof(EMUFS_Tipo) + sizeof(EMUFS_BLOCK_SIZE) + sizeof(EMUFS_REG_SIZE);
-       stats.media_fs = stats.total_fs/stats.cant_bloques;
+       stats.media_fs = emufs_fsc_get_media_fs(emu);
+       emufs_fsc_get_max_min_fs(emu,&stats.min_fs,&stats.max_fs);
+
        fclose(f);
        return stats;   
 }
@@ -406,7 +400,7 @@ void* emufs_tipo3_leer_registro_raw(EMUFS *emu, EMUFS_REG_ID ID, EMUFS_REG_SIZE
        if (emu->tam_reg < emu->tam_bloque) {
                /* Aca estoy en el caso de que 1 registro entra en 1 solo bloque */
                block = emufs_idx_buscar_registro(emu,ID);
-               if ( block == EMUFS_NOT_FOUND ){
+               if ( block == EMUFS_NOT_FOUND ) {
                        return NULL;
                }
                if ((bloque = emufs_tipo3_leer_bloque(emu, block, &err)) == NULL) {
@@ -437,13 +431,14 @@ void* emufs_tipo3_leer_registro_raw(EMUFS *emu, EMUFS_REG_ID ID, EMUFS_REG_SIZE
                if ( block == EMUFS_NOT_FOUND ){
                        return NULL;
                }
-               cant_bloques = emu->tam_reg / emu->tam_bloque + 1;
+               cant_bloques = emu->tam_reg / (emu->tam_bloque - sizeof(EMUFS_REG_ID))+1;
                *size = emu->tam_bloque*cant_bloques + cant_bloques*2 - sizeof(EMUFS_REG_ID)*(cant_bloques-1);
                bloque = (char *)malloc(*size);
                cur = bloque;
                *pos = 0; 
 
                /* El bloque 0 va completo */
+               err = 0;
                if ((tmp = emufs_tipo3_leer_bloque(emu, block, &err)) == NULL) {
                        /* Oops! ... un bloque no existe, todo mal! */
                        free(bloque);
@@ -457,6 +452,7 @@ void* emufs_tipo3_leer_registro_raw(EMUFS *emu, EMUFS_REG_ID ID, EMUFS_REG_SIZE
                
                /* En resto de los bloques no pongo el ID porque ya esta en el primero */
                for(i=1; i<cant_bloques; i++) {
+                       err = 0;
                        if ((tmp = emufs_tipo3_leer_bloque(emu, block+i, &err)) == NULL) {
                                /* Oops! ... un bloque no existe, todo mal! */
                                free(bloque);