* BUGFIX : Estaba siendo olvidado verificar de no sobrepasar

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

diff --git a/emufs_gui/articulos.c b/emufs_gui/articulos.c
index defdf7e5c884d7cf7c19591437d1fa07836b80a6..2e3602a47ba59bdec5fa1d4f90150ba35224c684 100644 (file)
--- a/emufs_gui/articulos.c
+++ b/emufs_gui/articulos.c
@@ -70,6 +70,7 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
        xmlDocPtr document;
        xmlNode *node, *inicio;
        int error = 0, i;
        xmlDocPtr document;
        xmlNode *node, *inicio;
        int error = 0, i;

+       char *prop;

        EMUFS_REG_SIZE size;
        t_LstArticulos *tmp;
        lst_articulos = NULL;
        EMUFS_REG_SIZE size;
        t_LstArticulos *tmp;
        lst_articulos = NULL;


@@ -81,8 +82,11 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
        tmp->primero = NULL;
 
        if (filename != NULL) {
        tmp->primero = NULL;
 
        if (filename != NULL) {

+               PERR("Voy a crear desde un XML");

                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {

+                       free(tmp);
+                       lst_articulos = NULL;

                        return NULL;
                }
 
                        return NULL;
                }
 


@@ -106,13 +110,20 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                                        t_Articulo art;
                                        void *save;
                                        memset(&art, '*', sizeof(t_Articulo));
                                        t_Articulo art;
                                        void *save;
                                        memset(&art, '*', sizeof(t_Articulo));

-                                       art.numero = atoi(xmlGetProp(node, "NroArtículo"));
-                                       strcpy(art.desc, xmlGetProp(node, "Descripción"));
-                                       strcpy(art.presentacion, xmlGetProp(node, "Presentación"));
-                                       strcpy(art.existencia, xmlGetProp(node, "Existencia"));
+                                       prop = xmlGetProp(node, "NroArtículo");
+                                       art.numero = atoi(prop);
+                                       xmlFree(prop);
+                                       strncpy(art.desc, prop = xmlGetProp(node, "Descripción"), 50); xmlFree(prop);
+                                       art.desc[50] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.presentacion, prop = xmlGetProp(node, "Presentación"), 30); xmlFree(prop);
+                                       art.presentacion[30] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.existencia, prop = xmlGetProp(node, "Existencia"), 8); xmlFree(prop);
+                                       art.existencia[8] = '\0'; /* Me aseguro de que este */

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

-                                       strcpy(art.pvu, xmlGetProp(node, "PVU"));
-                                       strcpy(art.emin, xmlGetProp(node, "Emín"));
+                                       strncpy(art.pvu, prop = xmlGetProp(node, "PVU"), 8); xmlFree(prop);
+                                       art.pvu[8] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.emin, prop = xmlGetProp(node, "Emín"), 8); xmlFree(prop);
+                                       art.emin[8] = '\0'; /* Me aseguro de que este */

                                        /* 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) {


@@ -127,7 +138,14 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {

+               PERR("Voy a recuperar desde un archivo");

                tmp->fp = emufs_abrir("articulos");
                tmp->fp = emufs_abrir("articulos");

+               if (tmp->fp == NULL) {
+                       PERR("No se pudo cargar archivo de articulos.");
+                       free(tmp);
+                       lst_articulos = NULL;
+                       return NULL;
+               }

                /* Ahora trato de recuperar la info */
                indices = emufs_idx_get(tmp->fp, &indices_cant);
                for(i=0; i<indices_cant; i++) {
                /* Ahora trato de recuperar la info */
                indices = emufs_idx_get(tmp->fp, &indices_cant);
                for(i=0; i<indices_cant; i++) {


@@ -149,11 +167,16 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
 
 int art_liberar(t_LstArticulos *l)
 {
 
 int art_liberar(t_LstArticulos *l)
 {

+       t_Reg_Articulo *del;

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


@@ -408,7 +431,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));
+                       memset(dst, 0, 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,6 +448,7 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
                        ini = fin+1;
                        fin = ini;
                        while (*fin!='\0') fin++;
                        ini = fin+1;
                        fin = ini;
                        while (*fin!='\0') fin++;

+                       fprintf(stderr, "Presentacion = %s\n", dst->presentacion);

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


@@ -463,9 +487,11 @@ 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]);

+                       fprintf(stderr, "ACA: %s\n", src->presentacion);

                        memcpy(tmp+i[0]+i[1]+i[2], src->existencia, i[3]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3], src->pvu, i[4]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], src->emin, i[5]);
                        memcpy(tmp+i[0]+i[1]+i[2], src->existencia, i[3]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3], src->pvu, i[4]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], src->emin, i[5]);


@@ -475,6 +501,7 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
                         * registro se vea bien 
                         */
                        tmp = (char *)malloc(sizeof(t_Articulo));
                         * 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);
        }
                        memcpy(tmp, from, sizeof(t_Articulo));
                        (*size) = sizeof(t_Articulo);
        }


@@ -491,15 +518,17 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
        int error;
        char *save;
 
        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 */
        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) {
        nuevo = emufs_crear("emufs_tmp", tipo, tam_bloque, sizeof(t_Articulo));
        if (nuevo == NULL) {

-               fprintf(stderr, "ARCHIVO NUEVO NO CREADO\n");
+               PERR("ARCHIVO NUEVO NO CREADO");

                return;
        }
 
                return;
        }
 


@@ -509,14 +538,15 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
        lst_nueva->fp = nuevo;
 
        /* Leo los indices del archivo viejo */
        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;
        }
 
        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++) {
        for(i=0; i<indices_total; i++) {

-               fprintf(stderr, "A procesar %lu de %lu\n",i, indices_total);

                error = 0;
                save = old->leer_registro(old, indices[i], &size, &error);
                if (procesar_leer_articulo(&art, save, size, lst_articulos) == 1) {
                error = 0;
                save = old->leer_registro(old, indices[i], &size, &error);
                if (procesar_leer_articulo(&art, save, size, lst_articulos) == 1) {


@@ -532,18 +562,26 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
                }
        }
 
                }
        }
 

-       ver_archivo_FS(old);
-       ver_archivo_FS(nuevo);
+       free(indices);

 
 

+       PERR("Libero lo viejo\n");

        art_liberar(lst_articulos);
        art_liberar(lst_articulos);

+
+       PERR("Ahora tengo lo nuevo\n");

        lst_articulos = lst_nueva;
 
        lst_articulos = lst_nueva;
 

-       fprintf(stderr, "Listo. Renombre y me voy\n");
+       /* 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! */
        /* 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");
        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");

 }
 
 }