X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/blobdiff_plain/0e6e058c514e07878085512095d5be4f5b50e8d9..d4f0853a534fc4283fa38cf4bb4a70cc614283ec:/emufs/tipo3.c

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index 062a948..feb000f 100644
--- 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;
 }
-