bugfix.

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

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index 062a94863545e42e2fea7c5113c699063487dcd9..feb000fde4df60d3b6896f21f090040005c95af8 100644 (file)
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -36,6 +36,7 @@
  */
 
 #include "tipo3.h"
+#include "error.h"
 #include <unistd.h>
 #include <sys/types.h>
 #include <stdio.h>
@@ -64,9 +65,8 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
        
        registro = (char*) malloc(emu->tam_reg);
        if (registro == NULL) {
-               /* TODO Manejo de errores */
                PERR("No hay memoria");
-               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+               *err = EMUFS_ERROR_OUT_OF_MEMORY;
                return NULL;
        }
 
@@ -116,8 +116,8 @@ void* emufs_tipo3_leer_bloque(EMUFS *emu, EMUFS_BLOCK_ID ID, int* err)
        
        if ((file = fopen(name_f, "r")) == NULL) {
                PERR("No se pudo abrir el archivo de datos");
-               *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
-               return NULL; /* FIXME ERROR */
+               *err = EMUFS_ERROR_CANT_OPEN_FILE;
+               return NULL;
        }
        fseek(file,sizeof(EMUFS_Tipo)+sizeof(EMUFS_BLOCK_SIZE)+sizeof(EMUFS_REG_SIZE),SEEK_SET);
        /*FIXME: verificar que no se pase de fin de archivo*/
@@ -129,16 +129,15 @@ void* emufs_tipo3_leer_bloque(EMUFS *emu, EMUFS_BLOCK_ID ID, int* err)
        
        block = (char*) malloc(emu->tam_bloque);
        if (block == NULL) {
-               /* TODO Manejo de errores */
                PERR("No hay memoria");
-               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+               *err = 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 */
+               *err = EMUFS_ERROR_FILE_READ;
                return NULL;
        }
 
@@ -350,16 +349,32 @@ int emufs_tipo3_borrar_registro(EMUFS *emu, EMUFS_REG_ID ID)
        /*actualizo archivo .fsc*/
        if ( emu->tam_bloque < emu->tam_reg ) {
                for (i=0; i<emu->tam_reg/(emu->tam_bloque-sizeof(EMUFS_REG_ID))+1; i++)
-                       if ( emufs_fsc_agregar(emu, num_bloque+i, emu->tam_bloque) != 0 ) return -1;
+                       if (emufs_fsc_agregar(emu, num_bloque+i, emu->tam_bloque)) {
+                               PERR("no se pudo agregar fsc"); 
+                               free(bloque);
+                               return -1;
+                       }
        } else { 
                fs = emufs_fsc_get_fs(emu, num_bloque);
-               if ( emufs_fsc_agregar(emu, num_bloque, fs + emu->tam_reg + sizeof(EMUFS_REG_ID)) != 0 ) return -1;
+               if (emufs_fsc_agregar(emu, num_bloque, fs + emu->tam_reg + sizeof(EMUFS_REG_ID))) {
+                       PERR("no se pudo agregar fsc"); 
+                       free(bloque);
+                       return -1;
+               }
        }
        /*actualizo archivo .did*/
-       if ( emufs_did_agregar(emu, ID) != 0 ) return -1;
+       if (emufs_did_agregar(emu, ID)) {
+               PERR("no se pudo agregar did"); 
+               free(bloque);
+               return -1;
+       }
                
        /*actualizo archivo .idx*/
-       if ( emufs_idx_borrar(emu, ID) != 0 ) return -1; 
+       if (emufs_idx_borrar(emu, ID)) {
+               PERR("no se pudo agregar idx"); 
+               free(bloque);
+               return -1;
+       }
 
        free(bloque);
        return 0;
@@ -513,7 +528,6 @@ void emufs_tipo3_compactar(EMUFS *emu)
                emufs_tipo3_borrar_registro(emu, i);
                ID_aux = emufs_tipo3_grabar_registro(emu, reg, emu->tam_reg, &err);
                free(reg);
-               i++;
        }
        /*tengo que truncar el archivo*/
        /*bloques_vacios = emufs_fsc_get_cant_bloques_vacios(emu)-1;
@@ -528,7 +542,8 @@ void emufs_tipo3_compactar(EMUFS *emu)
                PERR("NO TURNQUE EL FSC");
 }
 
-void emufs_tipo3_leer_bloque_raw(EMUFS *efs, EMUFS_BLOCK_ID id, char **actual, char **anterior, char **siguiente, EMUFS_BLOCK_SIZE *size1, EMUFS_BLOCK_SIZE *size2, EMUFS_BLOCK_SIZE *size3)
+void emufs_tipo3_leer_bloque_raw(EMUFS *efs, EMUFS_BLOCK_ID id, char **actual, char **anterior, char **siguiente,
+                                                                EMUFS_BLOCK_SIZE *size1, EMUFS_BLOCK_SIZE *size2, EMUFS_BLOCK_SIZE *size3)
 {
        int err;
        (*actual) = emufs_tipo3_leer_bloque(efs, id, &err);
@@ -544,4 +559,3 @@ void emufs_tipo3_leer_bloque_raw(EMUFS *efs, EMUFS_BLOCK_ID id, char **actual, c
        }
        (*size1) = (*size2) = (*size3) = efs->tam_bloque;
 }
-