Bugfix.

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

diff --git a/emufs/idx.c b/emufs/idx.c
index 9ae481a3f01414ccfb8e540d1be52111326bb199..35a01f074770f9feb97e7a2d08fac7ed2698c50e 100644 (file)
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -36,6 +36,7 @@
  */
 
 #include "idx.h"
  */
 
 #include "idx.h"

+#include "did.h"

 #include <stdlib.h>
 #include <strings.h>
 #include <unistd.h>
 #include <stdlib.h>
 #include <strings.h>
 #include <unistd.h>


@@ -66,32 +67,42 @@ int emufs_idx_crear(EMUFS *efs)
 }
 
 /* Devuelve el mayor id de registro utilizado so far en el archivo de datos, revisando el indice. */
 }
 
 /* Devuelve el mayor id de registro utilizado so far en el archivo de datos, revisando el indice. */

-EMUFS_REG_ID emufs_idx_buscar_mayor_id(EMUFS *emu)
+EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)

 {
        EMUFS_REG_ID max = 0;
 {
        EMUFS_REG_ID max = 0;

-       FILE *f_idx;    
+       FILE* f_idx;    

        EMUFS_IDX reg;
        char name_f_idx[255]; /* TODO usar malloc para no limitar el tamaño de nombre de archivo */
        EMUFS_IDX reg;
        char name_f_idx[255]; /* TODO usar malloc para no limitar el tamaño de nombre de archivo */

-       char found = 0;
+       int found = 0;

 
 

-       strcpy(name_f_idx,emu->nombre);
+       strcpy(name_f_idx, emu->nombre);

        strcat(name_f_idx, EMUFS_IDX_EXT);
 
        strcat(name_f_idx, EMUFS_IDX_EXT);
 

-       if ( (f_idx = fopen(name_f_idx,"r")) == NULL) return -1; /*ERROR*/
-       while ( !feof(f_idx) ){
+       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)) {

                /* Me aseguro de leer la cantidad de bytes correcta */
                /* Me aseguro de leer la cantidad de bytes correcta */

-               if (fread(&reg,sizeof(EMUFS_IDX),1,f_idx) != 1) continue;
+               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 >= max) {
                        max = reg.n_idreg;
                        found = 1;
                }
        }
        fclose(f_idx);
                if (reg.n_idreg >= max) {
                        max = reg.n_idreg;
                        found = 1;
                }
        }
        fclose(f_idx);

-       
-       if (!found)
-               return (0);
-       else
-               return(max+1);
+
+       if (found) {
+               return ++max;
+       } else {
+               return 0;
+       }

 }
 
 /* busca el registro ID en el archivo ".idx" y devuelve el nro de bloque en el que se encuentra */
 }
 
 /* busca el registro ID en el archivo ".idx" y devuelve el nro de bloque en el que se encuentra */


@@ -197,3 +208,23 @@ int emufs_idx_borrar(EMUFS *emu, EMUFS_REG_ID n_IdReg)
        free(buffer);
        return 0;
 }
        free(buffer);
        return 0;
 }

+
+EMUFS_REG_ID emufs_idx_get_new_id(EMUFS* efs, int* err)
+{
+       EMUFS_REG_ID id;
+
+       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");
+                       return id;
+               }
+       }
+       return id;      
+}
+