* BUGFIX : Luca olvide inicializar los errores molestos que quiso poner :-)

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

diff --git a/emufs/idx.c b/emufs/idx.c
index 6cf562ee7f44a96375d07a14652591666417414a..ce12761c29d83c4922b7f13a28345fdc25d303cb 100644 (file)
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -78,6 +78,7 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
        strcpy(name_f_idx, emu->nombre);
        strcat(name_f_idx, EMUFS_IDX_EXT);
 
+       (*err) = 0;
        if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
                PERR("No se puede abrir archivo");
                *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
@@ -86,6 +87,7 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
        while (!feof(f_idx)) {
                /* Me aseguro de leer la cantidad de bytes correcta */
                if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) {
+                       if (feof(f_idx)) break; /* No leyó por EOF */
                        PERR("Error al leer registros de idx");
                        *err = 3; /* EMUFS_ERROR_FILE_READ */
                        return EMUFS_NOT_FOUND;
@@ -105,30 +107,36 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
 }
 
 /* busca el registro ID en el archivo ".idx" y devuelve el nro de bloque en el que se encuentra */
-EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID n_IdReg)
+EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID reg_id)
 {
        FILE* f_idx;
        EMUFS_IDX reg;
        char name_f_idx[255];
-       unsigned short int b_Found = 0;
        
        strcpy(name_f_idx,emu->nombre);
        strcat(name_f_idx, EMUFS_IDX_EXT);
        
-       if ( (f_idx = fopen(name_f_idx,"r")) == NULL) return -1; /*ERROR*/
+       if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
+               PERR("No se puede abrir archivo");
+               /*    *err = 4; * EMUFS_ERROR_CANT_OPEN_FILE */
+               return EMUFS_NOT_FOUND;
+       }
 
-       while (!feof(f_idx) && !b_Found){
-               if (fread(&reg,sizeof(EMUFS_IDX),1,f_idx) != 1) continue;
-               if (reg.n_idreg == n_IdReg) b_Found = 1;
+       while (!feof(f_idx)) {
+               if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) {
+                       if (feof(f_idx)) break; /* No leyó por EOF */
+                       PERR("Error al leer registros de idx");
+                       /*   *err = 3; * EMUFS_ERROR_FILE_READ */
+                       return EMUFS_NOT_FOUND;
+               }
+               if (reg.n_idreg == reg_id) {
+                       fclose(f_idx);
+                       return reg.n_location;
+               }
        }
-       
        fclose(f_idx);
-       
-       /* Sino lo encontre devuelvo uno, otherwise el offset o bloque */
-       if (!b_Found)
-               return(-1);
-       else
-               return(reg.n_location);
+
+       return EMUFS_NOT_FOUND;
 }
 
 /* agrega un registro al final del archivo */
@@ -145,7 +153,7 @@ int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID n_idreg, EMUFS_BLOCK_ID n_locatio
                
     /* Note: Location = Bloque para Tipo 1 y 3, Offset para Tipo 2 */
     reg.n_idreg = n_idreg;
-               reg.n_location = n_location;
+       reg.n_location = n_location;
        fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx); 
        fclose(f_idx);
        return 0;
@@ -214,10 +222,6 @@ EMUFS_REG_ID emufs_idx_get_new_id(EMUFS* efs, int* err)
 
        id = emufs_did_get_last(efs, err);
        if (id == EMUFS_NOT_FOUND) {
-               if (*err) {
-                       PERR("error al obtener ultimo id");
-                       return id;
-               }
                id = emufs_idx_buscar_mayor_id_libre(efs, err);
                if (*err) {
                        PERR("error al obtener id mayor");