* BUGFIX: en tipo1 grabar_registro cuando se pregunta por espacio libre no se esta...

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

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 2331da1d6bef485f3e30fb0f551fbb17bc82fcb5..87b0833bf6becae7bacdf02828bfc07346c9040c 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -44,8 +44,6 @@
 #include <string.h>
 #include <unistd.h>
 
-#define PERR(msg) printf("%s:%d> %s.\n",__FILE__, __LINE__, msg);
-
 /*------------------ Funciones privadas ----------------------*/
 
 int emufs_tipo1_header_jump(FILE*);
@@ -69,7 +67,8 @@ int emufs_tipo1_inicializar(EMUFS* efs)
        return 0;
 }
 
-void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id, int *err)
+void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
+               EMUFS_REG_SIZE* reg_size, int *err)
 {
        char* block; /* bloque leido (en donde está el registro a leer) */
        char* registro; /* registro a leer */
@@ -111,6 +110,7 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id, int *err)
                                return NULL;
                        }
                        memcpy(registro, block + offset, curr_reg_size);
+                       *reg_size = curr_reg_size;
                        break;
                }
                /* Desplazo el offset */
@@ -162,7 +162,6 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
        EMUFS_REG_ID   reg_id;
        EMUFS_FREE     fs;
        EMUFS_BLOCK_ID block_id;
-       FILE*          file;
        char           name_f[255];
        char*          block;
        
@@ -170,26 +169,19 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
        strcat(name_f,".dat");
        
        /* me devuelve el ID del bloque donde quepa un registro y el espacio libre en "fs"*/
-       block_id = emufs_fsc_buscar_lugar(efs, reg_size, &fs);
+       block_id = emufs_fsc_buscar_lugar(efs, reg_size+sizeof(EMUFS_REG_ID)+sizeof(EMUFS_REG_SIZE), &fs);
        /* si no hay bloques con suficiente espacio creo un bloque nuevo */
        if (block_id == EMUFS_NOT_FOUND) {
-               if ((file = fopen(name_f, "a+b")) == NULL) {
-                       /* TODO Manejo de errores */
-                       PERR("Error al abrir archivo");
-                       *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
-                       return EMUFS_NOT_FOUND;
-               }
                /* crear un nuevo bloque en memoria */
                block = (char*) malloc(efs->tam_bloque);
                if (block == NULL) {
                        /* TODO Manejo de errores */
                        PERR("No hay memoria");
                        *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
-                       fclose(file);
                        return EMUFS_NOT_FOUND;
                }
                /* graba el registro al principio del bloque */
-               reg_id = emufs_tipo1_get_id(efs);
+               reg_id = emufs_idx_get_new_id(efs, err);
                /* graba registro en bloque */
                emufs_tipo1_escribir_reg_en_memoria(block, reg_id, reg_size, reg);
                /* graba el bloque en el archivo */
@@ -200,7 +192,6 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                        return EMUFS_NOT_FOUND;
                }
                free(block);
-               fclose(file);
                /* grabo el nuevo registro en el archivo de espacios libres */
                *err = emufs_fsc_agregar(efs, block_id, efs->tam_bloque - reg_size
                        - sizeof(EMUFS_REG_ID) - sizeof(EMUFS_REG_SIZE));
@@ -219,7 +210,7 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                }
                /* inserta el registro en el bloque */
                /* tengo que buscar un ID válido para el nuevo registro */
-               reg_id = emufs_tipo1_get_id(efs);
+               reg_id = emufs_idx_get_new_id(efs, err);
                /* graba registro en bloque */
                emufs_tipo1_escribir_reg_en_memoria(block + efs->tam_bloque - fs,
                                reg_id, reg_size, reg);
@@ -241,7 +232,7 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
        }
                
        /* actualizo el indice de bloques y registros */
-       *err = emufs_idx_agregar(efs, block_id, reg_id);
+       *err = emufs_idx_agregar(efs, reg_id, block_id);
        if (*err){
                PERR("No se pudo agregar idx");
                return EMUFS_NOT_FOUND;
@@ -305,12 +296,6 @@ EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block,
        return block_id;
 }
 
-/*Busco en el archivo de Id`s un Id valido para un nuevo registro*/
-EMUFS_REG_ID emufs_tipo1_get_id(EMUFS *emu)
-{
-       return -1; /* FIXME Error */
-}
-
 /*borra un registro de un bloque y acomoda los registros que quedan*/
 int emufs_tipo1_buscar_registro(EMUFS *emu, EMUFS_REG_ID id_reg)
 {
@@ -325,7 +310,7 @@ int emufs_tipo1_borrar_registro(EMUFS *emu, EMUFS_REG_ID id_reg,
 
 int emufs_tipo1_header_jump(FILE* fp)
 {
-       if (fseek(fp, 0l, SEEK_END)) {
+       if (fseek(fp, emufs_tipo1_header_size(), SEEK_CUR)) {
                PERR("No se pudo hacer fseek()");
                return 8; /* EMUFS_ERROR_SEEK_FILE */
        }