* agrego mas fclose faltantes

diff --git a/emufs/emufs.c b/emufs/emufs.c
index 734c936609c2fa1be718d258e0f663b29d58c136..d24386107c17ac12a463e86fdd8ec902246836c9 100644 (file)
--- a/emufs/emufs.c
+++ b/emufs/emufs.c
@@ -119,6 +119,7 @@ EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_b
        /* Crea archivo de índice. */
        if (emufs_idx_crear(efs)) {
                /* TODO ERROR */
        /* Crea archivo de índice. */
        if (emufs_idx_crear(efs)) {
                /* TODO ERROR */

+               fclose(fp);

                free(efs->nombre);
                free(efs);
                return NULL;
                free(efs->nombre);
                free(efs);
                return NULL;


@@ -127,6 +128,7 @@ EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_b
        /* Crea archivo de control de espacio libre. */
        if (emufs_fsc_crear(efs)) {
                /* TODO ERROR */
        /* Crea archivo de control de espacio libre. */
        if (emufs_fsc_crear(efs)) {
                /* TODO ERROR */

+               fclose(fp);

                free(efs->nombre);
                free(efs);
                return NULL;
                free(efs->nombre);
                free(efs);
                return NULL;


@@ -135,6 +137,7 @@ EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_b
        /* Crea archivo de identificadores borrados (recuperables). */
        if (emufs_did_crear(efs)) {
                /* TODO ERROR */
        /* Crea archivo de identificadores borrados (recuperables). */
        if (emufs_did_crear(efs)) {
                /* TODO ERROR */

+               fclose(fp);

                free(efs->nombre);
                free(efs);
                return NULL;
                free(efs->nombre);
                free(efs);
                return NULL;


@@ -148,6 +151,7 @@ EMUFS *emufs_crear(const char *filename, EMUFS_Tipo tipo, EMUFS_BLOCK_SIZE tam_b
                        if ((err = emufs_tipo1_inicializar(efs))) {
                                /* TODO ERROR */
                                PERR("No se pudo inicializar el EMUFS de tipo1");
                        if ((err = emufs_tipo1_inicializar(efs))) {
                                /* TODO ERROR */
                                PERR("No se pudo inicializar el EMUFS de tipo1");

+                               fclose(fp);

                                free(efs->nombre);
                                free(efs);
                                return NULL;
                                free(efs->nombre);
                                free(efs);
                                return NULL;

diff --git a/emufs/idx.c b/emufs/idx.c
index 06cb8246b2d2b95763528d0bb4c5106f7ff57640..6e33fda41ed442fd5e803a91b8de93cb07a374af 100644 (file)
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -52,7 +52,6 @@ FILE* emufs_idx_abrir(EMUFS* efs, const char* mode)
        filename = (char*) malloc(sizeof(char) * (strlen(efs->nombre)
                                + strlen(EMUFS_IDX_EXT) + 1));
        if (filename == NULL) {
        filename = (char*) malloc(sizeof(char) * (strlen(efs->nombre)
                                + strlen(EMUFS_IDX_EXT) + 1));
        if (filename == NULL) {

-               /* TODO Manejo de errores */

                return NULL;
        }
        strcpy(filename, efs->nombre);
                return NULL;
        }
        strcpy(filename, efs->nombre);

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index a8b5102fdb784c2de1a571430616ee91f821369a..e7033268ca151c059c6cf175bb3079fcadbec8f4 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -240,10 +240,12 @@ void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int* err)
        if (block == NULL) {
                PERR("No hay memoria");
                *err = EMUFS_ERROR_OUT_OF_MEMORY;
        if (block == NULL) {
                PERR("No hay memoria");
                *err = EMUFS_ERROR_OUT_OF_MEMORY;

+               fclose(file);

                return NULL;
        }
        if (fread(block, efs->tam_bloque, 1, file) != 1) {
                free(block);
                return NULL;
        }
        if (fread(block, efs->tam_bloque, 1, file) != 1) {
                free(block);

+               fclose(file);

                PERR("Error al leer bloque");
                *err = EMUFS_ERROR_FILE_READ;
                return NULL;
                PERR("Error al leer bloque");
                *err = EMUFS_ERROR_FILE_READ;
                return NULL;

diff --git a/emufs/tipo2.c b/emufs/tipo2.c
index 1858c75f30b12b4e75152d486aa9387f1aa7c19e..b2724e931d88667126b1ed8148322e07f97074d4 100644 (file)
--- a/emufs/tipo2.c
+++ b/emufs/tipo2.c
@@ -318,6 +318,7 @@ void emufs_tipo2_compactar(EMUFS *efs)
        /* Obtengo la cantidad de gaps */
        if ( (fscfile = fopen(name_ffsc,"rb")) == NULL){
                        PERR("No se pudo abrir el archivo");
        /* Obtengo la cantidad de gaps */
        if ( (fscfile = fopen(name_ffsc,"rb")) == NULL){
                        PERR("No se pudo abrir el archivo");

+                       fclose(datfile);

                        return;
        }
        fseek(fscfile,0,SEEK_END);
                        return;
        }
        fseek(fscfile,0,SEEK_END);