From f8bc7d94761f6153be606328183fac6766202914 Mon Sep 17 00:00:00 2001 From: David Oberhollenzer Date: Thu, 22 Jun 2017 14:00:04 +0200 Subject: Add const modifier to read only strings and string constants Signed-off-by: David Oberhollenzer --- tests/fs-tests/lib/tests.c | 16 ++++++++-------- tests/fs-tests/lib/tests.h | 6 +++--- tests/fs-tests/simple/ftrunc.c | 2 +- tests/fs-tests/stress/atoms/stress_2.c | 2 +- tests/fs-tests/utils/free_space.c | 2 +- 5 files changed, 14 insertions(+), 14 deletions(-) (limited to 'tests/fs-tests') diff --git a/tests/fs-tests/lib/tests.c b/tests/fs-tests/lib/tests.c index 8d4d83a..d1a2e0c 100644 --- a/tests/fs-tests/lib/tests.c +++ b/tests/fs-tests/lib/tests.c @@ -40,9 +40,9 @@ #include "tests.h" -char *tests_file_system_mount_dir = TESTS_DEFAULT_FILE_SYSTEM_MOUNT_DIR; +const char *tests_file_system_mount_dir = TESTS_DEFAULT_FILE_SYSTEM_MOUNT_DIR; -char *tests_file_system_type = TESTS_DEFAULT_FILE_SYSTEM_TYPE; +const char *tests_file_system_type = TESTS_DEFAULT_FILE_SYSTEM_TYPE; int tests_ok_to_sync = 0; /* Whether to use fsync */ @@ -62,7 +62,7 @@ int64_t tests_repeat_parameter = 0; int64_t tests_sleep_parameter = 0; /* Program name from argv[0] */ -char *program_name = "unknown"; +const char *program_name = "unknown"; /* General purpose test parameter to specify a file should be unlinked. May be used by different tests in different ways or not at all. */ @@ -1058,8 +1058,8 @@ void tests_remount(void) { int err; struct mntent mount_info; - char *source, *target, *filesystemtype, *data; - char cwd[4096]; + const char *source, *target, *filesystemtype; + char cwd[4096], *data; unsigned long mountflags, flags; unsigned int rorw1, um, um_ro, um_rorw, rorw2; @@ -1152,9 +1152,9 @@ void tests_remount(void) static void tests_mnt(int mnt) { static struct mntent mount_info; - char *source; - char *target; - char *filesystemtype; + const char *source; + const char *target; + const char *filesystemtype; unsigned long mountflags; char *data; static char cwd[4096]; diff --git a/tests/fs-tests/lib/tests.h b/tests/fs-tests/lib/tests.h index 01849bc..6bcfd83 100644 --- a/tests/fs-tests/lib/tests.h +++ b/tests/fs-tests/lib/tests.h @@ -164,9 +164,9 @@ int tests_fs_is_currfs(void); /* Concatenate a pid to a string in a signal safe way */ void tests_cat_pid(char *buf, const char *name, pid_t pid); -extern char *tests_file_system_mount_dir; +extern const char *tests_file_system_mount_dir; -extern char *tests_file_system_type; +extern const char *tests_file_system_type; /* General purpose test parameter to specify some aspect of test size. May be used by different tests in different ways. @@ -200,7 +200,7 @@ extern int tests_delete_flag; extern int tests_hole_flag; /* Program name from argv[0] */ -extern char *program_name; +extern const char *program_name; /* Maximum file name length of test file system (from statfs) */ extern long tests_max_fname_len; diff --git a/tests/fs-tests/simple/ftrunc.c b/tests/fs-tests/simple/ftrunc.c index 2df404b..52b660b 100644 --- a/tests/fs-tests/simple/ftrunc.c +++ b/tests/fs-tests/simple/ftrunc.c @@ -39,7 +39,7 @@ static void ftrunc(void) ssize_t written; int64_t remains; size_t block; - char *file_name; + const char *file_name; off_t actual; char buf[WRITE_BUFFER_SIZE]; diff --git a/tests/fs-tests/stress/atoms/stress_2.c b/tests/fs-tests/stress/atoms/stress_2.c index 5dea1b7..bd53ad3 100644 --- a/tests/fs-tests/stress/atoms/stress_2.c +++ b/tests/fs-tests/stress/atoms/stress_2.c @@ -40,7 +40,7 @@ static void stress_2(void) int64_t remains; int64_t repeat; size_t block; - char *file_name; + const char *file_name; char buf[WRITE_BUFFER_SIZE]; file_name = "stress_2_test_file"; diff --git a/tests/fs-tests/utils/free_space.c b/tests/fs-tests/utils/free_space.c index 88036aa..3421e7d 100644 --- a/tests/fs-tests/utils/free_space.c +++ b/tests/fs-tests/utils/free_space.c @@ -25,7 +25,7 @@ int main(int argc, char *argv[]) { - char *dir_name = "."; + const char *dir_name = "."; uint64_t free_space; struct statvfs fs_info; -- cgit v1.2.3