* 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 f3df42d8f4909b53754af7469f964643528c64e5..2e3602a47ba59bdec5fa1d4f90150ba35224c684 100644 (file)
--- a/emufs_gui/articulos.c
+++ b/emufs_gui/articulos.c
@@ -103,7 +103,6 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                        node = node->next;
                }
 
                        node = node->next;
                }
 

-               fprintf(stderr, "Creando articulos con 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) {
                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) {


@@ -114,12 +113,17 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                                        prop = xmlGetProp(node, "NroArtículo");
                                        art.numero = atoi(prop);
                                        xmlFree(prop);
                                        prop = xmlGetProp(node, "NroArtículo");
                                        art.numero = atoi(prop);
                                        xmlFree(prop);

-                                       strcpy(art.desc, prop = xmlGetProp(node, "Descripción")); xmlFree(prop);
-                                       strcpy(art.presentacion, prop = xmlGetProp(node, "Presentación")); xmlFree(prop);
-                                       strcpy(art.existencia, prop = xmlGetProp(node, "Existencia")); 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, prop = xmlGetProp(node, "PVU")); xmlFree(prop);
-                                       strcpy(art.emin, prop = xmlGetProp(node, "Emín")); xmlFree(prop);
+                                       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) {


@@ -163,9 +167,9 @@ 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;
        if (l == NULL) l = lst_articulos;
        if (l == NULL) return 1;

-       t_Reg_Articulo *del;

 
        emufs_destruir(l->fp);
        while (l->primero) {
 
        emufs_destruir(l->fp);
        while (l->primero) {


@@ -439,12 +443,12 @@ 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, "INI=%s\n", ini);

                        memcpy(dst->presentacion, ini, fin-ini+1);
                        
                        ini = fin+1;
                        fin = ini;
                        while (*fin!='\0') fin++;
                        memcpy(dst->presentacion, ini, fin-ini+1);
                        
                        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;


@@ -487,6 +491,7 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
                        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]);


@@ -523,7 +528,7 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
        PERR("Creo el archivo\n");
        nuevo = emufs_crear("emufs_tmp", tipo, tam_bloque, sizeof(t_Articulo));
        if (nuevo == NULL) {
        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");
+               PERR("ARCHIVO NUEVO NO CREADO");

                return;
        }
 
                return;
        }