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 2a071a6f443ce7d2ea0b84041377fa1b32197360..663feef0d46db776006be096956182d8cc698d7a 100644 (file)
--- a/emufs/did.c
+++ b/emufs/did.c
@@ -37,43 +37,83 @@
  */
 
 #include "did.h"
-#include <string.h>
 #include <unistd.h>
+#include <sys/types.h>
+#include <string.h>
 
 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)
+EMUFS_REG_ID emufs_did_get_last(EMUFS *efs, int* err)
 {
        FILE *f_did;
        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 = 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;
 }