Arreglo memory leak (god bless valgrind! :)

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

diff --git a/emufs_gui/articulos.c b/emufs_gui/articulos.c
index 150910ed35ab49d22a3e45fe9eeb07b86decef1a..fbe6fa34626d19ac113e49bde77c3dc54a878135 100644 (file)
--- a/emufs_gui/articulos.c
+++ b/emufs_gui/articulos.c
@@ -14,6 +14,11 @@ static t_Reg_Articulo *crear_nodo_articulo(EMUFS_REG_ID reg, unsigned int num);
 static int agregar_nodo_articulo(t_LstArticulos *lst, t_Reg_Articulo *nodo);
 static int eliminar_nodo_articulo(t_LstArticulos *lst, t_Reg_Articulo *nodo);
 
 static int agregar_nodo_articulo(t_LstArticulos *lst, t_Reg_Articulo *nodo);
 static int eliminar_nodo_articulo(t_LstArticulos *lst, t_Reg_Articulo *nodo);
 

+t_LstArticulos *art_get_lst()
+{
+       return lst_articulos;
+}
+

 int eliminar_nodo_articulo(t_LstArticulos *lst, t_Reg_Articulo *nodo)
 {
        if (nodo == NULL) return 0;
 int eliminar_nodo_articulo(t_LstArticulos *lst, t_Reg_Articulo *nodo)
 {
        if (nodo == NULL) return 0;


@@ -78,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) {
        if (filename != NULL) {
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {

+                       free(tmp);
+                       lst_articulos = NULL;

                        return NULL;
                }
 
                        return NULL;
                }
 


@@ -94,23 +101,24 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                        node = node->next;
                }
 
                        node = node->next;
                }
 

-               fprintf(stderr, "bloque = %d\n", tam_bloque);

                tmp->fp = emufs_crear("articulos", tipo-1, tam_bloque, sizeof(t_Articulo));
                for (node=inicio ; node ; node = node->next) {
                        if (node->type == XML_ELEMENT_NODE) {
                                if (strcmp(node->name, "ARTICULO") == 0) {
                                        t_Articulo art;
                                        void *save;
                tmp->fp = emufs_crear("articulos", tipo-1, tam_bloque, sizeof(t_Articulo));
                for (node=inicio ; node ; node = node->next) {
                        if (node->type == XML_ELEMENT_NODE) {
                                if (strcmp(node->name, "ARTICULO") == 0) {
                                        t_Articulo art;
                                        void *save;

+                                       memset(&art, '*', sizeof(t_Articulo));

                                        art.numero = atoi(xmlGetProp(node, "NroArtículo"));
                                        art.numero = atoi(xmlGetProp(node, "NroArtículo"));

-                                       strncpy(art.desc, xmlGetProp(node, "Descripción"), 50);
-                                       strncpy(art.presentacion, xmlGetProp(node, "Presentación"), 30);
-                                       strncpy(art.existencia, xmlGetProp(node, "Existencia"), 8);
+                                       strcpy(art.desc, xmlGetProp(node, "Descripción"));
+                                       strcpy(art.presentacion, xmlGetProp(node, "Presentación"));
+                                       strcpy(art.existencia, xmlGetProp(node, "Existencia"));

                                        /*strncpy(tmp->array[cant].ubicacion, xmlGetProp(node, "Ubicacion"), 30);*/
                                        /*strncpy(tmp->array[cant].ubicacion, xmlGetProp(node, "Ubicacion"), 30);*/

-                                       strncpy(art.pvu, xmlGetProp(node, "PVU"), 8);
-                                       strncpy(art.emin, xmlGetProp(node, "Emín"), 8);
+                                       strcpy(art.pvu, xmlGetProp(node, "PVU"));
+                                       strcpy(art.emin, xmlGetProp(node, "Emín"));

                                        /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                                        save = procesar_guardar_articulo(&art, &size, lst_articulos);
                                        if (save != NULL) {
                                        /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                                        save = procesar_guardar_articulo(&art, &size, lst_articulos);
                                        if (save != NULL) {

+                                               error = 0;

                                                id = tmp->fp->grabar_registro(tmp->fp, save, size, &error);
                                                agregar_nodo_articulo(tmp, crear_nodo_articulo(id, art.numero));
                                                free(save);
                                                id = tmp->fp->grabar_registro(tmp->fp, save, size, &error);
                                                agregar_nodo_articulo(tmp, crear_nodo_articulo(id, art.numero));
                                                free(save);


@@ -128,6 +136,7 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                        t_Articulo art;
                        void *save;
                        /* Leo el registro */
                        t_Articulo art;
                        void *save;
                        /* Leo el registro */

+                       error = 0;

                        save = tmp->fp->leer_registro(tmp->fp, indices[i], &size, &error);
                        if (procesar_leer_articulo(&art, save, size, tmp) == 1) {
                                agregar_nodo_articulo(tmp, crear_nodo_articulo(indices[i], art.numero));
                        save = tmp->fp->leer_registro(tmp->fp, indices[i], &size, &error);
                        if (procesar_leer_articulo(&art, save, size, tmp) == 1) {
                                agregar_nodo_articulo(tmp, crear_nodo_articulo(indices[i], art.numero));


@@ -144,9 +153,14 @@ int art_liberar(t_LstArticulos *l)
 {
        if (l == NULL) l = lst_articulos;
        if (l == NULL) return 1;
 {
        if (l == NULL) l = lst_articulos;
        if (l == NULL) return 1;

+       t_Reg_Articulo *del;

 
        emufs_destruir(l->fp);
 
        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;
        free(l);
 
        lst_articulos = NULL;


@@ -366,6 +380,7 @@ void art_agregar(char *s)
                /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                save = procesar_guardar_articulo(&art, &size, lst_articulos);
                if (save != NULL) {
                /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                save = procesar_guardar_articulo(&art, &size, lst_articulos);
                if (save != NULL) {

+                       error = 0;

                        id = lst_articulos->fp->grabar_registro(lst_articulos->fp, save, size, &error);
                        if (error) {
                                wattron(win, COLOR_PAIR(COLOR_YELLOW));
                        id = lst_articulos->fp->grabar_registro(lst_articulos->fp, save, size, &error);
                        if (error) {
                                wattron(win, COLOR_PAIR(COLOR_YELLOW));


@@ -400,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 :-) */
                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 */
                        memcpy(&dst->numero, ini, sizeof(unsigned int));
                        ini+=sizeof(unsigned int);
                        /* Ahora empieza el juego */


@@ -425,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;
                        
                        ini = fin+1;
                        fin = (char *)src+size;

-                       memcpy(dst->emin, ini, fin-ini+1);
+                       memcpy(dst->emin, ini, fin-ini);

 
                        break;
                case T3:
 
                        break;
                case T3:


@@ -454,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;
                        (*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]);
                        memcpy(tmp, &src->numero, i[0]);
                        memcpy(tmp+i[0], src->desc, i[1]);
                        memcpy(tmp+i[0]+i[1], src->presentacion, i[2]);


@@ -465,24 +482,88 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
                        /* Lleno el lugar no ocupado de los strings con *, para que el ver
                         * registro se vea bien 
                         */
                        /* Lleno el lugar no ocupado de los strings con *, para que el ver
                         * registro se vea bien 
                         */

-                       i[0] = sizeof(unsigned int);
-                       i[1] = sizeof(char)*(strlen(src->desc)+1);
-                       i[2] = sizeof(char)*(strlen(src->presentacion)+1);
-                       i[3] = sizeof(char)*(strlen(src->existencia)+1);
-/*                     i[4] = sizeof(char)*(strlen(src->ubicacion)+1); */
-                       i[4] = sizeof(char)*(strlen(src->pvu)+1);
-                       i[5] = sizeof(char)*(strlen(src->emin)+1);

                        tmp = (char *)malloc(sizeof(t_Articulo));
                        tmp = (char *)malloc(sizeof(t_Articulo));

-                       if (tmp == NULL) return NULL;
-                       memcpy(tmp, from, i[0]);
-                       memcpy(tmp+i[0], from+i[0], i[1]); memset(tmp+i[0]+i[1], '*', 51-i[1]);
-                       memcpy(tmp+i[0]+51, from+i[0]+51, i[2]); memset(tmp+i[0]+51+i[2], '*', 31-i[2]);
-                       memcpy(tmp+i[0]+82, from+i[0]+82, i[3]); memset(tmp+i[0]+82+i[3], '*', 9-i[3]);
-                       memcpy(tmp+i[0]+91, from+i[0]+91, i[4]); memset(tmp+i[0]+91+i[4], '*', 9-i[4]);
-                       memcpy(tmp+i[0]+100, from+i[0]+100, i[5]); memset(tmp+i[0]+100+i[5], '*', 9-i[5]);
-
+                       memset(tmp, '*', sizeof(t_Articulo));
+                       memcpy(tmp, from, sizeof(t_Articulo));

                        (*size) = sizeof(t_Articulo);
        }
        return tmp;
 }
 
                        (*size) = sizeof(t_Articulo);
        }
        return tmp;
 }
 

+void art_reformatear(int tipo, int tam_bloque, int tam_reg)
+{
+       EMUFS *nuevo, *old;
+       EMUFS_REG_ID *indices, id;
+       EMUFS_REG_SIZE indices_total, i, size;
+       t_Articulo art;
+       t_LstArticulos *lst_nueva;
+       int error;
+       char *save;
+
+       PERR("==== EMPIEZO ====\n");
+       old = lst_articulos->fp;
+
+       /* Si el tipo es el mismo, no tengo que hacer nada! */
+       if (old->tipo == tipo) return;
+
+       /* Creo el nuevo file */
+       PERR("Creo el archivo\n");
+       nuevo = emufs_crear("emufs_tmp", tipo, tam_bloque, sizeof(t_Articulo));
+       if (nuevo == NULL) {
+               fprintf(stderr, "ARCHIVO NUEVO NO CREADO\n");
+               return;
+       }
+
+       /* Creo la nueva lista */
+       lst_nueva = (t_LstArticulos *)malloc(sizeof(t_LstArticulos));
+       lst_nueva->primero = NULL;
+       lst_nueva->fp = nuevo;
+
+       /* Leo los indices del archivo viejo */
+       PERR("Obtengo Indices\n");
+       indices = emufs_idx_get(old, &indices_total);
+       if (indices == NULL) {
+               art_liberar(lst_nueva);
+               return;
+       }
+
+       PERR("Proceso datos\n");
+       for(i=0; i<indices_total; i++) {
+               error = 0;
+               save = old->leer_registro(old, indices[i], &size, &error);
+               if (procesar_leer_articulo(&art, save, size, lst_articulos) == 1) {
+                       free(save);
+                       /* Lei un registro Ok. Lo salvo en el archivo nuevo */
+                       save = procesar_guardar_articulo(&art, &size, lst_nueva);
+                       if (save) {
+                               error = 0;
+                               id = nuevo->grabar_registro(nuevo, save, size, &error);
+                               agregar_nodo_articulo(lst_nueva, crear_nodo_articulo(id, art.numero));
+                               free(save);
+                       }
+               }
+       }
+
+       free(indices);
+
+       PERR("Libero lo viejo\n");
+       art_liberar(lst_articulos);
+
+       PERR("Ahora tengo lo nuevo\n");
+       lst_articulos = lst_nueva;
+
+       /* El nuevo tiene como nombre emufs_tmp, lo cambio a mano! */
+       free(lst_articulos->fp->nombre);
+       lst_articulos->fp->nombre = (char *)malloc(sizeof(char)*(strlen("articulos")+1));
+       strcpy(lst_articulos->fp->nombre, "articulos");
+       
+       /* Muevo los archivos! */
+       /* TODO : Poner en otro lugar mas generico! */
+       PERR("Renombre!!\n");
+       rename("emufs_tmp.dat", "articulos.dat");
+       rename("emufs_tmp.idx", "articulos.idx");
+       rename("emufs_tmp.fsc", "articulos.fsc");
+       rename("emufs_tmp.did", "articulos.did");
+       PERR("==== TERMINE ====\n");
+}
+