Se generaliza el algoritmo de leer_registro(). Ahora, al igual que

grabar_registro() no necesita tratar distinto un registro que ocupa mas de un
bloque.

diff --git a/emufs/tipo1.c b/emufs/tipo1.c
index 827571db50dd835af93e39ead3175f08aa89d53d..a547479685b238a08539f317cc35188f7bc99086 100644 (file)
--- a/emufs/tipo1.c
+++ b/emufs/tipo1.c
@@ -114,8 +114,13 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_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 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;
                        *reg_size = curr_reg_header.size;

-                       registro = (char*) malloc(*reg_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);


@@ -123,37 +128,26 @@ 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;
                        }

-                       /* Si el registro ocupa más de un bloque */
-                       if (*reg_size > block_space) {
-                               /* TODO */
-                               /* 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 = registro;
-
-                               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;
+                       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;

                                }
                                }

-                       } else {
-                               memcpy(registro, block + offset, *reg_size);

                        }
                        break;
                }
                        }
                        break;
                }