Puto bug que no podia encontrar hace 3 dias, FIXED!!!!!

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

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 0e8f7dc69aced4f60f306216141a8a16eb50dddb..ffca1914cf923b2dc8b48a639a317039866fc43a 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -369,7 +369,7 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
        return header.id;
 }
 
        return header.id;
 }
 

-int emufs_tipo1_borrar_registro(EMUFS* efs, CLAVE k)
+int emufs_tipo1_borrar_registro(EMUFS* efs, CLAVE k, INDICE_DATO dato1)

 {
        char* block; /* bloque leido (en donde está el registro a leer) */
        EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */
 {
        char* block; /* bloque leido (en donde está el registro a leer) */
        EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */


@@ -399,6 +399,7 @@ int emufs_tipo1_borrar_registro(EMUFS* efs, CLAVE k)
                return err;
        }
 
                return err;
        }
 

+       emufs_indice_borrar(efs->indices, k, dato1);

        /* Busco secuencialmente en el bloque el registro a leer */
        offset = 0;
        do {
        /* Busco secuencialmente en el bloque el registro a leer */
        offset = 0;
        do {


@@ -461,10 +462,7 @@ int emufs_tipo1_borrar_registro(EMUFS* efs, CLAVE k)
                                /* si es necesario desplazar */
                                if (offset < offset_reg_end) {
                                        /* muevo la porción de bloque a izquierda */
                                /* si es necesario desplazar */
                                if (offset < offset_reg_end) {
                                        /* muevo la porción de bloque a izquierda */

-                                       /* XXX Este memcpy() puede copiar regiones de memoria que
-                                        * se superponen, si copia de principio a fin y byte a byte
-                                        * no debería haber problema */
-                                       memcpy(block + offset, block + offset_reg_end,
+                                       memmove(block + offset, block + offset_reg_end,

                                                efs->tam_bloque - offset_reg_end);
                                        /* rellena el espacio libre con ceros para la GUI */
                                        memset(block + efs->tam_bloque - offset_reg_end - orig_fs + offset,
                                                efs->tam_bloque - offset_reg_end);
                                        /* rellena el espacio libre con ceros para la GUI */
                                        memset(block + efs->tam_bloque - offset_reg_end - orig_fs + offset,


@@ -632,7 +630,7 @@ EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque_fsc(EMUFS *efs, void *block,
                PERR("Error al obtener tamaño del archivo.");
                return EMUFS_NOT_FOUND;
        }
                PERR("Error al obtener tamaño del archivo.");
                return EMUFS_NOT_FOUND;
        }

-
+       

        /* abre archivo */
        strcpy(name_f,efs->nombre);
        strcat(name_f,".dat");
        /* abre archivo */
        strcpy(name_f,efs->nombre);
        strcat(name_f,".dat");


@@ -698,9 +696,9 @@ EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque_fsc(EMUFS *efs, void *block,
 }
 
 EMUFS_REG_ID emufs_tipo1_modificar_registro(EMUFS* efs, CLAVE k,
 }
 
 EMUFS_REG_ID emufs_tipo1_modificar_registro(EMUFS* efs, CLAVE k,

-               void *data, EMUFS_REG_SIZE size, int* err)
+               void *data, EMUFS_REG_SIZE size, int* err, INDICE_DATO dato)

 {
 {

-       emufs_tipo1_borrar_registro(efs, k);
+       emufs_tipo1_borrar_registro(efs, k, dato);

        return emufs_tipo1_grabar_registro(efs, data, size, err);
 }
 
        return emufs_tipo1_grabar_registro(efs, data, size, err);
 }
 


@@ -755,7 +753,7 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
        EMUFS_REG_SIZE tam_reg, move_size;
        INDEX_DAT query;
        EMUFS_FREE fs;
        EMUFS_REG_SIZE tam_reg, move_size;
        INDEX_DAT query;
        EMUFS_FREE fs;

-       char *bloque, *aux, *new_bloque;
+       char *bloque = 0, *aux, *new_bloque = 0;

        int cant_reg, i, result, dif;
        
        /*le asigno un posible numero de bloque para el caso en que no encuentre donde meterlo*/
        int cant_reg, i, result, dif;
        
        /*le asigno un posible numero de bloque para el caso en que no encuentre donde meterlo*/


@@ -834,9 +832,11 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                        PERR("GRABO ORDENADO");                 
                        grabar_ordenado_en_bloque(emu, ptr, size, bloque, query.num_bloque, fs, err);
                        /*en teoria el nuevo registro no debe cambiar el ancla, por lo cual no actualizo el arbol*/
                        PERR("GRABO ORDENADO");                 
                        grabar_ordenado_en_bloque(emu, ptr, size, bloque, query.num_bloque, fs, err);
                        /*en teoria el nuevo registro no debe cambiar el ancla, por lo cual no actualizo el arbol*/

+                       free(bloque);

                        return 0;
                /*aca va lo que saque*/
                } else { /* el registro no entra en el bloque, hay que crear uno nuevo y desparramar */
                        return 0;
                /*aca va lo que saque*/
                } 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);
                        move_size=0;
                        for(i=0; i<cant_reg/2; i++){ /*copio mitad aca y mitad en el nuevo*/
                        new_bloque = (char*)malloc(emu->tam_bloque);
                        move_size=0;
                        for(i=0; i<cant_reg/2; i++){ /*copio mitad aca y mitad en el nuevo*/


@@ -845,14 +845,18 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                                aux += sizeof(EMUFS_TIPO1_REG_HEADER)+tam_reg;
                                move_size += sizeof(EMUFS_TIPO1_REG_HEADER)+tam_reg;
                        }
                                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");

                        /*copio el resto del bloque al nuevo bloque*/
                        memcpy(new_bloque, aux, emu->tam_bloque-move_size);
                        /*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 (footer)*/
                        memcpy(bloque+emu->tam_bloque-sizeof(int), &i, sizeof(int));
                        /*copio el resto del bloque al nuevo bloque*/
                        memcpy(new_bloque, aux, emu->tam_bloque-move_size);
                        /*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 (footer)*/
                        memcpy(bloque+emu->tam_bloque-sizeof(int), &i, sizeof(int));

+                       PERR("PASE");

                        /*grabo el bloque original*/
                        /*grabo el bloque original*/

+                       printf("voya a grabar en bloque: %d\n", num_bloque);

                        emufs_tipo1_grabar_bloque_fsc(emu, bloque, num_bloque, EMUFS_NOT_FOUND, err);
                        emufs_tipo1_grabar_bloque_fsc(emu, bloque, num_bloque, EMUFS_NOT_FOUND, err);

+                       PERR("GRABE EL BLOQUE 1 de 2");

                        /*actualizo el footer del nuevo bloque*/
                        dif = cant_reg - i; /*por las dudas*/
                        memcpy(new_bloque+emu->tam_bloque, &dif, sizeof(int));
                        /*actualizo el footer del nuevo bloque*/
                        dif = cant_reg - i; /*por las dudas*/
                        memcpy(new_bloque+emu->tam_bloque, &dif, sizeof(int));


@@ -861,6 +865,12 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                        query.num_bloque = emufs_create_new_block(emu);
                        /*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);
                        query.num_bloque = emufs_create_new_block(emu);
                        /*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);

+                       if(*err != 0){
+                               PERR("NO SE PUDO GRABAR ORDENADO");
+                               free(new_bloque);
+                               free(bloque);
+                               return -1;
+                       }

                        /*actualizo el arbol con la nueva clave*/
                        emufs_b_plus_insertar(emu->indices, &query);
                        free(new_bloque);
                        /*actualizo el arbol con la nueva clave*/
                        emufs_b_plus_insertar(emu->indices, &query);
                        free(new_bloque);


@@ -868,6 +878,8 @@ int emufs_tipo1_insertar_ordenado(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, in
                        return 0;
                }
        }
                        return 0;
                }
        }

+       if (new_bloque) free(new_bloque);
+       if (bloque) free(bloque);

        return 0;
 }
 
        return 0;
 }
 


@@ -886,7 +898,6 @@ CLAVE grabar_ordenado_en_bloque(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, void
        /*cargo el header*/
        header.id = emufs_idx_get_new_id(emu, err);
        header.size = size;
        /*cargo el header*/
        header.id = emufs_idx_get_new_id(emu, err);
        header.size = size;

-       cant_reg++; /*voy a poner un reg mas*/

        new_bloque = (char*)malloc(emu->tam_bloque);
        aux = new_bloque;
        for (i=0; i<cant_reg+1; i++){
        new_bloque = (char*)malloc(emu->tam_bloque);
        aux = new_bloque;
        for (i=0; i<cant_reg+1; i++){


@@ -925,6 +936,7 @@ CLAVE grabar_ordenado_en_bloque(EMUFS *emu, void *ptr, EMUFS_REG_SIZE size, void
        }
        /*grabo el bloque en el archivo*/
        new_bloque = aux; /*apunto al principio del bloque*/
        }
        /*grabo el bloque en el archivo*/
        new_bloque = aux; /*apunto al principio del bloque*/

+       cant_reg++; /*voy a poner un reg mas*/

        memcpy(new_bloque+emu->tam_bloque-sizeof(int), &cant_reg, sizeof(int));
        emufs_tipo1_grabar_bloque_fsc(emu, new_bloque, num_bloque, EMUFS_NOT_FOUND, err);
        clave = emufs_indice_generar_clave(emu->indices, new_bloque+sizeof(EMUFS_TIPO1_REG_HEADER));
        memcpy(new_bloque+emu->tam_bloque-sizeof(int), &cant_reg, sizeof(int));
        emufs_tipo1_grabar_bloque_fsc(emu, new_bloque, num_bloque, EMUFS_NOT_FOUND, err);
        clave = emufs_indice_generar_clave(emu->indices, new_bloque+sizeof(EMUFS_TIPO1_REG_HEADER));