diff options
| author | Bruce Momjian <bruce@momjian.us> | 2012-10-12 13:35:40 -0400 |
|---|---|---|
| committer | Bruce Momjian <bruce@momjian.us> | 2012-10-12 13:35:43 -0400 |
| commit | 49ec613201b2e9debdf9e9ad9a2ad7c6c8083729 (patch) | |
| tree | 16626d363ecc7b021b2fda82bb27b811e2335c09 /contrib/pgbench | |
| parent | a29f7ed5544ef583747c0dcc3fc2afac1fb191ef (diff) | |
| download | postgresql-49ec613201b2e9debdf9e9ad9a2ad7c6c8083729.tar.gz | |
In our source code, make a copy of getopt's 'optarg' string arguments,
rather than just storing a pointer.
Diffstat (limited to 'contrib/pgbench')
| -rw-r--r-- | contrib/pgbench/pgbench.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/contrib/pgbench/pgbench.c b/contrib/pgbench/pgbench.c index c399d59d25..5d48aeeae4 100644 --- a/contrib/pgbench/pgbench.c +++ b/contrib/pgbench/pgbench.c @@ -1995,7 +1995,7 @@ main(int argc, char **argv) is_init_mode++; break; case 'h': - pghost = optarg; + pghost = pg_strdup(optarg); break; case 'n': is_no_vacuum++; @@ -2004,7 +2004,7 @@ main(int argc, char **argv) do_vacuum_accounts++; break; case 'p': - pgport = optarg; + pgport = pg_strdup(optarg); break; case 'd': debug++; @@ -2090,14 +2090,14 @@ main(int argc, char **argv) } break; case 'U': - login = optarg; + login = pg_strdup(optarg); break; case 'l': use_log = true; break; case 'f': ttype = 3; - filename = optarg; + filename = pg_strdup(optarg); if (process_file(filename) == false || *sql_files[num_files - 1] == NULL) exit(1); break; @@ -2143,10 +2143,10 @@ main(int argc, char **argv) /* This covers long options which take no argument. */ break; case 2: /* tablespace */ - tablespace = optarg; + tablespace = pg_strdup(optarg); break; case 3: /* index-tablespace */ - index_tablespace = optarg; + index_tablespace = pg_strdup(optarg); break; case 4: sample_rate = atof(optarg); |