X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/blobdiff_plain/38e8783579e4b158b1fced1caec93e16ae05afc4..b172089853a56455742c11dfaf940544f57fef70:/emufs/emufs.c

diff --git a/emufs/emufs.c b/emufs/emufs.c
index a6a774c..7ddd4bd 100644
--- a/emufs/emufs.c
+++ b/emufs/emufs.c
@@ -79,22 +79,26 @@ int emufs_crear_archivo_auxiliar(const char* name, const char* ext)
 }
 
 
-EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, unsigned long tam_reg)
+EMUFS *emufs_crear(const char *filename, EMUFS_TYPE tipo,
+		EMUFS_BLOCK_SIZE tam_bloque, EMUFS_REG_SIZE tam_reg)
 {
 	char name[255];
 	FILE *fp;
 	EMUFS *efs;
 
-	/* Si no es un tipo conocido, sale */
+	/* Si no es un tipo conocido, sale. */
 	if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {
 		return NULL;
 	}
 
 	/* Inicializa parámetros comunes. */
 	efs = (EMUFS*) malloc(sizeof(EMUFS));
-	efs->tipo = tipo;
+	if (efs == NULL) {
+		return NULL;
+	}
+	efs->tipo       = tipo;
 	efs->tam_bloque = tam_bloque;
-	efs->nombre = str_dup(filename);
+	efs->nombre     = str_dup(filename);
 
 	/* Abre archivo de datos. */
 	strcpy(name, filename);
@@ -108,7 +112,7 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
 	}
 
 	/* Guarda cabecera común. */
-	fwrite(&tipo, sizeof(char), 1, fp); /* FIXME no debería ser sizeof(EMUFS_TYPE) ? */
+	fwrite(&tipo, sizeof(EMUFS_TYPE), 1, fp);
 
 	/* Crea archivo de índice. */
 	if (emufs_idx_crear(efs)) {
@@ -138,14 +142,10 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
 	switch (tipo) {
 
 		case T1:
-			/* Asigna punteros a funciones. */
-			efs->leer_bloque     = emufs_tipo1_leer_bloque;
-			efs->leer_registro   = emufs_tipo1_leer_registro;
-			efs->grabar_registro = emufs_tipo1_grabar_registro;
-			efs->borrar_registro = emufs_tipo1_borrar_registro;
+			emufs_tipo1_inicializar(efs);
 
 			/* Guarda cabeceras propias. */
-			fwrite(&tam_bloque, sizeof(unsigned long), 1, fp);
+			fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);
 
 			break;
 
@@ -160,8 +160,8 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
 			efs->borrar_registro = emufs_tipo3_borrar_registro;
 
 			/* Guarda cabeceras propias. */
-			fwrite(&tam_bloque, sizeof(unsigned int), 1, fp);
-			fwrite(&tam_reg, sizeof(unsigned int), 1, fp);
+			fwrite(&tam_bloque, sizeof(EMUFS_BLOCK_SIZE), 1, fp);
+			fwrite(&tam_reg, sizeof(EMUFS_REG_SIZE), 1, fp);
 			
 			break;
 
@@ -173,7 +173,7 @@ EMUFS *emufs_crear(const char *filename, char tipo, unsigned long tam_bloque, un
 
 EMUFS *emufs_abrir(const char *filename)
 {
-	EMUFS *tmp;
+	EMUFS *efs;
 	char name[255];
 	char tipo;
 	FILE *fp;
@@ -184,35 +184,53 @@ EMUFS *emufs_abrir(const char *filename)
 	/* Trato de determinar el tipo de archivo */
 	fp = fopen(name, "r");
 	if (fp == NULL) return NULL;
-	fread(&tipo, sizeof(char), 1, fp);
-	if ((tipo < 0) || (tipo > 2)) {
+	fread(&tipo, sizeof(EMUFS_TYPE), 1, fp);
+
+	/* Si no es un tipo conocido, sale. */
+	if ((tipo != T1) && (tipo != T2) && (tipo != T3)) {
 		fclose(fp);
 		return NULL;
 	}
 	
-	tmp = (EMUFS *)malloc(sizeof(EMUFS));
-	if (tmp == NULL) {
+	/* Inicializa parámetros comunes. */
+	efs = (EMUFS*) malloc(sizeof(EMUFS));
+	if (efs == NULL) {
 		fclose(fp);
 		return NULL;
 	}
+	efs->tipo = tipo;
+	efs->nombre = str_dup(filename);
 
 	switch (tipo) {
 		case T1:
-		break;
+			emufs_tipo1_inicializar(efs);
+			/* Lee cabeceras propias. */
+			if (!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) {
+				free(efs->nombre);
+				free(efs);
+				fclose(fp);
+				return NULL;
+			}
+			break;
 		case T2:
-		break;
+			break;
 		case T3:
-			tmp->tipo = tipo;
-			fread(&tmp->tam_bloque, sizeof(int), 1, fp);
-			tmp->leer_bloque = emufs_tipo3_leer_bloque;
-			tmp->leer_registro = emufs_tipo3_leer_registro;
-			tmp->grabar_registro = emufs_tipo3_grabar_registro;
-			tmp->borrar_registro = emufs_tipo3_borrar_registro;
-			tmp->nombre = str_dup(filename);
+			if (!fread(&(efs->tam_bloque), sizeof(EMUFS_BLOCK_SIZE), 1, fp)) {
+				free(efs->nombre);
+				free(efs);
+				fclose(fp);
+				return NULL;
+			}
+			/* FIXME no falta leer el tamaño del registro????  */
+			efs->leer_bloque = emufs_tipo3_leer_bloque;
+			efs->leer_registro = emufs_tipo3_leer_registro;
+			efs->grabar_registro = emufs_tipo3_grabar_registro;
+			efs->borrar_registro = emufs_tipo3_borrar_registro;
+			break;
 	}
 
 	fclose(fp);
-	return tmp;
+	return efs;
 }
 
 int emufs_destruir(EMUFS *e)