Se arreglan un par de includes y memory leaks. Me queda por encontrar uno.

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

diff --git a/emufs/did.c b/emufs/did.c
index 9eb47daa235b01e65648f0a02ba5e2979d31738d..663feef0d46db776006be096956182d8cc698d7a 100644 (file)
--- a/emufs/did.c
+++ b/emufs/did.c
@@ -37,67 +37,88 @@
  */
 
 #include "did.h"
-#include <string.h>
 #include <unistd.h>
+#include <sys/types.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);
 }
 
-/**********************************************************************/
-/* 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;
-       EMUFS_REG_ID n_RegId;
-       EMUFS_OFFSET n_Offset;
+       EMUFS_REG_ID n_regid;
+       EMUFS_OFFSET n_offset;
        char name_f_did[255];
+       long pos;
        
        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 = 4; /* 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 = 8; /* 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*/
-               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 = 8; /* EMUFS_ERROR_SEEK_FILE */
+                       return EMUFS_NOT_FOUND;
+               }
                /* 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 = 3; /* EMUFS_ERROR_FILE_READ */
+                       return EMUFS_NOT_FOUND;
+               }
                /* 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 = 8; /* 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 = 9; /* EMUFS_ERROR_TELL_FILE */
+                       return EMUFS_NOT_FOUND;
+               }
                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 = 10; /* EMUFS_ERROR_TRUNCATE_FILE */
+                       return EMUFS_NOT_FOUND;
+               }
+       } else if (!pos) {
                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 = 9; /* 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];
@@ -106,8 +127,9 @@ int emufs_did_agregar(EMUFS *efs, EMUFS_REG_ID n_RegId)
        strcat(name_f_did,  EMUFS_DID_EXT);
        
        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;
 }
+