gout

commit bef0db0f3e04e7128ad7957496677a46671738df
parent 2db2c6b33655bf2605109da446075ea54604be64
Author: Chris Bracken <chris@bracken.jp>
Date:   Fri,  6 Mar 2026 17:07:24 +0900

Use estrlcpy instead of strcpy

Diffstat:
M
src/format.h
 | 
5
+----
M
src/fs_inmemory.c
 | 
5
+++--
M
src/git/commit_tests.c
 | 
4
+++-

3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/format.h b/src/format.h
@@ -44,9 +44,6 @@ void print_gopher_link_padded(FILE* out,
                               size_t width,
                               char pad_char);
 
-void print_utf8_padded(FILE* out,
-                       const char* str,
-                       size_t width,
-                       char pad_char);
+void print_utf8_padded(FILE* out, const char* str, size_t width, char pad_char);
 
 #endif  // GOUT_FORMAT_H_
diff --git a/src/fs_inmemory.c b/src/fs_inmemory.c
@@ -1,5 +1,6 @@
 #include "fs_inmemory.h"
 
+#include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -185,10 +186,10 @@ static int inmemory_access(const char* path, int amode) {
 
 static char* inmemory_realpath(const char* path, char* resolved_path) {
   if (resolved_path) {
-    strcpy(resolved_path, path);
+    estrlcpy(resolved_path, path, PATH_MAX);
     return resolved_path;
   }
-  return strdup(path);
+  return estrdup(path);
 }
 
 static const FileSystem kStdInmemoryFs = {
diff --git a/src/git/commit_tests.c b/src/git/commit_tests.c
@@ -8,6 +8,7 @@
 
 #include "git/internal.h"
 #include "utest.h"
+#include "utils.h"
 
 struct git_commit_test {
   char repo_path[1024];
@@ -23,7 +24,8 @@ static int remove_obj(const char* path,
 
 UTEST_F_SETUP(git_commit_test) {
   git_libgit2_init();
-  strcpy(utest_fixture->repo_path, "/tmp/gout_test_XXXXXX");
+  estrlcpy(utest_fixture->repo_path, "/tmp/gout_test_XXXXXX",
+           sizeof(utest_fixture->repo_path));
   if (!mkdtemp(utest_fixture->repo_path)) {
     exit(1);
   }