* Modifico el tipo que estaba mal.

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

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index 5ccb51999b58059e589c1e10c524cd736068b96b..afe97d406741b8a90733696a0d6c70e165844504 100644 (file)
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -38,25 +38,26 @@
 #include "tipo3.h"
 
 /** Leo un registro del archivo, devuelve cero si no lo encuentra.**/
 #include "tipo3.h"
 
 /** Leo un registro del archivo, devuelve cero si no lo encuentra.**/

-int emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID, void *ptr)
+void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
+               EMUFS_REG_SIZE* reg_size, int* err)

 {
        char* bloque;
 {
        char* bloque;

+       char* registro; /* registro a leer */

        EMUFS_BLOCK_ID block;
        EMUFS_REG_ID ID_aux;
        EMUFS_BLOCK_SIZE iterador = 0;
        
        /*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*/
        EMUFS_BLOCK_ID block;
        EMUFS_REG_ID ID_aux;
        EMUFS_BLOCK_SIZE iterador = 0;
        
        /*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*/

-       bloque = (char*)malloc(emu->tam_bloque);
-       if (bloque == NULL) {
-               printf("No hay memoria.\n");
-               return -1;
+       if ( block == EMUFS_NOT_FOUND ){
+               PERR("No se encontro el bloque");
+               return NULL;

        }
        }

-       
-       if (emufs_tipo3_leer_bloque(emu, block, bloque)==-1) {
-               free(bloque);
-               printf("no se pudo leer el bloque\n");
-               return -1; /*No se pudo leer el bloque*/
+       if ((bloque = emufs_tipo3_leer_bloque(emu, block, err)) == NULL) {
+               /* TODO Manejo de errores, queda en el codigo de error lo que devolvio
+                * emufs_tipo3_leer_bloque() */
+               PERR("no se pudo leer el bloque");
+               return NULL; /*No se pudo leer el bloque*/

        }
 
        ID_aux = -1;
        }
 
        ID_aux = -1;


@@ -65,36 +66,62 @@ int emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID, void *ptr)
                memcpy(&ID_aux, bloque+iterador, sizeof(EMUFS_REG_ID));
                iterador += sizeof(EMUFS_REG_ID);
                if ( ID_aux == ID ){
                memcpy(&ID_aux, bloque+iterador, sizeof(EMUFS_REG_ID));
                iterador += sizeof(EMUFS_REG_ID);
                if ( ID_aux == ID ){

-                       memcpy(ptr,bloque+iterador,emu->tam_reg);
+                       registro = (char*) malloc(emu->tam_reg);
+                       if (registro == NULL) {
+                               /* TODO Manejo de errores */
+                               free(bloque);
+                               PERR("No hay memoria");
+                               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+                               return NULL;
+                       }
+                       memcpy(registro,bloque+iterador,emu->tam_reg);
+                       *reg_size = emu->tam_reg;

                        break;
                }
                iterador += emu->tam_reg;
        }
        
        free(bloque);
                        break;
                }
                iterador += emu->tam_reg;
        }
        
        free(bloque);

-       return 0;
+       return registro;

 }
 
 /*leo el bloque "ID" del archivo que viene en "emu->nombre", y lo almaceno en "ptr"*/
 }
 
 /*leo el bloque "ID" del archivo que viene en "emu->nombre", y lo almaceno en "ptr"*/

-int emufs_tipo3_leer_bloque(EMUFS *emu, EMUFS_REG_ID ID, void* ptr)
+void* emufs_tipo3_leer_bloque(EMUFS *emu, EMUFS_REG_ID ID, int* err)

 {
        FILE* file;
 {
        FILE* file;

+       char* block; /* bloque leido (en donde está el registro a leer) */

        char name_f[255];
        
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");
        
        char name_f[255];
        
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");
        

-       if ( (file = fopen(name_f,"r"))==NULL ) return -1; /*ERROR*/
+       if ((file = fopen(name_f, "r")) == NULL) {
+               *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
+               return NULL; /* FIXME ERROR */
+       }

        fseek(file,sizeof(EMUFS_TYPE)+sizeof(EMUFS_BLOCK_SIZE)+sizeof(EMUFS_REG_SIZE),SEEK_SET);
        /*FIXME: verificar que no se pase de fin de archivo*/
        fseek(file,ID*emu->tam_bloque,SEEK_CUR);
        fseek(file,sizeof(EMUFS_TYPE)+sizeof(EMUFS_BLOCK_SIZE)+sizeof(EMUFS_REG_SIZE),SEEK_SET);
        /*FIXME: verificar que no se pase de fin de archivo*/
        fseek(file,ID*emu->tam_bloque,SEEK_CUR);

-       if (fread(ptr,emu->tam_bloque,1,file)!=1) return -1;
+       block = (char*) malloc(emu->tam_bloque);
+       if (block == NULL) {
+               /* TODO Manejo de errores */
+               PERR("No hay memoria");
+               *err = 2; /* 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");
+               *err = 3; /* EMUFS_ERROR_FILE_READ */
+               return NULL;
+       }

 
        fclose(file);
 
        fclose(file);

-       return 0;
+       return block;

 }
 
 }
 

-EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr)
+EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE tam, int* err)

 {
        EMUFS_REG_ID ID_aux;
        EMUFS_FREE fs;
 {
        EMUFS_REG_ID ID_aux;
        EMUFS_FREE fs;


@@ -108,7 +135,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr)
        strcat(name_f,".dat");
        
        /* me devuelve el ID del bloque donde quepa un registro y el espacio libre en "fs"*/
        strcat(name_f,".dat");
        
        /* 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, &fs);
+       num_bloque = emufs_fsc_buscar_lugar(emu, emu->tam_reg+sizeof(EMUFS_REG_ID), &fs);

        /*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*/


@@ -116,7 +143,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr)
                bloque = (char*)malloc(emu->tam_bloque);
                /* grabar el registro al principio del bloque */
                /*tengo que buscar un ID valido para el nuevo registro*/
                bloque = (char*)malloc(emu->tam_bloque);
                /* grabar el registro al principio del bloque */
                /*tengo que buscar un ID valido para el nuevo registro*/

-               ID_aux = emufs_tipo3_get_id(emu);
+               ID_aux = emufs_idx_get_new_id(emu, err);

                /*grabo el id en el bloque*/
                memcpy(bloque,&ID_aux,sizeof(EMUFS_REG_ID));
                /*grabo el registro en el bloque*/
                /*grabo el id en el bloque*/
                memcpy(bloque,&ID_aux,sizeof(EMUFS_REG_ID));
                /*grabo el registro en el bloque*/


@@ -129,47 +156,43 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr)
                /*tengo que buscar la cantidad de bloques que existen*/
                /*me paro al principio salteando el encabezado del archivo*/
                fseek(file, 0, SEEK_END); /* Me paro al final */
                /*tengo que buscar la cantidad de bloques que existen*/
                /*me paro al principio salteando el encabezado del archivo*/
                fseek(file, 0, SEEK_END); /* Me paro al final */

-               /* FIXME FIXME FIXME FALTA TRADUCIR A EMUFS_XXXX */
-               cant = (ftell(file)-(sizeof(int)*2+sizeof(char))) / emu->tam_bloque;
+               cant = (ftell(file)-(sizeof(EMUFS_TYPE)+sizeof(EMUFS_REG_SIZE)+sizeof(EMUFS_BLOCK_SIZE))) / emu->tam_bloque;

                cant--; /* Resto uno porque el numero de bloque debe empezar en 0 */
                fclose(file);
                num_bloque = cant;
                /* grabo el nuevo registro en el archivo de espacios libres */
                cant--; /* Resto uno porque el numero de bloque debe empezar en 0 */
                fclose(file);
                num_bloque = cant;
                /* grabo el nuevo registro en el archivo de espacios libres */

-               /* FIXME FIXME FIXME FALTA TRADUCIR A EMUFS_XXXX */
-               if ( emufs_fsc_agregar(emu, num_bloque, emu->tam_bloque - emu->tam_reg - sizeof(int)) != 0 ) {
+               if ( emufs_fsc_agregar(emu, num_bloque, emu->tam_bloque - emu->tam_reg - sizeof(EMUFS_REG_ID)) != 0 ) {

                        free(bloque);
                        return -1;
                }
        } else {
                /*cargo el bloque en "bloque"*/
                        free(bloque);
                        return -1;
                }
        } else {
                /*cargo el bloque en "bloque"*/

-               bloque = (char*)malloc(emu->tam_bloque);
-               if ( emufs_tipo3_leer_bloque(emu,num_bloque,bloque)== -1) {
-                       printf("Error: no se pudo leer bloque\n");
-                       return -1; 
+               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*/
                }
                /*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_tipo3_get_id(emu);
+               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);
                /*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);

-               /*guardo el bloque en el archivo*/

                if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) != 0) {
                if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) != 0) {

-                       printf("error al grabar bloque\n");
+                       PERR("error al grabar bloque");

                        return -1; /* se produjo un error */    
                }
                /*actualizo el archivo de espacios libres*/
                        return -1; /* se produjo un error */    
                }
                /*actualizo el archivo de espacios libres*/

-               /* FIXME FIXME FIXME FALTA TRADUCIR A EMUFS_XXXX */
-               if ( emufs_fsc_actualizar(emu, num_bloque, fs - emu->tam_reg - sizeof(int)) != 0 ){
+               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*/
                        free(bloque);
                        return -1;
                }
        }
                
        /*actualizo el archivo de bloques y registros*/

-       if ( emufs_idx_agregar(emu, num_bloque, ID_aux) != 0 ){
+       if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){

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


@@ -178,16 +201,6 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr)
        return ID_aux;
 }
 
        return ID_aux;
 }
 

-/*Busco en el archivo de Id`s un Id valido para un nuevo registro*/
-EMUFS_REG_ID emufs_tipo3_get_id(EMUFS *emu)
-{
-       EMUFS_REG_ID id;
-
-       if ( (id = emufs_did_get_last(emu)) == -1 )
-               id = emufs_idx_buscar_mayor_id(emu);
-       return id;      
-}
-

 /*Graba un bloque en el archivo*/
 int emufs_tipo3_grabar_bloque(EMUFS *emu, void *ptr, EMUFS_BLOCK_ID num)
 {
 /*Graba un bloque en el archivo*/
 int emufs_tipo3_grabar_bloque(EMUFS *emu, void *ptr, EMUFS_BLOCK_ID num)
 {


@@ -199,7 +212,7 @@ int emufs_tipo3_grabar_bloque(EMUFS *emu, void *ptr, EMUFS_BLOCK_ID num)
        
        if ( (file = fopen(name_f,"r+"))==NULL ) return -1; /*ERROR*/
        /* Salto el header del archivo */
        
        if ( (file = fopen(name_f,"r+"))==NULL ) return -1; /*ERROR*/
        /* Salto el header del archivo */

-       fseek(file, sizeof(char)+sizeof(int)*2, SEEK_SET);
+       fseek(file, sizeof(EMUFS_TYPE)+sizeof(EMUFS_REG_SIZE)+sizeof(EMUFS_BLOCK_SIZE), SEEK_SET);

        fseek(file, num*emu->tam_bloque, SEEK_CUR);     
        fwrite(ptr, emu->tam_bloque, 1, file);
        
        fseek(file, num*emu->tam_bloque, SEEK_CUR);     
        fwrite(ptr, emu->tam_bloque, 1, file);
        


@@ -216,12 +229,12 @@ 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;

 
        num_bloque = emufs_idx_buscar_registro(emu, ID);
 
        num_bloque = emufs_idx_buscar_registro(emu, ID);

-       bloque = (char*)malloc(emu->tam_bloque);
-       if ( emufs_tipo3_leer_bloque(emu,num_bloque, bloque) == -1 ) {
-               printf("No se encontro el bloque\n");
-               free(bloque);
+       if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque, &err))) {
+               /* TODO Manejo de errores */
+               PERR("no se pudo leer el bloque");

                return -1;
        }
 
                return -1;
        }
 


@@ -233,7 +246,7 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
                        break;
                ptr_elim += emu->tam_reg + sizeof(EMUFS_REG_ID);
        }
                        break;
                ptr_elim += emu->tam_reg + sizeof(EMUFS_REG_ID);
        }

-       
+

        /*apunto al registro que voy a mover*/
        ptr_mov = ptr_elim + emu->tam_reg + sizeof(EMUFS_REG_ID);
        
        /*apunto al registro que voy a mover*/
        ptr_mov = ptr_elim + emu->tam_reg + sizeof(EMUFS_REG_ID);
        


@@ -242,14 +255,14 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
                ptr_elim = ptr_mov;
                ptr_mov += sizeof(EMUFS_REG_ID) + emu->tam_reg;
        }
                ptr_elim = ptr_mov;
                ptr_mov += sizeof(EMUFS_REG_ID) + emu->tam_reg;
        }

-       
+

        /*grabo el bloque en el archivo*/       
        if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) == -1 ){
                free(bloque);
        /*grabo el bloque en el archivo*/       
        if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) == -1 ){
                free(bloque);

-               printf("No se pudo grabar el bloque\n"); 
+               PERR("No se pudo grabar el bloque"); 

                return -1;
        }
                return -1;
        }

-       
+

        /*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;
        /*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;


@@ -259,7 +272,7 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
                
        /*actualizo archivo .idx*/
        if ( emufs_idx_borrar(emu, ID) != 0 ) return -1; 
                
        /*actualizo archivo .idx*/
        if ( emufs_idx_borrar(emu, ID) != 0 ) return -1; 

-       
+

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