feat: add sentry__path_get_mtime() (#1317)

#1317

Author: jpnurmi

src/path/sentry_path_unix.c

@@ -267,6 +267,17 @@ sentry__path_get_size(const sentry_path_t *path)
     }
 }
 
+time_t
+sentry__path_get_mtime(const sentry_path_t *path)
+{
+    struct stat buf;
+    if (stat(path->path, &buf) == 0) {
+        return (time_t)buf.st_mtime;
+    } else {
+        return 0;
+    }
+}
+
 sentry_path_t *
 sentry__path_append_str(const sentry_path_t *base, const char *suffix)
 {

src/path/sentry_path_windows.c

@@ -329,6 +329,17 @@ sentry__path_get_size(const sentry_path_t *path)
     }
 }
 
+time_t
+sentry__path_get_mtime(const sentry_path_t *path)
+{
+    struct _stat buf;
+    if (_wstat(path->path, &buf) == 0) {
+        return (time_t)buf.st_mtime;
+    } else {
+        return 0;
+    }
+}
+
 sentry_path_t *
 sentry__path_append_str(const sentry_path_t *base, const char *suffix)
 {

src/sentry_path.h

@@ -4,6 +4,7 @@
 #include "sentry_boot.h"
 
 #include <stdio.h>
+#include <time.h>
 
 #ifdef SENTRY_PLATFORM_WINDOWS
 typedef wchar_t sentry_pathchar_t;
@@ -152,6 +153,12 @@ int sentry__path_touch(const sentry_path_t *path);
  */
 size_t sentry__path_get_size(const sentry_path_t *path);
 
+/**
+ * This will return the last modification time of the file at `path`, or 0 on
+ * failure.
+ */
+time_t sentry__path_get_mtime(const sentry_path_t *path);
+
 /**
  * This will read all the content of `path` into a newly allocated buffer, and
  * write its size into `size_out`.

tests/unit/test_path.c

@@ -2,6 +2,14 @@
 #include "sentry_string.h"
 #include "sentry_testsupport.h"
 
+#ifdef SENTRY_PLATFORM_WINDOWS
+#    include <windows.h>
+#    define sleep_s(SECONDS) Sleep(SECONDS * 1000)
+#else
+#    include <unistd.h>
+#    define sleep_s(SECONDS) sleep(SECONDS)
+#endif
+
 SENTRY_TEST(recursive_paths)
 {
     sentry_path_t *base = sentry__path_from_str(SENTRY_TEST_PATH_PREFIX ".foo");
@@ -240,3 +248,30 @@ SENTRY_TEST(path_directory)
     sentry__path_free(path_1);
     sentry__path_free(path_2);
 }
+
+SENTRY_TEST(path_mtime)
+{
+    sentry_path_t *path
+        = sentry__path_from_str(SENTRY_TEST_PATH_PREFIX "foo.txt");
+    TEST_ASSERT(!!path);
+
+    TEST_CHECK(sentry__path_remove(path) == 0);
+    TEST_CHECK(!sentry__path_is_file(path));
+    TEST_CHECK(sentry__path_get_mtime(path) <= 0);
+
+    sentry__path_touch(path);
+    TEST_CHECK(sentry__path_is_file(path));
+
+    int before = sentry__path_get_mtime(path);
+    TEST_CHECK(before > 0);
+
+    sleep_s(1);
+
+    sentry__path_write_buffer(path, "after", 5);
+    int after = sentry__path_get_mtime(path);
+    TEST_CHECK(after > 0);
+    TEST_CHECK(before < after);
+
+    sentry__path_remove(path);
+    sentry__path_free(path);
+}

tests/unit/tests.inc

@@ -92,6 +92,7 @@ XX(path_from_str_n_wo_null_termination)
 XX(path_from_str_null)
 XX(path_joining_unix)
 XX(path_joining_windows)
+XX(path_mtime)
 XX(path_relative_filename)
 XX(procmaps_parser)
 XX(propagation_context_init)