* BUGFIX : en emufs_abrir par tipo1 se inicializaba antes de leer el tamaño

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

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 87b0833bf6becae7bacdf02828bfc07346c9040c..0c3f32dd2b55b1ca075b671e8ff0b4500efb428a 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -1,4 +1,4 @@
-/* vim: set noexpandtab tabstop=4 shiftwidth=4:
+/* vim: set noexpandtab tabstop=4 shiftwidth=4 wrap:

  *----------------------------------------------------------------------------
  *                                  emufs
  *----------------------------------------------------------------------------
  *----------------------------------------------------------------------------
  *                                  emufs
  *----------------------------------------------------------------------------


@@ -44,27 +44,64 @@
 #include <string.h>
 #include <unistd.h>
 
 #include <string.h>
 #include <unistd.h>
 

-/*------------------ Funciones privadas ----------------------*/
+#ifndef MIN
+#      define MIN(x, y) (((x) > (y)) ? (y) : (x))
+#endif

 
 

-int emufs_tipo1_header_jump(FILE*);
+/*------------------ Declaraciones privadas ----------------------*/

 
 

-size_t emufs_tipo1_header_size(void);
+/** Cabecera de un registro de un archivo tipo1. */
+typedef struct {
+       EMUFS_REG_ID   id;   /**< Identificador del registro. */
+       EMUFS_REG_SIZE size; /**< Tamaño del registro. */
+} EMUFS_TIPO1_REG_HEADER;

 
 

-int emufs_tipo1_block_jump(EMUFS*, FILE*, EMUFS_BLOCK_ID);
+static size_t emufs_tipo1_header_size(void);

 
 

-void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_REG_ID reg_id,
-               EMUFS_REG_SIZE reg_size, char* reg);
+static int emufs_tipo1_header_jump(FILE*);
+
+static int emufs_tipo1_block_jump(EMUFS*, FILE*, EMUFS_BLOCK_ID);
+
+static void emufs_tipo1_escribir_reg_en_memoria(char*, EMUFS_TIPO1_REG_HEADER,
+               char*);
+
+static void emufs_tipo1_escribir_reg_chunk_en_memoria(char* dst,
+               EMUFS_TIPO1_REG_HEADER header, char* reg, EMUFS_REG_SIZE reg_size);
+
+/** Lee el bloque \param num_bloque y lo almacena en \c ptr. */
+static void* emufs_tipo1_leer_bloque(EMUFS*, EMUFS_BLOCK_ID, int*);
+
+/** Graba el bloque apuntado por \c ptr en el archivo. */
+static EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS*, void*, EMUFS_BLOCK_ID,
+               int*);
+
+/** Obtiene el tamaño del archivo. */
+static long emufs_tipo1_get_file_size(EMUFS*, int*);
+
+/** Guarda un registro con un id determinado */
+static int emufs_tipo1_grabar_registro_con_id(EMUFS* efs, void* reg,
+               EMUFS_TIPO1_REG_HEADER header);

 
 /*------------------ Funciones públicas ----------------------*/
 
 int emufs_tipo1_inicializar(EMUFS* efs)
 {
 
 /*------------------ Funciones públicas ----------------------*/
 
 int emufs_tipo1_inicializar(EMUFS* efs)
 {

+       /* como mínimo el tamaño de bloque debe ser 2 veces el tamaño de la cabecera
+        * (una relación 1/2 entre datos e info de control ya es lo suficientemente
+        * mala */
+       if (efs->tam_bloque < (sizeof(EMUFS_TIPO1_REG_HEADER) * 2)) {
+               PERR("bloque demasiado chico");
+               return 1000; /* EMUFS_ERROR_BLOCK_SIZE_TOO_SMALL */
+       }

        /* Asigna punteros a funciones. */
        /* Asigna punteros a funciones. */

-       efs->leer_bloque     = emufs_tipo1_leer_bloque;
-       efs->leer_registro   = emufs_tipo1_leer_registro;
-       efs->grabar_registro = emufs_tipo1_grabar_registro;
-       /*efs->borrar_registro = emufs_tipo1_borrar_registro;*/
-       return 0;
+       efs->leer_bloque       = emufs_tipo1_leer_bloque;
+       efs->grabar_registro   = emufs_tipo1_grabar_registro;
+       efs->borrar_registro   = emufs_tipo1_borrar_registro;
+       efs->leer_registro     = emufs_tipo1_leer_registro;
+       efs->leer_registro_raw = emufs_tipo1_leer_registro_raw;
+       efs->leer_estadisticas = emufs_tipo1_leer_estadisticas;
+       efs->compactar         = emufs_tipo1_compactar;
+       return 0; /* EMUFS_OK */

 }
 
 void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
 }
 
 void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,


@@ -74,9 +111,7 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
        char* registro; /* registro a leer */
        EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */
        EMUFS_BLOCK_SIZE offset; /* offset del bloque leído */
        char* registro; /* registro a leer */
        EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */
        EMUFS_BLOCK_SIZE offset; /* offset del bloque leído */

-       EMUFS_BLOCK_SIZE block_size; /* tamaño del bloque leído */
-       EMUFS_REG_SIZE curr_reg_size; /* tamaño del registro leído secuencialmente */
-       EMUFS_REG_ID curr_reg_id; /* id del registro leído secuencialmente */
+       EMUFS_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */

 
        block_id = emufs_idx_buscar_registro(efs, reg_id);
        if (block_id == EMUFS_NOT_FOUND) {
 
        block_id = emufs_idx_buscar_registro(efs, reg_id);
        if (block_id == EMUFS_NOT_FOUND) {


@@ -87,21 +122,28 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
        }
        if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, err))) {
                /* TODO Manejo de errores */
        }
        if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, err))) {
                /* TODO Manejo de errores */

-               PERR("no se pudo leer el bloque");
+               PERR("no se pudo reservar memoria");
+               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */

                return NULL;
        }
 
        /* Busco secuencialmente en el bloque el registro a leer */
        offset = 0;
        do {
                return NULL;
        }
 
        /* Busco secuencialmente en el bloque el registro a leer */
        offset = 0;
        do {

-               /* Copio el id del registro de la cabecera. */
-               memcpy(&curr_reg_id, block + offset, sizeof(EMUFS_REG_ID));
-               offset += sizeof(EMUFS_REG_ID);
-               /* Copio el tamaño del registro de la cabecera. */
-               memcpy(&curr_reg_size, block + offset, sizeof(EMUFS_REG_SIZE));
-               offset += sizeof(EMUFS_REG_SIZE);
-               if (curr_reg_id == reg_id) {
-                       registro = (char*) malloc(curr_reg_size);
+               /* Copio la cabecera del registro actual. */
+               memcpy(&curr_reg_header, block + offset, sizeof(EMUFS_TIPO1_REG_HEADER));
+               offset += sizeof(EMUFS_TIPO1_REG_HEADER);
+               if (curr_reg_header.id == reg_id) {
+                       /* tamaño máximo ultilizable para datos en un bloque */
+                       EMUFS_BLOCK_SIZE block_space
+                                       = efs->tam_bloque - sizeof(EMUFS_TIPO1_REG_HEADER);
+                       /* tamaño de la porción de registro que se guarda */
+                       EMUFS_REG_SIZE chunk_size = 0; 
+                       /* puntero a la porción actual del registro */
+                       char* chunk_ptr;
+
+                       *reg_size = curr_reg_header.size;
+                       registro = chunk_ptr = (char*) malloc(*reg_size);

                        if (registro == NULL) {
                                /* TODO Manejo de errores */
                                free(block);
                        if (registro == NULL) {
                                /* TODO Manejo de errores */
                                free(block);


@@ -109,18 +151,76 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
                                *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                                return NULL;
                        }
                                *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                                return NULL;
                        }

-                       memcpy(registro, block + offset, curr_reg_size);
-                       *reg_size = curr_reg_size;
+                       while (1) {
+                               chunk_ptr += chunk_size; /* Avanzo para guardar prox chunk */
+                               curr_reg_header.size -= chunk_size; /* Resto lo que ya guardé */
+                               chunk_size = MIN(curr_reg_header.size, block_space);
+                               /* copio porción de registro en el buffer */
+                               memcpy(chunk_ptr, block + offset, chunk_size);
+                                /* falta leer un bloque */
+                               if (curr_reg_header.size > block_space) {
+                                       free(block);
+                                       if (!(block = (char*) emufs_tipo1_leer_bloque(efs,
+                                                                       ++block_id, err))) {
+                                               /* TODO Manejo de errores */
+                                               free(registro);
+                                               PERR("no se pudo reservar memoria");
+                                               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+                                               return NULL;
+                                       }
+                               } else { /* se terminó de leer */
+                                       break;
+                               }
+                       }

                        break;
                }
                /* Desplazo el offset */
                        break;
                }
                /* Desplazo el offset */

-               offset += curr_reg_size;
-       } while (offset < block_size);
+               offset += curr_reg_header.size;
+
+       /* esto no debería ser nunca false porque sé positivamente que el */
+       } while (offset < efs->tam_bloque); /* registro está en el bloque */

 
        free(block);
        return registro;
 }
 
 
        free(block);
        return registro;
 }
 

+/* @todo TODO hacer que soporte registros de más de un bloque */
+void* emufs_tipo1_leer_registro_raw(EMUFS *efs, EMUFS_REG_ID id,
+               EMUFS_REG_SIZE *size, int *pos)
+{
+       char* block; /* bloque leido (en donde está el registro a leer) */
+       EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */
+       EMUFS_BLOCK_SIZE offset; /* offset del bloque leído */
+       EMUFS_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */
+       int err;
+
+       block_id = emufs_idx_buscar_registro(efs, id);
+       if (block_id == EMUFS_NOT_FOUND) {
+               return NULL;
+       }
+       err = 0;
+       if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, &err))) {
+               return NULL;
+       }
+
+       /* Busco secuencialmente en el bloque el registro a leer */
+       offset = 0;
+       do {
+               /* Copio la cabecera del registro. */
+               memcpy(&curr_reg_header, block + offset, sizeof(EMUFS_TIPO1_REG_HEADER));
+               offset += sizeof(EMUFS_TIPO1_REG_HEADER);
+               if (curr_reg_header.id == id) {
+                       *pos = offset - sizeof(EMUFS_TIPO1_REG_HEADER);
+                       break;
+               }
+               /* Desplazo el offset */
+               offset += curr_reg_header.size;
+       } while (offset < efs->tam_bloque);
+
+       (*size) = efs->tam_bloque;
+       return block;
+}
+

 void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err)
 {
        FILE* file;
 void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err)
 {
        FILE* file;


@@ -156,92 +256,241 @@ void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err)
        return block;
 }
 
        return block;
 }
 

-EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
-               EMUFS_REG_SIZE reg_size, int* err)
+EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg, EMUFS_REG_SIZE reg_size, int* err)

 {
 {

-       EMUFS_REG_ID   reg_id;
-       EMUFS_FREE     fs;
-       EMUFS_BLOCK_ID block_id;
-       char           name_f[255];
-       char*          block;
-       
-       strcpy(name_f,efs->nombre);
-       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+sizeof(EMUFS_REG_ID)+sizeof(EMUFS_REG_SIZE), &fs);
-       /* si no hay bloques con suficiente espacio creo un bloque nuevo */
+       EMUFS_TIPO1_REG_HEADER header; /* cabecera del registro a leer */
+       /* obtengo identificador que corresponderá al registro */
+       header.id = emufs_idx_get_new_id(efs, err);
+       if (*err) {
+               PERR("no se pudo obtener un id para el registro nuevo");
+               return EMUFS_NOT_FOUND;
+       }
+       header.size = reg_size; /* tamaño del registro */
+       if ((*err = emufs_tipo1_grabar_registro_con_id(efs, reg, header))) {
+               PERR("error al grabar el registro");
+               return EMUFS_NOT_FOUND;
+       }
+       return header.id;
+}
+
+int emufs_tipo1_borrar_registro(EMUFS* efs, EMUFS_REG_ID reg_id)
+{
+       char* block; /* bloque leido (en donde está el registro a leer) */
+       EMUFS_BLOCK_ID block_id; /* id del bloque en donde esta el registro a leer */
+       EMUFS_BLOCK_SIZE offset; /* offset del bloque leído */
+       EMUFS_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */
+       int err = 0; /* para almacenar código de error */
+
+       block_id = emufs_idx_buscar_registro(efs, reg_id);

        if (block_id == EMUFS_NOT_FOUND) {
        if (block_id == 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 */
-                       return EMUFS_NOT_FOUND;
-               }
-               /* graba el registro al principio del bloque */
-               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 */
-               block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err);
-               if (*err) {
-                       PERR("error al grabar bloque");
-                       free(block);
-                       return EMUFS_NOT_FOUND;
-               }
-               free(block);
-               /* 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));
-               if (*err) {
-                       PERR("No se pudo agregar fsc");
-                       return EMUFS_NOT_FOUND;
-               }
+               /* TODO Manejo de errores */
+               PERR("Registro no encontrado");
+               return EMUFS_NOT_FOUND;
+       }
+       if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, &err))) {
+               /* TODO Manejo de errores */
+               PERR("no se pudo reservar memoria");
+               return err;
+       }

 
 

-       /* Encontró espacio en un bloque existente, graba registro ahí */
-       } else {
-               /* cargo el bloque en block_id */
-               if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, err))) {
-                       /* TODO Manejo de errores */
-                       PERR("no se pudo leer el bloque");
-                       return EMUFS_NOT_FOUND;
+       /* Busco secuencialmente en el bloque el registro a leer */
+       offset = 0;
+       do {
+               /* Copio la cabecera del registro actual. */
+               memcpy(&curr_reg_header, block + offset, sizeof(EMUFS_TIPO1_REG_HEADER));
+               if (curr_reg_header.id == reg_id) {
+                       /* identificador del bloque actual */
+                       EMUFS_BLOCK_ID curr_block_id = block_id;
+                       /* tamaño máximo ultilizable para datos en un bloque */
+                       EMUFS_BLOCK_SIZE block_space
+                                       = efs->tam_bloque - sizeof(EMUFS_TIPO1_REG_HEADER);
+                       EMUFS_FREE fs; /* cantidad de espacio libre en el bloque */
+
+                       while (1) {
+                               /* actualizo archivo de espacio libre por bloque */
+                               fs = emufs_fsc_get_fs(efs, curr_block_id)
+                                       + MIN(curr_reg_header.size, block_space)
+                                       + sizeof(EMUFS_TIPO1_REG_HEADER);
+                               if ((err = emufs_fsc_actualizar(efs, curr_block_id, fs))) {
+                                       /* TODO Manejo de errores */
+                                       PERR("no se pudo actualizar .fsc");
+                                       free(block);
+                                       return err;
+                               }
+                               /* falta liberar un bloque (o porción) */
+                               if (curr_reg_header.size > block_space) {
+                                       free(block);
+                                       if (!(block = (char*) emufs_tipo1_leer_bloque(efs,
+                                                                       ++curr_block_id, &err))) {
+                                               /* TODO Manejo de errores */
+                                               PERR("no se pudo leer el bloque");
+                                               return err;
+                                       }
+                                       /* copio la cabecera del primer registro (si ocupa más de un
+                                        * registro está en bloques contiguos) */
+                                       memcpy(&curr_reg_header, block,
+                                                       sizeof(EMUFS_TIPO1_REG_HEADER));
+                               } else { /* se terminó de leer */
+                                       break;
+                               }
+                       }
+
+                       /* actualizo archivo de identificadores de registros borrados */
+                       if ((err = emufs_did_agregar(efs, reg_id))) {
+                               /* TODO Manejo de errores */
+                               PERR("no se pudo actualizar .did");
+                               free(block);
+                               return err;
+                       }
+                       /*actualizo archivo .idx*/
+                       if ((err = emufs_idx_borrar(efs, reg_id))) {
+                               /* TODO Manejo de errores */
+                               PERR("no se pudo actualizar .did");
+                               free(block);
+                               return err;
+                       }
+
+                       /* desplazo registros a izquierda */
+                       {   /* offset del fin del registro a borrar */
+                               EMUFS_BLOCK_SIZE offset_reg_end = offset
+                                       + sizeof(EMUFS_TIPO1_REG_HEADER) + curr_reg_header.size;
+                               /* si es necesario desplazar */
+                               if (offset < offset_reg_end) {
+                                       /* muevo la porción de bloque a izquierda */
+                                       memcpy(block + offset, block + offset_reg_end,
+                                               efs->tam_bloque - offset_reg_end);
+                               }
+                       }
+                       /* guardo el bloque en disco */
+                       emufs_tipo1_grabar_bloque(efs, block, curr_block_id, &err);
+                       if (err) {
+                               /* TODO Manejo de errores */
+                               PERR("no se pudo grabar bloque en disco");
+                               free(block);
+                               return err;
+                       }
+
+                       break; /* salgo del loop, ya hice todo lo que tenía que hacer */

                }
                }

-               /* inserta el registro en el bloque */
-               /* tengo que buscar un ID válido para el nuevo registro */
-               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);
-               /* graba el bloque en el archivo */
-               block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err);
-               if (*err) {
-                       PERR("error al grabar bloque");
-                       free(block);
-                       return EMUFS_NOT_FOUND;
+               /* desplazo el offset */
+               offset += sizeof(EMUFS_TIPO1_REG_HEADER) + curr_reg_header.size;
+
+       /* esto no debería ser nunca false porque sé positivamente que el */
+       } while (offset < efs->tam_bloque); /* registro está en el bloque */
+
+       free(block);
+       return 0; /* EMUFS_OK */
+}
+
+EMUFS_Estadisticas emufs_tipo1_leer_estadisticas(EMUFS* efs)
+{
+       int err = 0;
+       EMUFS_Estadisticas stats;
+       memset(&stats, 0, sizeof(EMUFS_Estadisticas));
+
+       stats.tam_archivo_bytes = emufs_tipo1_get_file_size(efs, &err);
+       if (err) {
+               /* TODO manejo de errores */
+               PERR("no se pudo obtener el tamaño del archivo");
+               return stats;
+       }
+
+       /* obtengo cantidad de bloques */
+       stats.cant_bloques = (stats.tam_archivo_bytes - emufs_tipo1_header_size())
+                       / efs->tam_bloque;
+
+       /* obtengo la cantidad de registros en el archivo */
+       {
+               EMUFS_REG_ID *tmp = emufs_idx_get(efs, &stats.tam_archivo);
+               if (tmp) free(tmp); /* libera memoria innecesaria */
+       }
+
+       /* obtengo total de información de control que guarda el archivo */
+       stats.info_control = emufs_tipo1_header_size() /* cabecera del archivo */
+                       /* mas las cabeceras de todos los registros */
+                       + stats.tam_archivo * sizeof(EMUFS_TIPO1_REG_HEADER);
+
+       /* obtengo las estadísticas del archivo de espacio libre por bloque */
+       stats.total_fs = emufs_fsc_get_total_fs(efs);
+       stats.media_fs = emufs_fsc_get_media_fs(efs);
+       emufs_fsc_get_max_min_fs(efs, &stats.min_fs, &stats.max_fs);
+
+       return stats;   
+}
+
+void emufs_tipo1_compactar(EMUFS* efs)
+{
+       EMUFS_REG_ID total_ids; /* cantidad total de registros en el array */
+       /* array con los identificadores de los registros */
+       EMUFS_REG_ID* reg_ids = emufs_idx_get(efs, &total_ids);
+       int i; /* índice de elemento actual del array */
+
+       /* recorro cada registro válido del archivo */
+       for (i = 0; i < total_ids; ++i) {
+               EMUFS_FREE fs; /* espacio libre en el bloque */
+               EMUFS_TIPO1_REG_HEADER header; /* cabecera del registro a leer */
+               int err = 0; /* para almacenar código de error */
+               /* bloque al que pertenece el registro */
+               EMUFS_BLOCK_ID block_id = emufs_idx_buscar_registro(efs, reg_ids[i]);
+               /* obtengo un bloque con espacio suficiente para almacenarlo */
+               EMUFS_BLOCK_ID free_block_id = emufs_fsc_buscar_lugar(efs,
+                               sizeof(EMUFS_TIPO1_REG_HEADER) + reg_ids[i], &fs);
+
+               /* si el registro está borrado, continúo con el próximo */
+               if (block_id == EMUFS_NOT_FOUND) {
+                       continue;

                }
                }

-               free(block);
-               /* actualizo el archivo de espacios libres */
-               *err = emufs_fsc_actualizar(efs, block_id, fs - reg_size
-                               - sizeof(EMUFS_REG_ID) - sizeof(EMUFS_REG_SIZE));
-               if (*err) {
-                       PERR("No se pudo actualizar fsc");
-                       return EMUFS_NOT_FOUND;
+               /* obtengo identificador del registro actual */
+               header.id = reg_ids[i];
+               /* TODO analizar caso de registros multibloque */
+               /* si el bloque obtenido está antes del bloque actual */
+               if (free_block_id < block_id) {
+                       /* leo el registro */
+                       char* reg = (char*) efs->leer_registro(efs, header.id,
+                                       &header.size, &err);
+                       if (err) {
+                               PERR("error al leer registro");
+                               return;
+                       }
+                       /* borro el registro */
+                       if ((err = efs->borrar_registro(efs, header.id))) {
+                               PERR("error al borrar registro");
+                               free(reg);
+                               return;
+                       }
+                       /* lo inserto en la nueva posición */
+                       if ((err = emufs_tipo1_grabar_registro_con_id(efs, reg, header))) {
+                               PERR("error al borrar registro");
+                               free(reg);
+                               return;
+                       }
+                       free(reg);

                }
        }
                }
        }

-               
-       /* actualizo el indice de bloques y registros */
-       *err = emufs_idx_agregar(efs, reg_id, block_id);
-       if (*err){
-               PERR("No se pudo agregar idx");
-               return EMUFS_NOT_FOUND;
+       free(reg_ids); /* libero lista de ids */
+
+       /* truncamos el archivo si hay bloques libres al final */
+       {
+               EMUFS_FREE fs; /* espacio libre en el bloque */
+               /* busco si hay algún bloque completo libre */
+               EMUFS_BLOCK_ID block_id = emufs_fsc_buscar_lugar(efs, efs->tam_bloque
+                               - sizeof(EMUFS_TIPO1_REG_HEADER), &fs);
+               /* si hay, el resto del archivo tiene que estar vacío */
+               if (block_id != EMUFS_NOT_FOUND) {
+                       long size = emufs_tipo1_header_size() /* cabecera del archivo */
+                               + block_id * efs->tam_bloque; /* mas los bloques compactos */
+                       char filename[255];
+
+                       /* trunca archivo de datos */
+                       strcpy(filename, efs->nombre);
+                       strcat(filename, ".dat");
+                       truncate(filename, size);
+                       /* trunca archivo de de espacio libre */
+                       emufs_fsc_truncate(efs, block_id);
+               }

        }
        }

-       
-       return reg_id;

 }
 
 }
 

-/*Graba un bloque en el archivo*/

 EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block,
                EMUFS_BLOCK_ID block_id, int* err)
 {
 EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block,
                EMUFS_BLOCK_ID block_id, int* err)
 {


@@ -296,16 +545,116 @@ EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block,
        return block_id;
 }
 
        return block_id;
 }
 

-/*borra un registro de un bloque y acomoda los registros que quedan*/
-int emufs_tipo1_buscar_registro(EMUFS *emu, EMUFS_REG_ID id_reg)
+static int emufs_tipo1_grabar_registro_con_id(EMUFS* efs, void* reg,
+               EMUFS_TIPO1_REG_HEADER header)

 {
 {

-       return -1; /* FIXME Error */
+       EMUFS_FREE     fs; /* espacio libre en el bloque */
+       EMUFS_BLOCK_ID block_id; /* identificador del 1er bloque */
+       char*          block; /* buffer del bloque a guardar en disco */
+       char           name_f[255];
+       int            err = 0;
+
+       strcpy(name_f, efs->nombre);
+       strcat(name_f, ".dat");
+
+       /* busco lugar para el registro en un bloque existente */
+       block_id = emufs_fsc_buscar_lugar(efs, sizeof(EMUFS_TIPO1_REG_HEADER)
+                       + header.size, &fs);
+       /* si no hay bloques con suficiente espacio creo un bloque nuevo */
+       if (block_id == EMUFS_NOT_FOUND) {
+               /* tamaño máximo ultilizable para datos en un bloque */
+               EMUFS_BLOCK_SIZE block_space = efs->tam_bloque - sizeof(EMUFS_TIPO1_REG_HEADER);
+               /* identificador del bloque que se guarda */
+               EMUFS_BLOCK_ID curr_block_id = EMUFS_NOT_FOUND;
+               /* tamaño de la porción de registro que se guarda */
+               EMUFS_REG_SIZE chunk_size = 0; 
+               /* puntero a la poción del registro */
+               char* chunk_ptr = reg; 
+
+               /* crear un nuevo bloque en memoria */
+               block = (char*) malloc(efs->tam_bloque);
+               if (block == NULL) {
+                       /* TODO Manejo de errores */
+                       PERR("No hay memoria");
+                       return 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
+               }
+               do {
+                       memset(block, 0, efs->tam_bloque); /* inicializa bloque */
+                       chunk_ptr += chunk_size; /* Avanzo para guardar prox chunk */
+                       header.size -= chunk_size; /* Resto lo que ya guardé */
+                       chunk_size = MIN(header.size, block_space);
+                       /* graba porción del registro en bloque */
+                       emufs_tipo1_escribir_reg_chunk_en_memoria(block, header, chunk_ptr, chunk_size);
+                       /* graba el bloque en el archivo */
+                       curr_block_id = emufs_tipo1_grabar_bloque(efs, block, EMUFS_NOT_FOUND, &err);
+                       if (err) {
+                               PERR("error al grabar bloque");
+                               free(block);
+                               return err;
+                       }
+                       /* grabo el nuevo registro en el archivo de espacios libres */
+                       err = emufs_fsc_agregar(efs, curr_block_id, block_space - chunk_size);
+                       if (err) {
+                               PERR("No se pudo agregar fsc");
+                               free(block);
+                               return err;
+                       }
+                       /* si es el primer id de bloque obtenido, lo guardo para
+                        * agregarlo después al archivo de índices. */
+                       if (block_id == EMUFS_NOT_FOUND) {
+                               block_id = curr_block_id;
+                       }
+               } while (header.size > block_space);
+               free(block);
+
+       /* Encontró espacio en un bloque existente, graba registro ahí */
+       } else {
+               /* cargo el bloque en block_id */
+               if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, &err))) {
+                       /* TODO Manejo de errores */
+                       PERR("no se pudo leer el bloque");
+                       return err;
+               }
+               /* graba registro en bloque */
+               emufs_tipo1_escribir_reg_en_memoria(block + efs->tam_bloque - fs,
+                               header, reg);
+               /* graba el bloque en el archivo */
+               block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, &err);
+               if (err) {
+                       PERR("error al grabar bloque");
+                       free(block);
+                       return err;
+               }
+               free(block);
+               /* actualizo el archivo de espacios libres */
+               err = emufs_fsc_actualizar(efs, block_id, fs - header.size
+                               - sizeof(EMUFS_TIPO1_REG_HEADER));
+               if (err) {
+                       PERR("No se pudo actualizar fsc");
+                       return err;
+               }
+       }
+               
+       /* actualizo el indice de bloques y registros */
+       err = emufs_idx_agregar(efs, header.id, block_id);
+       if (err){
+               PERR("No se pudo agregar idx");
+               return err;
+       }
+       
+       return err;
+}
+
+EMUFS_REG_ID emufs_tipo1_modificar_registro(EMUFS *emu, EMUFS_REG_ID id,
+               void *data, EMUFS_REG_SIZE size, int *error)
+{
+       emufs_tipo1_borrar_registro(emu, id);
+       return emufs_tipo1_grabar_registro(emu, data, size, error);

 }
 
 }
 

-int emufs_tipo1_borrar_registro(EMUFS *emu, EMUFS_REG_ID id_reg,
-               EMUFS_REG_SIZE tam_reg)
+size_t emufs_tipo1_header_size(void)

 {
 {

-       return -1; /* FIXME Error */
+       return sizeof(EMUFS_Tipo) + sizeof(EMUFS_BLOCK_SIZE);

 }
 
 int emufs_tipo1_header_jump(FILE* fp)
 }
 
 int emufs_tipo1_header_jump(FILE* fp)


@@ -326,22 +675,39 @@ int emufs_tipo1_block_jump(EMUFS* efs, FILE* fp, EMUFS_BLOCK_ID block_count)
        return 0; /* EMUFS_OK */
 }
 
        return 0; /* EMUFS_OK */
 }
 

-size_t emufs_tipo1_header_size(void)
+void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_TIPO1_REG_HEADER header,
+               char* reg)

 {
 {

-       return sizeof(EMUFS_TYPE) +      /* Cabecera de tipo de archivo */
-              sizeof(EMUFS_BLOCK_SIZE); /* Cabecera de tamaño del bloque */
+       emufs_tipo1_escribir_reg_chunk_en_memoria(dst, header, reg, header.size);

 }
 
 }
 

-void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_REG_ID reg_id,
-               EMUFS_REG_SIZE reg_size, char* reg) {
-       /* grabo el id en el bloque */
-       memcpy(dst, &reg_id, sizeof(EMUFS_REG_ID));
-       /* incremento puntero de escritura */
-       dst += sizeof(EMUFS_REG_ID);
-       /* grabo el tamaño del registro en el bloque */
-       memcpy(dst, &reg_size, sizeof(EMUFS_REG_SIZE));
+void emufs_tipo1_escribir_reg_chunk_en_memoria(char* dst,
+               EMUFS_TIPO1_REG_HEADER header, char* reg, EMUFS_REG_SIZE reg_size)
+{
+       /* grabo cabecera del registro en el bloque */
+       memcpy(dst, &header, sizeof(EMUFS_TIPO1_REG_HEADER));

        /* incremento puntero de escritura */
        /* incremento puntero de escritura */

-       dst += sizeof(EMUFS_REG_SIZE);
+       dst += sizeof(EMUFS_TIPO1_REG_HEADER);

        /* grabo el registro en el bloque */
        memcpy(dst, reg, reg_size);
 }
        /* grabo el registro en el bloque */
        memcpy(dst, reg, reg_size);
 }

+
+long emufs_tipo1_get_file_size(EMUFS* efs, int* err)
+{
+       long  file_size;
+       FILE* file;
+       char  name_f[255];
+
+       strcpy(name_f, efs->nombre);
+       strcat(name_f, ".dat");
+       if ((file = fopen(name_f, "ab")) == NULL) {
+               /* TODO Manejo de errores */
+               PERR("Error al abrir archivo");
+               *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
+               return 0;
+       }
+       file_size = ftell(file);
+       fclose(file);
+       return file_size;
+}
+