cambios en los sizeof

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

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index 5ccb51999b58059e589c1e10c524cd736068b96b..4971f5e8452ad3bf0b03413b3b7ed2e0bafd59ef 100644 (file)
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -38,25 +38,21 @@
 #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, 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 (emufs_tipo3_leer_bloque(emu, block, bloque)==-1) {
-               free(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() */

                printf("no se pudo leer el bloque\n");
                printf("no se pudo leer el bloque\n");

-               return -1; /*No se pudo leer el bloque*/
+               return NULL; /*No se pudo leer el bloque*/

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


@@ -65,36 +61,61 @@ 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);
+                               printf("No hay memoria.\n");
+                               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+                               return NULL;
+                       }
+                       memcpy(registro,bloque+iterador,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 */
+               printf("No hay memoria.\n");
+               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+               return NULL;
+       }
+       if (fread(block, emu->tam_bloque, 1, file) != 1) {
+               /* TODO Manejo de errores */
+               free(block);
+               printf("Error al leer bloque.\n");
+               *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)

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


@@ -103,6 +124,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr)
        FILE *file;
        char name_f[255];
        char* bloque;
        FILE *file;
        char name_f[255];
        char* bloque;

+       int err = 0;

        
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");
        
        strcpy(name_f,emu->nombre);
        strcat(name_f,".dat");


@@ -129,23 +151,21 @@ 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 */
+                       printf("no se pudo leer el bloque\n");
+                       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*/
                }
                /*El error puede haberse producido porque la funcion leer_bloque devolvio -1, el cual es un bloque invalido*/
                /*insertar el registro en el bloque*/


@@ -155,14 +175,12 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr)
                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);
                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) {
                        printf("error al grabar bloque\n");
                        return -1; /* se produjo un error */    
                }
                /*actualizo el archivo de espacios libres*/
                if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque) != 0) {
                        printf("error al grabar bloque\n");
                        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;
                }
                        free(bloque);
                        return -1;
                }


@@ -199,7 +217,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 +234,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;
+printf("pase %d\n",__LINE__);

        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 */
+               printf("no se pudo leer el bloque\n");

                return -1;
        }
 
                return -1;
        }
 


@@ -233,7 +251,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);
        }

-       
+printf("pase %d\n",__LINE__);  

        /*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 +260,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;
        }

-       
+printf("pase %d\n",__LINE__);  

        /*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"); 
                return -1;
        }
        /*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"); 
                return -1;
        }

-       
+printf("pase %d\n",__LINE__);  

        /*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 +277,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; 

-       
+printf("pase %d\n",__LINE__);  

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