* 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 bbfcf2087b5a9cc446d1b18171c68f0f8b5e0162..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;
 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;
        }
                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)
 }
 
 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);
                                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);
                                        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("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;
                        free(tmp);
                        lst_facturas = NULL;
                        return NULL;


@@ -195,7 +197,7 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                /* En el registro no guardo los punteros de nota ni items. Si guardo la cantidad de items
                 * y los items al final del registro.
                 */
                /* En el registro no guardo los punteros de nota ni items. Si guardo la cantidad de items
                 * y los items al final del registro.
                 */

-               if (tipo == 3) {
+               if (tipo == T3) {

                        /* Limito a 10 items en el caso de registro constante! */
                        cant_items = 10;
                } else {
                        /* Limito a 10 items en el caso de registro constante! */
                        cant_items = 10;
                } else {


@@ -292,6 +294,7 @@ int fact_liberar(t_LstFacturas *l)
        if (l == NULL) return 1;
 
        emufs_destruir(l->fp);
        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;
        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");
        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", "");
        do {
                form_set_valor(form, "Nro de Articulo (* == fin)", "");
                form_set_valor(form, "CV", "");


@@ -562,21 +566,24 @@ void fact_agregar(char *s)
                        /* Agrego el Item */
                        cant++;
                        its = (t_Item *)realloc(its, cant*sizeof(t_Item));
                        /* 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] != '*');
 
                }
        } while (entrada[0] != '*');
 

-       if (lst_facturas->fp->tipo == 3) {
+       if (lst_facturas->fp->tipo == T3) {

                if (cant != 10) {
                        /* TODO Limitar en la GUI en lugar de truncar! */
                if (cant != 10) {
                        /* TODO Limitar en la GUI en lugar de truncar! */

-                       fprintf(stderr, "Tengo %d\n", cant);

                        its = (t_Item *)realloc(its, 10*sizeof(t_Item));
                        its = (t_Item *)realloc(its, 10*sizeof(t_Item));

-                       fprintf(stderr, "Realoque ok\n");
-                       memset(its+sizeof(t_Item)*cant, 0, (10-cant)*sizeof(t_Item));
-                       fprintf(stderr, "Seteo a 0 sin priblemas\n");
-                       cant = 10;
+                       if (its == NULL) {
+                               cant = 0;
+                       } else {
+                               memset(its+sizeof(t_Item)*cant, 0, (10-cant)*sizeof(t_Item));
+                               cant = 10;
+                       }

                }
        }
        fact.items = its;
                }
        }
        fact.items = its;