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 1bc3d2b63c2993370d665957933b072670ee433f..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*);
 
 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 ----------------------*/
                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;
                }
                        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");
                header.id = emufs_idx_get_new_id(emu, err);
                if (*err) {
                        PERR("NO SE PUDO OBTENER UN ID");


@@ -834,47 +835,58 @@ 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);
                } 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;
                        move_size=0;
                        aux = bloque;

-                       for(i=0; i<cant_reg/2; i++){ /*copio mitad aca y mitad en el nuevo*/
+                       for(i=0; i<cant_reg/2; i++){ /*copio mitad-1 aca y mitad en el nuevo*/

                                /*avanzo*/
                                memcpy(&tam_reg, aux+sizeof(EMUFS_REG_ID), sizeof(EMUFS_REG_SIZE));
                                aux += sizeof(EMUFS_TIPO1_REG_HEADER)+tam_reg;
                                move_size += sizeof(EMUFS_TIPO1_REG_HEADER)+tam_reg;
                                /*avanzo*/
                                memcpy(&tam_reg, aux+sizeof(EMUFS_REG_ID), sizeof(EMUFS_REG_SIZE));
                                aux += sizeof(EMUFS_TIPO1_REG_HEADER)+tam_reg;
                                move_size += sizeof(EMUFS_TIPO1_REG_HEADER)+tam_reg;

-                       }
-                       PERR("ME MOVI LA MITAD DE LOS REGISTROS");
+                       }/*apunto al reg mitad_mas_uno para copiar todo a otro bloque*/

                        /*copio el resto del bloque al nuevo bloque*/      /*footer ¿?*/
                        /*copio el resto del bloque al nuevo bloque*/      /*footer ¿?*/

-                       memcpy(new_bloque, aux, emu->tam_bloque-move_size/*-sizeof(int)*/);
+                       memcpy(new_bloque, aux, emu->tam_bloque-move_size-sizeof(int));

                        /*borro lo que sobra en el bloque original, guardando el footer*/
                        memset(aux, 0, emu->tam_bloque - move_size - sizeof(int));
                        /*borro lo que sobra en el bloque original, guardando el footer*/
                        memset(aux, 0, emu->tam_bloque - move_size - sizeof(int));

-                       /*actualizo la cant de registros del bloque original (footer)*/
-                       i++; /* 0 a n hay n+1 */
-                       memcpy(bloque+emu->tam_bloque-sizeof(int), &i, sizeof(int));
-                       /*actualizo el footer del nuevo bloque*/
-                       dif = cant_reg - i; /*por las dudas*/
-                       memcpy(new_bloque+emu->tam_bloque-sizeof(int), &dif, 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 */
                        if ( emufs_indice_es_menor(emu->indices, menor_clave_bloque_nuevo, clave) ){
                        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 */
                        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*/
+                               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*/
                                /*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;

                                query.num_bloque = emufs_create_new_block(emu);
                                query.num_bloque = emufs_create_new_block(emu);

-                               printf("numero de bloque nuevo: %d\n", query.num_bloque);

                                /*inserto el nuevo registro en el nuevo bloque y obtengo la clave del menor*/
                                clave = grabar_ordenado_en_bloque(emu,ptr,size,new_bloque,query.num_bloque, emu->tam_bloque-move_size,err);
                                /*actualizo el arbol con la nueva clave*/
                                query.clave = clave;
                                emufs_b_plus_insertar(emu->indices, &query);
                                /*grabo el bloque original*/
                                /*inserto el nuevo registro en el nuevo bloque y obtengo la clave del menor*/
                                clave = grabar_ordenado_en_bloque(emu,ptr,size,new_bloque,query.num_bloque, emu->tam_bloque-move_size,err);
                                /*actualizo el arbol con la nueva clave*/
                                query.clave = clave;
                                emufs_b_plus_insertar(emu->indices, &query);
                                /*grabo el bloque original*/

-                               printf("numero de bloque viejo: %d\n", num_bloque);

                                emufs_tipo1_grabar_bloque_fsc(emu, bloque, num_bloque, EMUFS_NOT_FOUND, err);
                        } else {
                                emufs_tipo1_grabar_bloque_fsc(emu, bloque, num_bloque, EMUFS_NOT_FOUND, err);
                        } else {

-                               /*grabo el bloque nuevo*/
+                               PERR("GRABO EN BLOQUE VIEJO");
+                               /* GRABO EN EL BLOQUE VIEJO */
+                               /*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*/
+                               /*meto el footer en el reg */
+                               memcpy(new_bloque+emu->tam_bloque-sizeof(int), &dif, sizeof(int));
+                               /*guardo el num de bloque original */
+                               num_bloque = query.num_bloque;
+                               /* pido un bloque nuevo para guardar */

                                query.num_bloque = emufs_create_new_block(emu);
                                query.num_bloque = emufs_create_new_block(emu);

-                               printf("numero de bloque nuevo: %d\n", query.num_bloque);
-                               emufs_tipo1_grabar_bloque_fsc(emu, new_bloque, emufs_create_new_block(emu), EMUFS_NOT_FOUND, err);
-                               printf("numero de bloque viejo %d\n", num_bloque);
-                               grabar_ordenado_en_bloque(emu,ptr,size,bloque,query.num_bloque, fs-move_size,err);
-                               
+                               /*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*/
+                               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");
                        }
                        if(*err != 0){
                                PERR("NO SE PUDO GRABAR ORDENADO");


@@ -908,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);
        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*/
        aux = new_bloque;
        for (i=0; i<cant_reg+1; i++){
                bloque += sizeof(EMUFS_TIPO1_REG_HEADER); /*salteo el encabezado del registro*/


@@ -965,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);
        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;
        if ( result == 1 ){
                PERR("SE PRODUJO UN ERROR EN EL ARBOL");
                return -1;


@@ -993,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));
        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*/
                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*/
                        memset(aux, 0, tam_reg+sizeof(EMUFS_TIPO1_REG_HEADER));
                        /*hay que reacomodar todo*/
                        /*me posiciono en el reg siguiente*/


@@ -1005,9 +1021,8 @@ int emufs_tipo1_eliminar_ordenado(EMUFS *emu, CLAVE clave, int *err)
                iter += tam_reg+sizeof(EMUFS_TIPO1_REG_HEADER);
                aux += iter;
        }
                iter += tam_reg+sizeof(EMUFS_TIPO1_REG_HEADER);
                aux += iter;
        }

-       

        /*reacomodo el bloque */
        /*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));
        /*le vuelvo a copiar la cantidad de registros*/
        cant_reg--;
        memcpy(bloque+emu->tam_bloque-sizeof(int), &cant_reg, sizeof(int));