* Bugfix en idx_get para saltar los registros no utilizados

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

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 8bccc4ce03bf2a1d8b40f14150e9aa9d8f77938e..a547479685b238a08539f317cc35188f7bc99086 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -88,7 +88,6 @@ 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 */
-       EMUFS_BLOCK_SIZE block_size; /* tamaño del bloque leído */
        EMUFS_TIPO1_REG_HEADER curr_reg_header; /* cabecera del registro a leer */
 
        block_id = emufs_idx_buscar_registro(efs, reg_id);
@@ -100,7 +99,8 @@ 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 */
-               PERR("no se pudo leer el bloque");
+               PERR("no se pudo reservar memoria");
+               *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                return NULL;
        }
 
@@ -111,7 +111,16 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
                memcpy(&curr_reg_header, block + offset, sizeof(EMUFS_TIPO1_REG_HEADER));
                offset += sizeof(EMUFS_TIPO1_REG_HEADER);
                if (curr_reg_header.id == reg_id) {
-                       registro = (char*) malloc(curr_reg_header.size);
+                       /* 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);
@@ -119,13 +128,32 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id,
                                *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */
                                return NULL;
                        }
-                       memcpy(registro, block + offset, curr_reg_header.size);
-                       *reg_size = curr_reg_header.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 */
                offset += curr_reg_header.size;
-       } while (offset < block_size);
+       } while (offset < efs->tam_bloque);
 
        free(block);
        return registro;
@@ -223,7 +251,7 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
        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);
+                               = 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 */
@@ -257,9 +285,10 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                                return EMUFS_NOT_FOUND;
                        }
                        /* grabo el nuevo registro en el archivo de espacios libres */
-                       *err = emufs_fsc_agregar(efs, block_id, block_space - chunk_size);
+                       *err = emufs_fsc_agregar(efs, curr_block_id, block_space - chunk_size);
                        if (*err) {
                                PERR("No se pudo agregar fsc");
+                               free(block);
                                return EMUFS_NOT_FOUND;
                        }
                        /* si es el primer id de bloque obtenido, lo guardo para
@@ -267,7 +296,7 @@ EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg,
                        if (block_id == EMUFS_NOT_FOUND) {
                                block_id = curr_block_id;
                        }
-               } while (chunk_size > block_space);
+               } while (reg_header.size > block_space);
                free(block);
 
        /* Encontró espacio en un bloque existente, graba registro ahí */