X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/blobdiff_plain/07dca6914e0894f05543795c7c453168e0d572cd..46681084b842380621fe70d8a34fffe8f1efb5f6:/emufs/idx.c?ds=sidebyside

diff --git a/emufs/idx.c b/emufs/idx.c
index 6cf562e..ce12761 100644
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -78,6 +78,7 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
 	strcpy(name_f_idx, emu->nombre);
 	strcat(name_f_idx, EMUFS_IDX_EXT);
 
+	(*err) = 0;
 	if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
 		PERR("No se puede abrir archivo");
 		*err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
@@ -86,6 +87,7 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
 	while (!feof(f_idx)) {
 		/* Me aseguro de leer la cantidad de bytes correcta */
 		if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) {
+			if (feof(f_idx)) break; /* No leyó por EOF */
 			PERR("Error al leer registros de idx");
 			*err = 3; /* EMUFS_ERROR_FILE_READ */
 			return EMUFS_NOT_FOUND;
@@ -105,30 +107,36 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
 }
 
 /* busca el registro ID en el archivo ".idx" y devuelve el nro de bloque en el que se encuentra */
-EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID n_IdReg)
+EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID reg_id)
 {
 	FILE* f_idx;
 	EMUFS_IDX reg;
 	char name_f_idx[255];
-	unsigned short int b_Found = 0;
 	
 	strcpy(name_f_idx,emu->nombre);
 	strcat(name_f_idx, EMUFS_IDX_EXT);
 	
-	if ( (f_idx = fopen(name_f_idx,"r")) == NULL) return -1; /*ERROR*/
+	if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
+		PERR("No se puede abrir archivo");
+		/*    *err = 4; * EMUFS_ERROR_CANT_OPEN_FILE */
+		return EMUFS_NOT_FOUND;
+	}
 
-	while (!feof(f_idx) && !b_Found){
-		if (fread(&reg,sizeof(EMUFS_IDX),1,f_idx) != 1) continue;
-		if (reg.n_idreg == n_IdReg) b_Found = 1;
+	while (!feof(f_idx)) {
+		if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) {
+			if (feof(f_idx)) break; /* No leyó por EOF */
+			PERR("Error al leer registros de idx");
+			/*   *err = 3; * EMUFS_ERROR_FILE_READ */
+			return EMUFS_NOT_FOUND;
+		}
+		if (reg.n_idreg == reg_id) {
+			fclose(f_idx);
+			return reg.n_location;
+		}
 	}
-	
 	fclose(f_idx);
-	
-	/* Sino lo encontre devuelvo uno, otherwise el offset o bloque */
-	if (!b_Found)
-		return(-1);
-	else
-		return(reg.n_location);
+
+	return EMUFS_NOT_FOUND;
 }
 
 /* agrega un registro al final del archivo */
@@ -145,7 +153,7 @@ int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID n_idreg, EMUFS_BLOCK_ID n_locatio
 		
     /* Note: Location = Bloque para Tipo 1 y 3, Offset para Tipo 2 */
     reg.n_idreg = n_idreg;
-		reg.n_location = n_location;
+	reg.n_location = n_location;
 	fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx); 
 	fclose(f_idx);
 	return 0;
@@ -214,10 +222,6 @@ EMUFS_REG_ID emufs_idx_get_new_id(EMUFS* efs, int* err)
 
 	id = emufs_did_get_last(efs, err);
 	if (id == EMUFS_NOT_FOUND) {
-		if (*err) {
-			PERR("error al obtener ultimo id");
-			return id;
-		}
 		id = emufs_idx_buscar_mayor_id_libre(efs, err);
 		if (*err) {
 			PERR("error al obtener id mayor");