[PATCH] Warn on writable strings
authorJens Axboe <jens.axboe@oracle.com>
Tue, 7 Nov 2006 07:36:14 +0000 (08:36 +0100)
committerJens Axboe <jens.axboe@oracle.com>
Tue, 7 Nov 2006 07:36:14 +0000 (08:36 +0100)
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Makefile
fio.c
fio.h
init.c
parse.h
stat.c

index 2746ce7..ad9aa3b 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,5 +1,5 @@
 CC     = gcc
-CFLAGS = -W -Wall -O2 -g -D_GNU_SOURCE -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64
+CFLAGS = -W -Wwrite-strings -Wall -O2 -g -D_GNU_SOURCE -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64
 PROGS  = fio
 SCRIPTS = fio_generate_plots
 OBJS = fio.o ioengines.o init.o stat.o log.o time.o md5.o crc32.o \
diff --git a/fio.c b/fio.c
index 011c13a..8c059df 100644 (file)
--- a/fio.c
+++ b/fio.c
@@ -45,7 +45,7 @@ int groupid = 0;
 int thread_number = 0;
 int shm_id = 0;
 int temp_stall_ts;
-char *fio_inst_prefix = _INST_PREFIX;
+const char *fio_inst_prefix = _INST_PREFIX;
 
 static volatile int startup_sem;
 
diff --git a/fio.h b/fio.h
index b94bf43..434c34b 100644 (file)
--- a/fio.h
+++ b/fio.h
@@ -356,7 +356,7 @@ extern int groupid;
 extern int terse_output;
 extern FILE *f_out;
 extern FILE *f_err;
-extern char *fio_inst_prefix;
+extern const char *fio_inst_prefix;
 extern int temp_stall_ts;
 
 extern struct thread_data *threads;
diff --git a/init.c b/init.c
index 0352c55..54e2eca 100644 (file)
--- a/init.c
+++ b/init.c
@@ -539,8 +539,8 @@ static char *to_kmg(unsigned int val)
  */
 static int add_job(struct thread_data *td, const char *jobname, int job_add_num)
 {
-       char *ddir_str[] = { "read", "write", "randread", "randwrite",
-                            "rw", NULL, "randrw" };
+       const char *ddir_str[] = { "read", "write", "randread", "randwrite",
+                                  "rw", NULL, "randrw" };
        struct stat sb;
        int numjobs, ddir, i;
        struct fio_file *f;
diff --git a/parse.h b/parse.h
index 9b34de9..adbb320 100644 (file)
--- a/parse.h
+++ b/parse.h
@@ -19,7 +19,7 @@ enum fio_opt_type {
  * Option define
  */
 struct fio_option {
-       char *name;
+       const char *name;
        enum fio_opt_type type;
        unsigned int off1;
        unsigned int off2;
diff --git a/stat.c b/stat.c
index 0ae19b2..0ccf37e 100644 (file)
--- a/stat.c
+++ b/stat.c
@@ -333,7 +333,7 @@ static void show_disk_util(void)
 static void show_ddir_status(struct thread_data *td, struct group_run_stats *rs,
                             int ddir)
 {
-       char *ddir_str[] = { "read ", "write" };
+       const char *ddir_str[] = { "read ", "write" };
        unsigned long min, max;
        unsigned long long bw;
        double mean, dev;