* BUGFIX : un error de orden de condiciones hacia que las claves multiples

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

diff --git a/emufs/did.c b/emufs/did.c
index 9eb47daa235b01e65648f0a02ba5e2979d31738d..6e13fd5cae3d5907cd8b35f7e370d7c7185480d4 100644 (file)
--- a/emufs/did.c
+++ b/emufs/did.c
@@ -37,77 +37,108 @@
  */
 
 #include "did.h"
  */
 
 #include "did.h"

-#include <string.h>
+#include "error.h"
+#include "common.h"

 #include <unistd.h>
 #include <unistd.h>

+#include <string.h>

 
 

-/**********************************************************************/
-/* int emufs_did_crear(EMUFS* efs)                                    */
-/*                                                                                                                                                  */
-/* Objetivo: Crea un archivo de ID's Liberados.                       */
-/* Parametros: EMUFS *efs // Struct con handlers de un Data File.     */
-/**********************************************************************/

 int emufs_did_crear(EMUFS* efs)
 {
        return emufs_crear_archivo_auxiliar(efs->nombre, EMUFS_DID_EXT);
 }
 
 int emufs_did_crear(EMUFS* efs)
 {
        return emufs_crear_archivo_auxiliar(efs->nombre, EMUFS_DID_EXT);
 }
 

-/**********************************************************************/
-/* EMUFS_REG_ID emufs_did_get_last(EMUFS *efs)                        */
-/*                                                                                                                                                  */
-/* Objetivo: Devuelve el ID Libre, liberado mas recientemente (pila)  */
-/* Parametros: EMUFS *efs // Struct con handlers + info del openfile. */
-/**********************************************************************/
-EMUFS_REG_ID emufs_did_get_last(EMUFS *efs)
+EMUFS_REG_ID emufs_did_get_last(EMUFS *efs, int* err)

 {
        FILE *f_did;
 {
        FILE *f_did;

-       EMUFS_REG_ID n_RegId;
-       EMUFS_OFFSET n_Offset;
+       EMUFS_REG_ID n_regid;
+       EMUFS_OFFSET n_offset;

        char name_f_did[255];
        char name_f_did[255];

+       long pos;

        
        strcpy(name_f_did, efs->nombre);
        strcat(name_f_did, EMUFS_DID_EXT);
        
        
        strcpy(name_f_did, efs->nombre);
        strcat(name_f_did, EMUFS_DID_EXT);
        

-       if ( (f_did = fopen(name_f_did,"r")) == NULL) return -1; /*ERROR*/
-       fseek(f_did, 0, SEEK_END);
+       if ((f_did = fopen(name_f_did, "rb")) == NULL) {
+               PERR("No se puede abrir archivo");
+               *err = EMUFS_ERROR_CANT_OPEN_FILE;
+               return EMUFS_NOT_FOUND;
+       }
+       if (fseek(f_did, 0l, SEEK_END)) {
+               PERR("No se pudo hacer fseek()");
+               fclose(f_did);
+               *err = EMUFS_ERROR_SEEK_FILE;
+               return EMUFS_NOT_FOUND;
+       }

        
        

-       if (ftell(f_did) > 0){
+       if ((pos = ftell(f_did)) > 0){

                /* si el archivo no esta vacio es porque hay un nro disponible*/
                /* si el archivo no esta vacio es porque hay un nro disponible*/

-               fseek(f_did, -sizeof(EMUFS_REG_ID),SEEK_END);
+               if (fseek(f_did, -sizeof(EMUFS_REG_ID), SEEK_END)) {
+                       PERR("No se pudo hacer fseek()");
+                       fclose(f_did);
+                       *err = EMUFS_ERROR_SEEK_FILE;
+                       return EMUFS_NOT_FOUND;
+               }

                /* leo el ultimo numero */
                /* leo el ultimo numero */

-               fread(&n_RegId,sizeof(EMUFS_REG_ID),1,f_did);
+               if (fread(&n_regid, sizeof(EMUFS_REG_ID), 1, f_did) != 1) {
+                       fclose(f_did);
+                       PERR("Error al leer did");
+                       *err = EMUFS_ERROR_FILE_READ;
+                       return EMUFS_NOT_FOUND;
+               }

                /* voy al final */
                /* voy al final */

-               fseek(f_did, 0, SEEK_END);
-               /* mido el tamaño del archivo*/
-               n_Offset = ftell(f_did);
+               if (fseek(f_did, 0l, SEEK_END)) {
+                       PERR("No se pudo hacer fseek()");
+                       fclose(f_did);
+                       *err = EMUFS_ERROR_SEEK_FILE;
+                       return EMUFS_NOT_FOUND;
+               }
+               /* mido el tamaño del archivo */
+               if ((n_offset = ftell(f_did)) == -1) {
+                       PERR("No se pudo hacer ftell()");
+                       fclose(f_did);
+                       *err = EMUFS_ERROR_TELL_FILE;
+                       return EMUFS_NOT_FOUND;
+               }

                fclose(f_did);
                fclose(f_did);

-               /*lo trunco */
-               truncate(name_f_did, n_Offset - sizeof(EMUFS_REG_ID));
-       } else {
+               /* lo trunco */
+               if (truncate(name_f_did, n_offset - sizeof(EMUFS_REG_ID))) {
+                       PERR("No se pudo truncar el archivo did");
+                       *err = EMUFS_ERROR_TRUNCATE_FILE;
+                       return EMUFS_NOT_FOUND;
+               }
+       } else if (!pos) {

                fclose(f_did);
                /* si el archivo esta vacio */
                fclose(f_did);
                /* si el archivo esta vacio */

-               n_RegId = -1;
+               n_regid = EMUFS_NOT_FOUND;
+       } else {
+               PERR("No se pudo hacer ftell()");
+               fclose(f_did);
+               *err = EMUFS_ERROR_TELL_FILE;
+               return EMUFS_NOT_FOUND;

        }
        }

-       return n_RegId;
+       return n_regid;

 }
 
 }
 

-/**********************************************************************/
-/* int emufs_did_agregar(EMUFS *efs, EMUFS_REG_ID n_RegId)            */
-/*                                                                                                                                                  */
-/* Objetivo: Agrega un registro al archivo de ID's Libres (pila)      */
-/* Parametros: EMUFS *efs // Struct con handlers + info del openfile. */
-/*             EMUFS_REG_ID n_RegId // Id Libre a agregar.            */
-/**********************************************************************/
-int emufs_did_agregar(EMUFS *efs, EMUFS_REG_ID n_RegId)
+int emufs_did_agregar(EMUFS *efs, EMUFS_REG_ID n_regid)

 {
        FILE *f_did;
        char name_f_did[255];
        
        strcpy(name_f_did, efs->nombre);
 {
        FILE *f_did;
        char name_f_did[255];
        
        strcpy(name_f_did, efs->nombre);

-       strcat(name_f_did,  EMUFS_DID_EXT);
+       strcat(name_f_did, EMUFS_DID_EXT);

        
        if ( (f_did = fopen(name_f_did,"a+")) == NULL) return -1;
        
        if ( (f_did = fopen(name_f_did,"a+")) == NULL) return -1;

-       fwrite(&n_RegId, sizeof(EMUFS_REG_ID), 1, f_did);
+       fwrite(&n_regid, sizeof(EMUFS_REG_ID), 1, f_did);

        fclose(f_did);
        
        return 0;
 }
        fclose(f_did);
        
        return 0;
 }

+
+long emufs_did_get_file_size(EMUFS* efs, int* err)
+{
+       char name[255];
+       strcpy(name, efs->nombre);
+       strcat(name, EMUFS_DID_EXT);
+       return emufs_common_get_file_size(name, err);
+}
+