Se arregla memory leak (aunque en el ejemplo no importe demasiado :)

[z.facultad/75.06/emufs.git] / emufs / tipo2.c

diff --git a/emufs/tipo2.c b/emufs/tipo2.c
index 9f28dc687a53d74ea625d35a9cd90a23eca4367e..92cb2a4a61347f0888dede0f2b46a184d345d958 100644 (file)
--- a/emufs/tipo2.c
+++ b/emufs/tipo2.c
@@ -41,6 +41,11 @@
 #include "idx.h"
 #include "fsc.h"
 #include "did.h"
 #include "idx.h"
 #include "fsc.h"
 #include "did.h"

+#include "error.h"
+#include <unistd.h>
+#include <sys/types.h>
+#include <stdio.h>
+#include <string.h>

 
 /* Asigna los punteros a las funciones apropiadas para el Tipo2 */
 int emufs_tipo2_inicializar(EMUFS* efs)
 
 /* Asigna los punteros a las funciones apropiadas para el Tipo2 */
 int emufs_tipo2_inicializar(EMUFS* efs)


@@ -69,7 +74,6 @@ void *emufs_tipo2_leer_registro(EMUFS* efs, EMUFS_REG_ID id_reg, EMUFS_REG_SIZE*
        /* Obtenemos la posicion del registro en el .dat */
        reg_offset = emufs_idx_buscar_registro(efs, id_reg);
        if (reg_offset == EMUFS_NOT_FOUND) {
        /* Obtenemos la posicion del registro en el .dat */
        reg_offset = emufs_idx_buscar_registro(efs, id_reg);
        if (reg_offset == EMUFS_NOT_FOUND) {

-               /* TODO Manejo de errores */

                PERR("Registro no encontrado");
                *err = EMUFS_NOT_FOUND;
                return NULL;
                PERR("Registro no encontrado");
                *err = EMUFS_NOT_FOUND;
                return NULL;


@@ -78,8 +82,8 @@ void *emufs_tipo2_leer_registro(EMUFS* efs, EMUFS_REG_ID id_reg, EMUFS_REG_SIZE*
        /* Levantamos el registro */
        if ((f_data = fopen(name_f, "rb")) == NULL) {
                PERR("No se puede abrir archivo");
        /* Levantamos el registro */
        if ((f_data = fopen(name_f, "rb")) == NULL) {
                PERR("No se puede abrir archivo");

-               *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
-               return NULL; /* FIXME ERROR */
+               *err = EMUFS_ERROR_CANT_OPEN_FILE;
+               return NULL;

        }
        fseek(f_data,reg_offset+sizeof(EMUFS_REG_ID),0);
        fread(reg_size,sizeof(EMUFS_REG_SIZE),1,f_data);
        }
        fseek(f_data,reg_offset+sizeof(EMUFS_REG_ID),0);
        fread(reg_size,sizeof(EMUFS_REG_SIZE),1,f_data);


@@ -210,8 +214,7 @@ int emufs_tipo2_dummyfill(EMUFS *efs, EMUFS_OFFSET reg_pos, EMUFS_REG_SIZE amoun
        FILE *f_data;
        char name_f[255];
        char *dummyfill;
        FILE *f_data;
        char name_f[255];
        char *dummyfill;

-       char *ptr_cur;
-       unsigned long fill_size,byte_count;
+       unsigned long fill_size;

        
        /* Armamos el filename del archivo de datos */
        strcpy(name_f,efs->nombre);
        
        /* Armamos el filename del archivo de datos */
        strcpy(name_f,efs->nombre);


@@ -223,8 +226,6 @@ int emufs_tipo2_dummyfill(EMUFS *efs, EMUFS_OFFSET reg_pos, EMUFS_REG_SIZE amoun
        fill_size = amount+sizeof(EMUFS_REG_ID)+sizeof(EMUFS_REG_SIZE);
        dummyfill = (char*)malloc(fill_size);
        memset(dummyfill, 0, fill_size);
        fill_size = amount+sizeof(EMUFS_REG_ID)+sizeof(EMUFS_REG_SIZE);
        dummyfill = (char*)malloc(fill_size);
        memset(dummyfill, 0, fill_size);

-       ptr_cur = dummyfill;
-       for (byte_count = 0; byte_count < fill_size; ++byte_count) memcpy(ptr_cur+byte_count,0,1);

        fseek(f_data,reg_pos,SEEK_SET);
        fwrite(dummyfill,fill_size,1,f_data);
        fclose(f_data);
        fseek(f_data,reg_pos,SEEK_SET);
        fwrite(dummyfill,fill_size,1,f_data);
        fclose(f_data);


@@ -314,7 +315,7 @@ void emufs_tipo2_compactar(EMUFS *efs)
        EMUFS_FSC reg1,reg2;
        unsigned long cant_gaps = 0,mustmove_bytes = 0,source = 0,
                                  destination = 0,datsize = 0,totalfsc = 0;
        EMUFS_FSC reg1,reg2;
        unsigned long cant_gaps = 0,mustmove_bytes = 0,source = 0,
                                  destination = 0,datsize = 0,totalfsc = 0;

-       
+               

        strcpy(name_fdat,efs->nombre);
        strcpy(name_ffsc,efs->nombre);
        strcat(name_fdat,".dat");
        strcpy(name_fdat,efs->nombre);
        strcpy(name_ffsc,efs->nombre);
        strcat(name_fdat,".dat");