X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/blobdiff_plain/9becab83c083dd57876b1d29ec4bc73cbda912ac..28ec60bde7175661afed582949f32390ebafaeaa:/emufs/tipo3.c

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index 049c63d..97752e7 100644
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -36,6 +36,10 @@
  */
 
 #include "tipo3.h"
+#include <unistd.h>
+#include <sys/types.h>
+#include <stdio.h>
+#include <string.h>
 
 /** Leo un registro del archivo, devuelve cero si no lo encuentra.**/
 void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
@@ -61,7 +65,6 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
 	registro = (char*) malloc(emu->tam_reg);
 	if (registro == NULL) {
 		/* TODO Manejo de errores */
-		free(bloque);
 		PERR("No hay memoria");
 		*err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
 		return NULL;
@@ -73,6 +76,7 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
 			/* TODO Manejo de errores, queda en el codigo de error lo que devolvio
 			 * emufs_tipo3_leer_bloque() */
 			PERR("no se pudo leer el bloque");
+			free(registro);
 			return NULL; /*No se pudo leer el bloque*/
 		}
 		ID_aux = -1;
@@ -96,7 +100,7 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
 		}
 		free(bloque);
 	}
-	
+
 	return registro;
 }
 
@@ -150,7 +154,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
 	EMUFS_BLOCK_SIZE cant;
 	FILE *file;
 	char name_f[255];
-	char* bloque;
+	char* bloque = NULL;
 	int cant_bloques, resto, i=0, lugar;
 	
 	strcpy(name_f,emu->nombre);
@@ -244,6 +248,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
 			/*grabo el bloque en el archivo*/
 			if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque+i) != 0) {
 				PERR("error al grabar bloque");
+				if (bloque) free(bloque);
 				return -1; /* se produjo un error */	
 			}
 			
@@ -252,7 +257,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
 				resto = emu->tam_reg;
 				if ( emufs_fsc_agregar(emu, num_bloque, fs - resto - sizeof(EMUFS_REG_ID) ) != 0 ) {
 					fclose(file);
-					if (bloque != NULL) free(bloque);
+					if (bloque) free(bloque);
 					return -1;
 				}
 			} else {	
@@ -260,19 +265,19 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
 					resto = emu->tam_reg - i*(emu->tam_bloque - sizeof(EMUFS_REG_ID));
 				if ( emufs_fsc_agregar(emu, num_bloque+i, fs-resto) !=0 ){
 					fclose(file);
-					if (bloque != NULL) free(bloque);
+					if (bloque) free(bloque);
 					return -1;
 				}
 			}
 			if ( i == 0 ){
 				if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
-					if (bloque != NULL) free(bloque);
+					if (bloque) free(bloque);
 					return -1;
 				}
 			}
 		}
 	}
-	free(bloque);
+	if (bloque) free(bloque);
 	return ID_aux;
 }
 
@@ -507,6 +512,7 @@ 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*/
@@ -520,5 +526,4 @@ void emufs_tipo3_compactar(EMUFS *emu)
 	if(emu->tam_bloque<emu->tam_reg-sizeof(EMUFS_REG_ID)) block_id = block_id/2;
 	if (emufs_fsc_truncate(emu, block_id)!= 0)
 		PERR("NO TURNQUE EL FSC");
-	free(reg);
 }