* Bugfix en tipo1 leer_raw que se estaba usando una variable

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

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index fd59a683ae197ed1f5cf47885c135a7ed287f4c5..6375a7efd5001e8ddab4127a2c6389cf993237c8 100644 (file)
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -48,7 +48,7 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
        EMUFS_BLOCK_SIZE iterador = 0;
        int cant_bloques = 0, resto, i, copiado=0;
        
        EMUFS_BLOCK_SIZE iterador = 0;
        int cant_bloques = 0, resto, i, copiado=0;
        

-       cant_bloques = emu->tam_reg / emu->tam_bloque + 1;
+       cant_bloques = (emu->tam_reg / (emu->tam_bloque-sizeof(EMUFS_REG_ID))) + 1;

        
        /*si existe, lo busco en el archivo de bloques*/
        block = emufs_idx_buscar_registro(emu,ID); /*me devuelve el nro de bloque al que pertenece el registro*/
        
        /*si existe, lo busco en el archivo de bloques*/
        block = emufs_idx_buscar_registro(emu,ID); /*me devuelve el nro de bloque al que pertenece el registro*/


@@ -149,16 +149,16 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");
        
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");
        

-       cant_bloques = emu->tam_reg / emu->tam_bloque + 1;
+       cant_bloques = (emu->tam_reg / (emu->tam_bloque-sizeof(EMUFS_REG_ID))) + 1;

        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);
        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);

+       fprintf(stderr, "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*/
                /*tengo que buscar un ID valido para el nuevo registro*/
                ID_aux = emufs_idx_get_new_id(emu, err);
        /*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*/
                /*tengo que buscar un ID valido para el nuevo registro*/
                ID_aux = emufs_idx_get_new_id(emu, err);

-

                /* El free esta al final de la funcion! */
                bloque = (char*)malloc(emu->tam_bloque);
                for (i=0; i<cant_bloques; i++) {
                /* El free esta al final de la funcion! */
                bloque = (char*)malloc(emu->tam_bloque);
                for (i=0; i<cant_bloques; i++) {


@@ -191,46 +191,86 @@ 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;
                                if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
                                        free(bloque);
                                        return -1;

-                               }
+                                       }

                        }
                        }

+

                        /* grabo el nuevo registro en el archivo de espacios libres */
                        /* grabo el nuevo registro en el archivo de espacios libres */

-                       if ( emufs_fsc_agregar(emu, num_bloque, emu->tam_bloque - resto - sizeof(EMUFS_REG_ID)) != 0 ) {
+                       if ( emu->tam_bloque > emu->tam_reg ) resto = emu->tam_reg;
+                       if ( emufs_fsc_agregar(emu, num_bloque+i, emu->tam_bloque - resto - sizeof(EMUFS_REG_ID)) != 0 ) {

                                fclose(file);
                                free(bloque);
                                return -1;
                        }
                                fclose(file);
                                free(bloque);
                                return -1;
                        }

+                               

                }
                fclose(file);
        } else {
                }
                fclose(file);
        } else {

-               /*cargo el bloque en "bloque"*/
-               if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque, err))) {
-                       /* TODO Manejo de errores */
-                       PERR("no se pudo leer el bloque");
-                       return -1;
-               }
-               /*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*/
-               memcpy(bloque+emu->tam_bloque-fs,&ID_aux,sizeof(EMUFS_REG_ID));
-               /*grabo el registro en el bloque*/
-               memcpy(bloque+emu->tam_bloque-fs+sizeof(EMUFS_REG_ID),ptr,emu->tam_reg);
-               if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) != 0) {
-                       PERR("error al grabar bloque");
-                       return -1; /* se produjo un error */    
-               }
-               /*actualizo el archivo de espacios libres*/
-               if ( emufs_fsc_actualizar(emu, num_bloque, fs - emu->tam_reg - sizeof(EMUFS_REG_ID)) != 0 ){
-                       free(bloque);
-                       return -1;
-               }
-               /*actualizo el archivo de bloques y registros*/
-               if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
-                       free(bloque);
-                       return -1;
+               for (i=0; i<cant_bloques; i++){
+                       /*cargo el bloque en "bloque"*/
+                       if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque+i, err))) {
+                               /* TODO Manejo de errores */
+                               PERR("no se pudo leer el bloque");
+                               return -1;
+                       }
+                       /*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);
+                       fprintf(stderr, "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));
+                       else
+                               memcpy(bloque+emu->tam_bloque-fs,&ID_aux,sizeof(EMUFS_REG_ID));
+                       /*grabo el registro en el bloque*/
+                       if ( cant_bloques == 1 ){
+                               memcpy(bloque+emu->tam_bloque-fs+sizeof(EMUFS_REG_ID),ptr,emu->tam_reg);
+                       } else {
+                               if ( cant_bloques-1 == i )
+                                       resto = emu->tam_reg - i*(emu->tam_bloque - sizeof(EMUFS_REG_ID));
+                               memcpy(bloque+sizeof(EMUFS_REG_ID),((char*)ptr)+i*(emu->tam_bloque-sizeof(EMUFS_REG_ID)),resto);
+                       }                               
+                       
+                       /*grabo el bloque en el archivo*/
+                       if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque+i) != 0) {
+                               PERR("error al grabar bloque");
+                               return -1; /* se produjo un error */    
+                       }
+                       
+                       /*actualizo el archivo de espacios libres*/
+                       if ( emu->tam_bloque-sizeof(EMUFS_REG_ID) > emu->tam_reg ){
+                               resto = emu->tam_reg;
+                               if ( emufs_fsc_agregar(emu, num_bloque, fs - resto - sizeof(EMUFS_REG_ID) ) != 0 ) {
+                                       fclose(file);
+                                       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;
+                                       }
+                               }
+                       }                               

                }
                }

-

        }
        free(bloque);
        return ID_aux;
        }
        free(bloque);
        return ID_aux;


@@ -264,7 +304,7 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
        EMUFS_REG_ID ID_aux;
        EMUFS_FREE fs;
        char *bloque;
        EMUFS_REG_ID ID_aux;
        EMUFS_FREE fs;
        char *bloque;

-       int err = 0;
+       int err = 0, i;

 
        num_bloque = emufs_idx_buscar_registro(emu, ID);
        if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque, &err))) {
 
        num_bloque = emufs_idx_buscar_registro(emu, ID);
        if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque, &err))) {


@@ -294,6 +334,8 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
        }
 
        /*grabo el bloque en el archivo*/       
        }
 
        /*grabo el bloque en el archivo*/       

+       if ( emu->tam_bloque < emu->tam_reg ) 
+               memset(bloque, 0, emu->tam_bloque);

        if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) == -1 ){
                free(bloque);
                PERR("No se pudo grabar el bloque"); 
        if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) == -1 ){
                free(bloque);
                PERR("No se pudo grabar el bloque"); 


@@ -301,9 +343,13 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
        }
 
        /*actualizo archivo .fsc*/
        }
 
        /*actualizo archivo .fsc*/

-       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 ( 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;
+       } 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;
+       }

        /*actualizo archivo .did*/
        if ( emufs_did_agregar(emu, ID) != 0 ) return -1;
                
        /*actualizo archivo .did*/
        if ( emufs_did_agregar(emu, ID) != 0 ) return -1;