From: Leandro Lucarella Date: Sun, 18 Apr 2004 00:44:14 +0000 (+0000) Subject: Se arreglan un par de includes y memory leaks. Me queda por encontrar uno. X-Git-Tag: svn_import_r684~404 X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/commitdiff_plain/28ec60bde7175661afed582949f32390ebafaeaa?ds=inline;hp=771d5d90ad3fa04217d2f517a79bea62f2c4e9e1 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 82fe63d..663feef 100644 --- a/emufs/did.c +++ b/emufs/did.c @@ -37,8 +37,9 @@ */ #include "did.h" -#include #include +#include +#include int emufs_did_crear(EMUFS* efs) { diff --git a/emufs/fsc.c b/emufs/fsc.c index 9063c81..465d19f 100644 --- a/emufs/fsc.c +++ b/emufs/fsc.c @@ -36,8 +36,9 @@ */ #include "fsc.h" -#include #include +#include +#include /* 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 eeaef09..5812bfe 100644 --- a/emufs/tipo1.c +++ b/emufs/tipo1.c @@ -39,11 +39,12 @@ #include "idx.h" #include "fsc.h" #include "did.h" +#include +#include #include #include #include #include -#include #ifndef MIN # define MIN(x, y) (((x) > (y)) ? (y) : (x)) diff --git a/emufs/tipo2.c b/emufs/tipo2.c index 10867d6..87dd90e 100644 --- a/emufs/tipo2.c +++ b/emufs/tipo2.c @@ -41,6 +41,10 @@ #include "idx.h" #include "fsc.h" #include "did.h" +#include +#include +#include +#include /* 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 049c63d..97752e7 100644 --- a/emufs/tipo3.c +++ b/emufs/tipo3.c @@ -36,6 +36,10 @@ */ #include "tipo3.h" +#include +#include +#include +#include /** 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_bloquetam_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 f3563b2..feff81f 100644 --- 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], ®_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], ®_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], ®_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]); }