bugfix.

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

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index b96482399e55ae50cfa6d4266558fdeb28148f5f..feb000fde4df60d3b6896f21f090040005c95af8 100644 (file)
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -36,6 +36,11 @@
  */
 
 #include "tipo3.h"
  */
 
 #include "tipo3.h"

+#include "error.h"
+#include <unistd.h>
+#include <sys/types.h>
+#include <stdio.h>
+#include <string.h>

 
 /** Leo un registro del archivo, devuelve cero si no lo encuentra.**/
 void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
 
 /** Leo un registro del archivo, devuelve cero si no lo encuentra.**/
 void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,


@@ -60,10 +65,8 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
        
        registro = (char*) malloc(emu->tam_reg);
        if (registro == NULL) {
        
        registro = (char*) malloc(emu->tam_reg);
        if (registro == NULL) {

-               /* TODO Manejo de errores */
-               free(bloque);

                PERR("No hay memoria");
                PERR("No hay memoria");

-               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+               *err = EMUFS_ERROR_OUT_OF_MEMORY;

                return NULL;
        }
 
                return NULL;
        }
 


@@ -73,6 +76,7 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
                        /* TODO Manejo de errores, queda en el codigo de error lo que devolvio
                         * emufs_tipo3_leer_bloque() */
                        PERR("no se pudo leer el bloque");
                        /* TODO Manejo de errores, queda en el codigo de error lo que devolvio
                         * emufs_tipo3_leer_bloque() */
                        PERR("no se pudo leer el bloque");

+                       free(registro);

                        return NULL; /*No se pudo leer el bloque*/
                }
                ID_aux = -1;
                        return NULL; /*No se pudo leer el bloque*/
                }
                ID_aux = -1;


@@ -96,7 +100,7 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
                }
                free(bloque);
        }
                }
                free(bloque);
        }

-       
+

        return registro;
 }
 
        return registro;
 }
 


@@ -112,8 +116,8 @@ void* emufs_tipo3_leer_bloque(EMUFS *emu, EMUFS_BLOCK_ID ID, int* err)
        
        if ((file = fopen(name_f, "r")) == NULL) {
                PERR("No se pudo abrir el archivo de datos");
        
        if ((file = fopen(name_f, "r")) == NULL) {
                PERR("No se pudo abrir el archivo de datos");

-               *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
-               return NULL; /* FIXME ERROR */
+               *err = EMUFS_ERROR_CANT_OPEN_FILE;
+               return NULL;

        }
        fseek(file,sizeof(EMUFS_Tipo)+sizeof(EMUFS_BLOCK_SIZE)+sizeof(EMUFS_REG_SIZE),SEEK_SET);
        /*FIXME: verificar que no se pase de fin de archivo*/
        }
        fseek(file,sizeof(EMUFS_Tipo)+sizeof(EMUFS_BLOCK_SIZE)+sizeof(EMUFS_REG_SIZE),SEEK_SET);
        /*FIXME: verificar que no se pase de fin de archivo*/


@@ -125,16 +129,15 @@ void* emufs_tipo3_leer_bloque(EMUFS *emu, EMUFS_BLOCK_ID ID, int* err)
        
        block = (char*) malloc(emu->tam_bloque);
        if (block == NULL) {
        
        block = (char*) malloc(emu->tam_bloque);
        if (block == NULL) {

-               /* TODO Manejo de errores */

                PERR("No hay memoria");
                PERR("No hay memoria");

-               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+               *err = EMUFS_ERROR_OUT_OF_MEMORY;

                return NULL;
        }
        if (fread(block, emu->tam_bloque, 1, file) != 1) {
                /* TODO Manejo de errores */
                free(block);
                PERR("Error al leer bloque");
                return NULL;
        }
        if (fread(block, emu->tam_bloque, 1, file) != 1) {
                /* TODO Manejo de errores */
                free(block);
                PERR("Error al leer bloque");

-               *err = 3; /* EMUFS_ERROR_FILE_READ */
+               *err = EMUFS_ERROR_FILE_READ;

                return NULL;
        }
 
                return NULL;
        }
 


@@ -150,7 +153,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
        EMUFS_BLOCK_SIZE cant;
        FILE *file;
        char name_f[255];
        EMUFS_BLOCK_SIZE cant;
        FILE *file;
        char name_f[255];

-       char* bloque;
+       char* bloque = NULL;

        int cant_bloques, resto, i=0, lugar;
        
        strcpy(name_f,emu->nombre);
        int cant_bloques, resto, i=0, lugar;
        
        strcpy(name_f,emu->nombre);


@@ -163,7 +166,6 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                lugar = emu->tam_bloque;
        /* me devuelve el ID del bloque donde quepa un registro y el espacio libre en "fs"*/
        num_bloque = emufs_fsc_buscar_lugar(emu, lugar, &fs);
                lugar = emu->tam_bloque;
        /* me devuelve el ID del bloque donde quepa un registro y el espacio libre en "fs"*/
        num_bloque = emufs_fsc_buscar_lugar(emu, lugar, &fs);

-       printf("lugar encontrado 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*/
        /*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*/


@@ -222,7 +224,6 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                        if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque+i, err))) {
                                /* TODO Manejo de errores */
                                PERR("no se pudo leer el bloque");
                        if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque+i, err))) {
                                /* TODO Manejo de errores */
                                PERR("no se pudo leer el bloque");

-                               printf("bloque: %d  i= %d\n", num_bloque, i);

                                return -1;
                        }
                        /*El error puede haberse producido porque la funcion leer_bloque devolvio -1, el cual es un bloque invalido*/
                                return -1;
                        }
                        /*El error puede haberse producido porque la funcion leer_bloque devolvio -1, el cual es un bloque invalido*/


@@ -246,6 +247,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                        /*grabo el bloque en el archivo*/
                        if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque+i) != 0) {
                                PERR("error al grabar bloque");
                        /*grabo el bloque en el archivo*/
                        if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque+i) != 0) {
                                PERR("error al grabar bloque");

+                               if (bloque) free(bloque);

                                return -1; /* se produjo un error */    
                        }
                        
                                return -1; /* se produjo un error */    
                        }
                        


@@ -254,7 +256,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                                resto = emu->tam_reg;
                                if ( emufs_fsc_agregar(emu, num_bloque, fs - resto - sizeof(EMUFS_REG_ID) ) != 0 ) {
                                        fclose(file);
                                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);
+                                       if (bloque) free(bloque);

                                        return -1;
                                }
                        } else {        
                                        return -1;
                                }
                        } else {        


@@ -262,19 +264,19 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                                        resto = emu->tam_reg - i*(emu->tam_bloque - sizeof(EMUFS_REG_ID));
                                if ( emufs_fsc_agregar(emu, num_bloque+i, fs-resto) !=0 ){
                                        fclose(file);
                                        resto = emu->tam_reg - i*(emu->tam_bloque - sizeof(EMUFS_REG_ID));
                                if ( emufs_fsc_agregar(emu, num_bloque+i, fs-resto) !=0 ){
                                        fclose(file);

-                                       if (bloque != NULL) free(bloque);
+                                       if (bloque) free(bloque);

                                        return -1;
                                }
                        }
                        if ( i == 0 ){
                                if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
                                        return -1;
                                }
                        }
                        if ( i == 0 ){
                                if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){

-                                       if (bloque != NULL) free(bloque);
+                                       if (bloque) free(bloque);

                                        return -1;
                                }
                        }
                }
        }
                                        return -1;
                                }
                        }
                }
        }

-       free(bloque);
+       if (bloque) free(bloque);

        return ID_aux;
 }
 
        return ID_aux;
 }
 


@@ -347,16 +349,32 @@ 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++)
        /*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_agregar(emu, num_bloque+i, emu->tam_bloque) != 0 ) return -1;
+                       if (emufs_fsc_agregar(emu, num_bloque+i, emu->tam_bloque)) {
+                               PERR("no se pudo agregar fsc"); 
+                               free(bloque);
+                               return -1;
+                       }

        } else { 
                fs = emufs_fsc_get_fs(emu, num_bloque);
        } else { 
                fs = emufs_fsc_get_fs(emu, num_bloque);

-               if ( emufs_fsc_agregar(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))) {
+                       PERR("no se pudo agregar fsc"); 
+                       free(bloque);
+                       return -1;
+               }

        }
        /*actualizo archivo .did*/
        }
        /*actualizo archivo .did*/

-       if ( emufs_did_agregar(emu, ID) != 0 ) return -1;
+       if (emufs_did_agregar(emu, ID)) {
+               PERR("no se pudo agregar did"); 
+               free(bloque);
+               return -1;
+       }

                
        /*actualizo archivo .idx*/
                
        /*actualizo archivo .idx*/

-       if ( emufs_idx_borrar(emu, ID) != 0 ) return -1; 
+       if (emufs_idx_borrar(emu, ID)) {
+               PERR("no se pudo agregar idx"); 
+               free(bloque);
+               return -1;
+       }

 
        free(bloque);
        return 0;
 
        free(bloque);
        return 0;


@@ -444,7 +462,7 @@ void* emufs_tipo3_leer_registro_raw(EMUFS *emu, EMUFS_REG_ID ID, EMUFS_REG_SIZE
                        return NULL;
                }
                cant_bloques = emu->tam_reg / (emu->tam_bloque - sizeof(EMUFS_REG_ID))+1;
                        return NULL;
                }
                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);
+               *size = emu->tam_bloque*cant_bloques /*+ cant_bloques*2*/ - sizeof(EMUFS_REG_ID)*(cant_bloques-1);

                bloque = (char *)malloc(*size);
                cur = bloque;
                *pos = 0; 
                bloque = (char *)malloc(*size);
                cur = bloque;
                *pos = 0; 


@@ -458,8 +476,8 @@ void* emufs_tipo3_leer_registro_raw(EMUFS *emu, EMUFS_REG_ID ID, EMUFS_REG_SIZE
                }
                memcpy(cur, tmp, emu->tam_bloque);
                cur += emu->tam_bloque;
                }
                memcpy(cur, tmp, emu->tam_bloque);
                cur += emu->tam_bloque;

-               memcpy(cur, "<>", 2);
-               cur += 2;
+/*             memcpy(cur, "<>", 2);
+               cur += 2;*/

                free(tmp);
                
                /* En resto de los bloques no pongo el ID porque ya esta en el primero */
                free(tmp);
                
                /* En resto de los bloques no pongo el ID porque ya esta en el primero */


@@ -472,8 +490,8 @@ void* emufs_tipo3_leer_registro_raw(EMUFS *emu, EMUFS_REG_ID ID, EMUFS_REG_SIZE
                        }
                        memcpy(cur, tmp+sizeof(EMUFS_REG_ID), emu->tam_bloque-sizeof(EMUFS_REG_ID));
                        cur += emu->tam_bloque - sizeof(EMUFS_REG_ID);
                        }
                        memcpy(cur, tmp+sizeof(EMUFS_REG_ID), emu->tam_bloque-sizeof(EMUFS_REG_ID));
                        cur += emu->tam_bloque - sizeof(EMUFS_REG_ID);

-                       memcpy(cur, "<>", 2);
-                       cur += 2;
+/*                     memcpy(cur, "<>", 2);
+                       cur += 2;*/

                        free(tmp);
                }
                (*cur) = '\0';
                        free(tmp);
                }
                (*cur) = '\0';


@@ -486,22 +504,17 @@ void emufs_tipo3_compactar(EMUFS *emu)
        EMUFS_REG_ID *tmp, max_id;
        EMUFS_BLOCK_ID block_id;
        EMUFS_REG_SIZE size;
        EMUFS_REG_ID *tmp, max_id;
        EMUFS_BLOCK_ID block_id;
        EMUFS_REG_SIZE size;

-       EMUFS_Estadisticas s;
+       EMUFS_FREE fs;

        char name[255];
        char *reg;
        char name[255];
        char *reg;

-       int err=0, ID_aux, cant_bloques, i, bloques_vacios=0, fs;
+       int err=0, ID_aux, i;
+       
+       strcpy(name, emu->nombre);
+       strcat(name, ".dat");

        
        /* si el bloque es mas chico que el registro no hace falta compactar */
        /*if( emu->tam_reg-sizeof(EMUFS_REG_ID) > emu->tam_bloque ) return;     */
        
        /* si el bloque es mas chico que el registro no hace falta compactar */
        /*if( emu->tam_reg-sizeof(EMUFS_REG_ID) > emu->tam_bloque ) return;     */

-               
-       s = emufs_tipo3_leer_estadisticas(emu);
-       cant_bloques = s.cant_bloques;

 
 

-       /* si solo hay un bloque no hace falta compactar */
-       if ( cant_bloques == 0 ){
-               return;
-       }
-       

        tmp = emufs_idx_get(emu, &max_id);
        if (tmp) free(tmp);
        for( i=0; i<=max_id; i++){
        tmp = emufs_idx_get(emu, &max_id);
        if (tmp) free(tmp);
        for( i=0; i<=max_id; i++){


@@ -514,20 +527,35 @@ void emufs_tipo3_compactar(EMUFS *emu)
                }
                emufs_tipo3_borrar_registro(emu, i);
                ID_aux = emufs_tipo3_grabar_registro(emu, reg, emu->tam_reg, &err);
                }
                emufs_tipo3_borrar_registro(emu, i);
                ID_aux = emufs_tipo3_grabar_registro(emu, reg, emu->tam_reg, &err);

-               i++;
+               free(reg);

        }
        /*tengo que truncar el archivo*/
        /*bloques_vacios = emufs_fsc_get_cant_bloques_vacios(emu)-1;
        */
        block_id = emufs_fsc_buscar_lugar(emu, emu->tam_bloque, &fs);
        size = sizeof(EMUFS_Tipo)+sizeof(EMUFS_REG_SIZE)+sizeof(EMUFS_BLOCK_SIZE)+block_id*emu->tam_bloque;
        }
        /*tengo que truncar el archivo*/
        /*bloques_vacios = emufs_fsc_get_cant_bloques_vacios(emu)-1;
        */
        block_id = emufs_fsc_buscar_lugar(emu, emu->tam_bloque, &fs);
        size = sizeof(EMUFS_Tipo)+sizeof(EMUFS_REG_SIZE)+sizeof(EMUFS_BLOCK_SIZE)+block_id*emu->tam_bloque;

-       printf("cantidad de bloques: %d\nturnco hasta %d\n",cant_bloques, block_id);
-       printf("bytes a truncar: %d\n", size);

        if (truncate(name, size)!=0)
                PERR("NO TRUNQUE NADA");
        /*hay que truncar el fsc!!!*/
        if (truncate(name, size)!=0)
                PERR("NO TRUNQUE NADA");
        /*hay que truncar el fsc!!!*/

-       printf("trunque el fsc hasta el bloque %d\n", block_id);
+       if(emu->tam_bloque<emu->tam_reg-sizeof(EMUFS_REG_ID)) block_id = block_id/2;

        if (emufs_fsc_truncate(emu, block_id)!= 0)
                PERR("NO TURNQUE EL FSC");
        if (emufs_fsc_truncate(emu, block_id)!= 0)
                PERR("NO TURNQUE EL FSC");

-       free(reg);
+}
+
+void emufs_tipo3_leer_bloque_raw(EMUFS *efs, EMUFS_BLOCK_ID id, char **actual, char **anterior, char **siguiente,
+                                                                EMUFS_BLOCK_SIZE *size1, EMUFS_BLOCK_SIZE *size2, EMUFS_BLOCK_SIZE *size3)
+{
+       int err;
+       (*actual) = emufs_tipo3_leer_bloque(efs, id, &err);
+       (*anterior) = emufs_tipo3_leer_bloque(efs, id-1, &err);
+       (*siguiente) = emufs_tipo3_leer_bloque(efs, id+1, &err);
+       if (!(*anterior)) {
+               (*anterior) = (char *)malloc(efs->tam_bloque);
+               memset(*anterior, 0, efs->tam_bloque);          
+       }       
+       if (!(*siguiente)) {
+               (*siguiente) = (char *)malloc(efs->tam_bloque);
+               memset(*siguiente, 0, efs->tam_bloque);         
+       }
+       (*size1) = (*size2) = (*size3) = efs->tam_bloque;

 }
 }