Se arreglan un par de includes y memory leaks. Me queda por encontrar uno.

diff --git a/emufs/did.c b/emufs/did.c
index 82fe63d9c61d614b6c0d6464cd140ac7c7b2e2a5..663feef0d46db776006be096956182d8cc698d7a 100644 (file)
--- a/emufs/did.c
+++ b/emufs/did.c
@@ -37,8 +37,9 @@
  */
 
 #include "did.h"
-#include <string.h>
 #include <unistd.h>
+#include <sys/types.h>
+#include <string.h>
 
 int emufs_did_crear(EMUFS* efs)
 {

diff --git a/emufs/fsc.c b/emufs/fsc.c
index 9063c81ebdbdd78f86377bf166fed803443bcda7..465d19fe9383e7a31f579d5d6130b3328a4fb59d 100644 (file)
--- a/emufs/fsc.c
+++ b/emufs/fsc.c
@@ -36,8 +36,9 @@
  */
 
 #include "fsc.h"
-#include <string.h>
 #include <unistd.h>
+#include <sys/types.h>
+#include <string.h>
 
 /* Crea un archivo de Gaps o Espacio Libre en Bloque */
 int emufs_fsc_crear(EMUFS* efs)

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index eeaef09e97908f73ac4135f6ef775fb09cf10fa3..5812bfe69302dbef5262bab6d1011492f89e7f70 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -39,11 +39,12 @@
 #include "idx.h"
 #include "fsc.h"
 #include "did.h"
+#include <unistd.h>
+#include <sys/types.h>
 #include <stdio.h>
 #include <math.h>
 #include <stdlib.h>
 #include <string.h>
-#include <unistd.h>
 
 #ifndef MIN
 #      define MIN(x, y) (((x) > (y)) ? (y) : (x))

diff --git a/emufs/tipo2.c b/emufs/tipo2.c
index 10867d678c7a89625d050d0e11a095316e8e67e6..87dd90e57749a0d7b67022dba97bef8baa6e92d5 100644 (file)
--- a/emufs/tipo2.c
+++ b/emufs/tipo2.c
@@ -41,6 +41,10 @@
 #include "idx.h"
 #include "fsc.h"
 #include "did.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)

diff --git a/emufs/tipo3.c b/emufs/tipo3.c
index 049c63d6bf85eb0ac1ed10c243109064ba89e0a2..97752e75ad4b8340a0208a48f5561babeb1d6215 100644 (file)
--- a/emufs/tipo3.c
+++ b/emufs/tipo3.c
@@ -36,6 +36,10 @@
  */
 
 #include "tipo3.h"
+#include <unistd.h>
+#include <sys/types.h>
+#include <stdio.h>
+#include <string.h>
 
 /** Leo un registro del archivo, devuelve cero si no lo encuentra.**/
 void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
@@ -61,7 +65,6 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
        registro = (char*) malloc(emu->tam_reg);
        if (registro == NULL) {
                /* TODO Manejo de errores */
-               free(bloque);
                PERR("No hay memoria");
                *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                return NULL;
@@ -73,6 +76,7 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
                        /* TODO Manejo de errores, queda en el codigo de error lo que devolvio
                         * emufs_tipo3_leer_bloque() */
                        PERR("no se pudo leer el bloque");
+                       free(registro);
                        return NULL; /*No se pudo leer el bloque*/
                }
                ID_aux = -1;
@@ -96,7 +100,7 @@ void* emufs_tipo3_leer_registro(EMUFS *emu, EMUFS_REG_ID ID,
                }
                free(bloque);
        }
-       
+
        return registro;
 }
 
@@ -150,7 +154,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
        EMUFS_BLOCK_SIZE cant;
        FILE *file;
        char name_f[255];
-       char* bloque;
+       char* bloque = NULL;
        int cant_bloques, resto, i=0, lugar;
        
        strcpy(name_f,emu->nombre);
@@ -244,6 +248,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                        /*grabo el bloque en el archivo*/
                        if ( emufs_tipo3_grabar_bloque(emu, bloque, num_bloque+i) != 0) {
                                PERR("error al grabar bloque");
+                               if (bloque) free(bloque);
                                return -1; /* se produjo un error */    
                        }
                        
@@ -252,7 +257,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                                resto = emu->tam_reg;
                                if ( emufs_fsc_agregar(emu, num_bloque, fs - resto - sizeof(EMUFS_REG_ID) ) != 0 ) {
                                        fclose(file);
-                                       if (bloque != NULL) free(bloque);
+                                       if (bloque) free(bloque);
                                        return -1;
                                }
                        } else {        
@@ -260,19 +265,19 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t
                                        resto = emu->tam_reg - i*(emu->tam_bloque - sizeof(EMUFS_REG_ID));
                                if ( emufs_fsc_agregar(emu, num_bloque+i, fs-resto) !=0 ){
                                        fclose(file);
-                                       if (bloque != NULL) free(bloque);
+                                       if (bloque) free(bloque);
                                        return -1;
                                }
                        }
                        if ( i == 0 ){
                                if ( emufs_idx_agregar(emu, ID_aux, num_bloque) != 0 ){
-                                       if (bloque != NULL) free(bloque);
+                                       if (bloque) free(bloque);
                                        return -1;
                                }
                        }
                }
        }
-       free(bloque);
+       if (bloque) free(bloque);
        return ID_aux;
 }
 
@@ -507,6 +512,7 @@ void emufs_tipo3_compactar(EMUFS *emu)
                }
                emufs_tipo3_borrar_registro(emu, i);
                ID_aux = emufs_tipo3_grabar_registro(emu, reg, emu->tam_reg, &err);
+               free(reg);
                i++;
        }
        /*tengo que truncar el archivo*/
@@ -520,5 +526,4 @@ void emufs_tipo3_compactar(EMUFS *emu)
        if(emu->tam_bloque<emu->tam_reg-sizeof(EMUFS_REG_ID)) block_id = block_id/2;
        if (emufs_fsc_truncate(emu, block_id)!= 0)
                PERR("NO TURNQUE EL FSC");
-       free(reg);
 }

diff --git a/emufs/tipo3_main.c b/emufs/tipo3_main.c
index f3563b2fe4c3939ce66d09bf0057ae69f0dad286..feff81fdd27d7bc81246c611eb62c535f37705b4 100644 (file)
--- a/emufs/tipo3_main.c
+++ b/emufs/tipo3_main.c
@@ -95,6 +95,7 @@ int main(int argc, char *argv[])
        for (j=0; j<8; j++){    
                b_ptr = fp->leer_registro(fp, v[j], &reg_size, &err);
                printf("Recuperado : %s\n", b_ptr);
+               free(b_ptr);
        }
        ver_archivo_FS(fp);     
        printf("BORRANDO REGISTROS....\n");
@@ -106,6 +107,7 @@ int main(int argc, char *argv[])
        for (j=1; j<8; j+=2){   
                b_ptr = fp->leer_registro(fp, v[j], &reg_size, &err);
                printf("Recuperado : %s\n", b_ptr);
+               free(b_ptr);
        }
 
        printf("COMPACTANDO........\n");
@@ -114,6 +116,7 @@ ver_archivo_FS(fp);
        for (j=1; j<8; j+=2){   
                b_ptr = fp->leer_registro(fp, v[j], &reg_size, &err);
                printf("Recuperado : %s\n", b_ptr);
+               free(b_ptr);
        }
 
        printf("GRABANDO REGISTROS....\n");
@@ -132,7 +135,7 @@ ver_archivo_FS(fp);
 
 
        printf("BORRANDO REGISTROS....\n");
-       for (j=0; j<16; j+=3){  
+       for (j=0; j<8; j+=3){
                fp->borrar_registro(fp, v[j]);
                printf("borrado : %lu\n", v[j]);
        }