* Valgrind rulez :-)

[z.facultad/75.06/emufs.git] / emufs_gui / facturas.c

diff --git a/emufs_gui/facturas.c b/emufs_gui/facturas.c
index 1ad3e5dd3e9a196a0c24b2e42186c33c3994997f..7a594d385658265c96b10d3d03ed97160091684f 100644 (file)
--- a/emufs_gui/facturas.c
+++ b/emufs_gui/facturas.c
@@ -22,12 +22,13 @@ t_LstFacturas *fact_get_lst()
 char *xml_get_prop(xmlNode *node, char *nombre)
 {
        char *s;
-       if (xmlGetProp(node, nombre) == NULL) {
+       s = xmlGetProp(node, nombre);
+       if (s == NULL) {
                s = malloc(1);
                s[0] = '\0';
                return s;
        }
-       return xmlGetProp(node, nombre);
+       return s;
 }
 
 int eliminar_nodo_factura(t_LstFacturas *lst, t_Reg_Factura *nodo)
@@ -91,7 +92,7 @@ t_Item *leer_items(xmlNode *node, int *cant, int size)
                                if (strcmp(node->name, "ITEMVENTA") == 0) {
                                        count++;
                                        tmp = realloc(tmp, sizeof(t_Item)*count);
-                                       memset(&tmp[count-1], '*', sizeof(t_Item));
+                                       memset(&tmp[count-1], 0, sizeof(t_Item));
                                        prop = xml_get_prop(node, "NroArtículo");
                                        tmp[count-1].numero = atoi(prop);
                                        xmlFree(prop);
@@ -174,6 +175,7 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                PERR("Voy a cargar de un XML");
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {
+                       PERR("Error al leer documento!!");
                        free(tmp);
                        lst_facturas = NULL;
                        return NULL;
@@ -292,6 +294,7 @@ int fact_liberar(t_LstFacturas *l)
        if (l == NULL) return 1;
 
        emufs_destruir(l->fp);
+       emufs_destruir(l->fp_texto);
        while (l->primero) {
                del = l->primero;
                l->primero = l->primero->sig;
@@ -541,6 +544,7 @@ void fact_agregar(char *s)
        mvwaddstr(win, 15, 2, "Numero");
        mvwaddstr(win, 15, 11, "CV");
        mvwaddstr(win, 15, 21, "PVU");
+       cant = 0;
        do {
                form_set_valor(form, "Nro de Articulo (* == fin)", "");
                form_set_valor(form, "CV", "");
@@ -562,9 +566,11 @@ void fact_agregar(char *s)
                        /* Agrego el Item */
                        cant++;
                        its = (t_Item *)realloc(its, cant*sizeof(t_Item));
-                       its[cant-1].numero = atoi(entrada);
-                       strcpy(its[cant-1].cv, form_obtener_valor_char(form, "CV"));
-                       strcpy(its[cant-1].pvu, form_obtener_valor_char(form, "PVU"));
+                       if (its != NULL) {
+                               its[cant-1].numero = atoi(entrada);
+                               strcpy(its[cant-1].cv, form_obtener_valor_char(form, "CV"));
+                               strcpy(its[cant-1].pvu, form_obtener_valor_char(form, "PVU"));
+                       }
                }
        } while (entrada[0] != '*');