subo para que me den una mano con el valgrind, el error esta rondando la linea 1025...

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

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 8b66be817658de1a9edde29940a17759e511696a..7de346f20563bdb773a670b72619fc265b7ec9e2 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -75,7 +75,7 @@ static void* emufs_tipo1_leer_bloque(EMUFS*, EMUFS_BLOCK_ID, int*);
 static EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque_fsc(EMUFS*, void*,
                EMUFS_BLOCK_ID, EMUFS_FREE, int*);
 
-CLAVE grabar_ordenado_en_bloque(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, 
+static CLAVE grabar_ordenado_en_bloque(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, 
                void *bloque, int num_bloque, EMUFS_FREE fs, int *err);
 
 /*------------------ Funciones públicas ----------------------*/
@@ -783,6 +783,7 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                        PERR("NO SE PUDO CREAR EL BLOQUE");
                        return -1;
                }
+               memset(bloque, 0, emu->tam_bloque);
                header.id = emufs_idx_get_new_id(emu, err);
                if (*err) {
                        PERR("NO SE PUDO OBTENER UN ID");
@@ -834,6 +835,7 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                } else { /* el registro no entra en el bloque, hay que crear uno nuevo y desparramar */
                        PERR(" COMO NO ENTRA DEBO DIVIDIR LOS REGISTROS EN 2 BLOQUES");
                        new_bloque = (char*)malloc(emu->tam_bloque);
+                       memset(new_bloque,0,emu->tam_bloque);
                        move_size=0;
                        aux = bloque;
                        for(i=0; i<cant_reg/2; i++){ /*copio mitad-1 aca y mitad en el nuevo*/
@@ -848,14 +850,12 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                        memset(aux, 0, emu->tam_bloque - move_size - sizeof(int));
                        menor_clave_bloque_nuevo = emufs_indice_generar_clave(emu->indices, new_bloque+sizeof(EMUFS_TIPO1_REG_HEADER));
                        /* TENGO QUE VER EN CUAL DE LOS DOS BLOQUES METO EL REGISTRO NUEVO */
-                       printf("I = %d\n", i ) ;
                        if ( emufs_indice_es_menor(emu->indices, menor_clave_bloque_nuevo, clave) ){
                                /* GRABO EN EL BLOQUE NUEVO */
                                /*actualizo la cant de registros del bloque original (footer)*/
                                memcpy(bloque+emu->tam_bloque-sizeof(int), &i, sizeof(int));
                                /*actualizo el footer del nuevo bloque*/
                                dif = cant_reg - i; /*por las dudas*/
-                               printf("DIF = %d\n", dif);
                                memcpy(new_bloque+emu->tam_bloque-sizeof(int), &dif, sizeof(int));
                                /*genero un nuevo espacio para un bloque en el archivo y lo cargo en queryla clave ya estaba en query desde antes*/
                                num_bloque = query.num_bloque;
@@ -883,8 +883,10 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                                /*grabo el bloque nuevo*/
                                emufs_tipo1_grabar_bloque_fsc(emu, new_bloque, query.num_bloque, EMUFS_NOT_FOUND, err);
                                /*grabo el registro en el bloque original*/
-                               printf("espacio libre = %d\n", fs+move_size);
-                               grabar_ordenado_en_bloque(emu,ptr,size,bloque,num_bloque, fs+move_size,err);
+                               grabar_ordenado_en_bloque(emu,ptr,size,bloque,num_bloque,fs+move_size,err);
+                               /*actualizo el arbol con la menor clave del bloque nuevo*/
+                               query.clave = emufs_indice_generar_clave(emu->indices, new_bloque+sizeof(EMUFS_TIPO1_REG_HEADER));
+                               emufs_b_plus_insertar(emu->indices, &query);
                        }
                        if(*err != 0){
                                PERR("NO SE PUDO GRABAR ORDENADO");
@@ -918,6 +920,7 @@ CLAVE grabar_ordenado_en_bloque(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, void
        header.id = emufs_idx_get_new_id(emu, err);
        header.size = size;
        new_bloque = (char*)malloc(emu->tam_bloque);
+       memset(new_bloque, 0, emu->tam_bloque);
        aux = new_bloque;
        for (i=0; i<cant_reg+1; i++){
                bloque += sizeof(EMUFS_TIPO1_REG_HEADER); /*salteo el encabezado del registro*/
@@ -975,6 +978,7 @@ int emufs_tipo1_eliminar_ordenado(EMUFS *emu, CLAVE clave, int *err)
        query.clave = clave;
        /*mando a buscar el bloque donde esta la clave que quiero eliminar*/
        result = emufs_b_plus_get_bloque(emu->indices, &query, 0);
+       printf("el bloque es: %d\n", query.num_bloque);
        if ( result == 1 ){
                PERR("SE PRODUJO UN ERROR EN EL ARBOL");
                return -1;
@@ -1003,9 +1007,11 @@ int emufs_tipo1_eliminar_ordenado(EMUFS *emu, CLAVE clave, int *err)
        while ( iter < emu->tam_bloque ){
                memcpy(&tam_reg, aux+sizeof(EMUFS_REG_ID), sizeof(EMUFS_REG_SIZE));
                clave_ajena = emufs_indice_generar_clave(emu->indices, aux+sizeof(EMUFS_TIPO1_REG_HEADER));
+               printf("lei clave: %d\n", clave_ajena);
                if ( emufs_indice_es_igual(emu->indices, clave, clave_ajena) ){
                        /*tenngo que borrar este registro*/
                        /*limpio el espacio que ocupaba*/
+                       PERR("LA ENCONTRE");
                        memset(aux, 0, tam_reg+sizeof(EMUFS_TIPO1_REG_HEADER));
                        /*hay que reacomodar todo*/
                        /*me posiciono en el reg siguiente*/
@@ -1015,9 +1021,8 @@ int emufs_tipo1_eliminar_ordenado(EMUFS *emu, CLAVE clave, int *err)
                iter += tam_reg+sizeof(EMUFS_TIPO1_REG_HEADER);
                aux += iter;
        }
-       
        /*reacomodo el bloque */
-       memcpy(bloque+iter-tam_reg-sizeof(EMUFS_TIPO1_REG_HEADER), aux+iter, emu->tam_bloque-iter-sizeof(int)); 
+       memcpy(aux, aux+tam_reg+sizeof(EMUFS_TIPO1_REG_HEADER), emu->tam_bloque-iter-sizeof(int)); 
        /*le vuelvo a copiar la cantidad de registros*/
        cant_reg--;
        memcpy(bloque+emu->tam_bloque-sizeof(int), &cant_reg, sizeof(int));