X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/blobdiff_plain/13e8a152bee88ed48a0987ddac713dd7cf525966..5720b3f8df66478279f271065447e93848437fb7:/emufs_gui/articulos.c

diff --git a/emufs_gui/articulos.c b/emufs_gui/articulos.c
index 1aad096..b9605c2 100644
--- a/emufs_gui/articulos.c
+++ b/emufs_gui/articulos.c
@@ -83,6 +83,8 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
 	if (filename != NULL) {
 		document = xmlReadFile(filename, "ISO-8859-1",0);
 		if (document == NULL) {
+			free(tmp);
+			lst_articulos = NULL;
 			return NULL;
 		}
 
@@ -151,9 +153,14 @@ int art_liberar(t_LstArticulos *l)
 {
 	if (l == NULL) l = lst_articulos;
 	if (l == NULL) return 1;
+	t_Reg_Articulo *del;
 
 	emufs_destruir(l->fp);
-	/* TODO : Liberar lista */
+	while (l->primero) {
+		del = l->primero;
+		l->primero = l->primero->sig;
+		free(del);
+	}
 	free(l);
 
 	lst_articulos = NULL;
@@ -408,6 +415,7 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
 		case T2:
 			ini = (char *)src;
 			/* Copio el primer campo, esto es facil :-) */
+			memset(dst, '*', sizeof(t_Articulo));
 			memcpy(&dst->numero, ini, sizeof(unsigned int));
 			ini+=sizeof(unsigned int);
 			/* Ahora empieza el juego */
@@ -433,7 +441,7 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
 			
 			ini = fin+1;
 			fin = (char *)src+size;
-			memcpy(dst->emin, ini, fin-ini+1);
+			memcpy(dst->emin, ini, fin-ini);
 
 			break;
 		case T3:
@@ -462,6 +470,7 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
 			(*size) = i[0]+i[1]+i[2]+i[3]+i[4]+i[5];
 			tmp = (char *)malloc((*size));
 			if (tmp == NULL) return NULL;
+			memset(tmp, 0, *size);
 			memcpy(tmp, &src->numero, i[0]);
 			memcpy(tmp+i[0], src->desc, i[1]);
 			memcpy(tmp+i[0]+i[1], src->presentacion, i[2]);
@@ -474,6 +483,7 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
 			 * registro se vea bien 
 			 */
 			tmp = (char *)malloc(sizeof(t_Articulo));
+			memset(tmp, '*', sizeof(t_Articulo));
 			memcpy(tmp, from, sizeof(t_Articulo));
 			(*size) = sizeof(t_Articulo);
 	}
@@ -495,8 +505,6 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
 	/* Si el tipo es el mismo, no tengo que hacer nada! */
 	if (old->tipo == tipo) return;
 
-	fprintf(stderr, "Me prepado para cambiar de archivo\n");
-
 	/* Creo el nuevo file */
 	nuevo = emufs_crear("emufs_tmp", tipo, tam_bloque, sizeof(t_Articulo));
 	if (nuevo == NULL) {
@@ -512,28 +520,21 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
 	/* Leo los indices del archivo viejo */
 	indices = emufs_idx_get(old, &indices_total);
 	if (indices == NULL) {
-		fprintf(stderr, "NO HAY INDICES!\n");
 		art_liberar(lst_nueva);
 		return;
 	}
 
 	for(i=0; i<indices_total; i++) {
-		fprintf(stderr, "Registro %lu de %lu\n", i, indices_total);
-		fprintf(stderr, "A leer : %lu\n", indices[i]);
-
+		fprintf(stderr, "A procesar %lu de %lu\n",i, indices_total);
 		error = 0;
 		save = old->leer_registro(old, indices[i], &size, &error);
-		fprintf(stderr, "Lei\n");
 		if (procesar_leer_articulo(&art, save, size, lst_articulos) == 1) {
-			fprintf(stderr, "Procese\n");
 			free(save);
 			/* Lei un registro Ok. Lo salvo en el archivo nuevo */
 			save = procesar_guardar_articulo(&art, &size, lst_nueva);
-			fprintf(stderr, "Procese guardar\n");
 			if (save) {
 				error = 0;
 				id = nuevo->grabar_registro(nuevo, save, size, &error);
-				fprintf(stderr, "Guarde\n");
 				agregar_nodo_articulo(lst_nueva, crear_nodo_articulo(id, art.numero));
 				free(save);
 			}
@@ -549,9 +550,9 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
 	fprintf(stderr, "Listo. Renombre y me voy\n");
 	/* Muevo los archivos! */
 	/* TODO : Poner en otro lugar mas generico! */
-/*	rename("emufs_tmp.dat", "articulos.dat");
+	rename("emufs_tmp.dat", "articulos.dat");
 	rename("emufs_tmp.idx", "articulos.idx");
 	rename("emufs_tmp.fsc", "articulos.fsc");
-	rename("emufs_tmp.did", "articulos.did");*/
+	rename("emufs_tmp.did", "articulos.did");
 }