Changeset 525 for trunk/src

Timestamp:

May 17, 2017, 8:46:51 PM (7 years ago)

Author:

katerina

Message:

Log error if the close() on the baseline database fails (ticket #420).

Location:

trunk/src

Files:

• sh_cat.c (modified) (2 diffs)
• sh_dbIO.c (modified) (5 diffs)
• slib.c (modified) (2 diffs)

trunk/src/sh_cat.c

@@ -72 +72 @@
   { MSG_CHECK_2,     SH_ERR_STAMP,   STAMP, N_("msg=\"File check starting.\"")},
   { MSG_STAMP,       SH_ERR_STAMP,   STAMP, N_("msg=\"---- TIMESTAMP ----\"")},
-
+  { MSG_DCLOSE,      SH_ERR_NOTICE,  RUN,   N_("msg=\"Finished writing baseline database.\"")},
+  
   { MSG_D_START,     SH_ERR_INFO,    RUN,   N_("msg=\"Downloading configuration file\"")},
   { MSG_D_DSTART,    SH_ERR_INFO,    RUN,   N_("msg=\"Downloading database file\"")},
@@ -403 +404 @@
   { MSG_CHECK_2,     SH_ERR_STAMP,   STAMP, N_("msg=<File check starting.>")},
   { MSG_STAMP,       SH_ERR_STAMP,   STAMP, N_("msg=<---- TIMESTAMP ---->")},
+  { MSG_DCLOSE,      SH_ERR_NOTICE,  RUN,   N_("msg=<Finished writing baseline database.>")},
 
   { MSG_D_START,     SH_ERR_INFO,    RUN,   N_("msg=<Downloading configuration file>")},

trunk/src/sh_dbIO.c

@@ -152 +152 @@
 }
 
+/* Seek to [SOF] and truncate remainder of file
+ */
 static int sh_dbIO_setdataent_old (SL_TICKET fd, char * line, int size, 
                                    const char * file)
@@ -1117 +1119 @@
 }
 
+/* Seek to [SOF] and truncate remainder
+ */
 static int seek_writeout_data_old(SL_TICKET fd, const char * path)
 {
   char * line = SH_ALLOC(MAX_PATH_STORE+1);
 
+  /* This will do an ftruncate() after the sof marker
+   */
   if (SL_ISERROR(sh_dbIO_setdataent_old (fd, line, MAX_PATH_STORE, path)))
     {
       SH_FREE(line);
       sh_error_handle((-1), FIL__, __LINE__, 0, MSG_E_SUBGPATH,
-                      _("Failed to seek to end of baseline database"),
+                      _("Failed to seek to start of baseline database"),
                       _("seek_writeout_data_old"),
                       path);
@@ -1414 +1420 @@
             {
               pushdata_fd = open_writeout_data(outpath);
+              /* Seek to eof */
               seek_writeout_data(pushdata_fd, outpath);
             }
@@ -1424 +1431 @@
           TPT((0, FIL__, __LINE__, _("msg=<Update.>\n")));
           pushdata_fd = open_writeout_data(outpath);
+          /* Seek to sof and truncate */
           seek_writeout_data_old(pushdata_fd, outpath);
         }
@@ -1479 +1487 @@
       if (sh.flag.checkSum != SH_CHECK_INIT || (buf == NULL && fileHash == NULL))
         {
-          sl_close (pushdata_fd);
+          if (SL_ISERROR(sl_close (pushdata_fd))) 
+            {
+              char * tmp = sh_util_safe_name(outpath);
+              sh_error_handle((-1), FIL__, __LINE__, 0, MSG_E_SUBGPATH,
+                              _("Failed to close baseline database"),
+                              _("sh_dbIO_data_write_int"),
+                              tmp);
+              SH_FREE(tmp);
+            }
+          else {
+            if (sh.flag.checkSum == SH_CHECK_INIT)
+              sh_error_handle((-1), FIL__, __LINE__, 0, MSG_DCLOSE);
+          }
           pushdata_fd = -1;
         }

trunk/src/slib.c

@@ -2383 +2383 @@
                _("msg=<Error fclosing file.>, fd=<%d>, err=<%s>\n"), 
                fd, strerror(errno)));
+          SL_IRETURN(SL_ECLOSE, _("sl_close")); 
         }
     }
@@ -2392 +2393 @@
                _("msg=<Error closing file.>, fd=<%d>, err=<%s>\n"), 
                fd, strerror(errno)));
+          SL_IRETURN(SL_ECLOSE, _("sl_close")); 
         }
     }