diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2015-06-29 12:42:52 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2015-06-29 12:42:52 -0400 |
commit | cbc8d65639344c390a1d1a7f646c186ff3ad8693 (patch) | |
tree | 46c7ad96d8f554aea527f30247363f4fc85f8ef5 /src/include/miscadmin.h | |
parent | 07cb8b02ab4c8b65bb2e3b87ad2402fdc6cce978 (diff) | |
download | postgresql-cbc8d65639344c390a1d1a7f646c186ff3ad8693.tar.gz postgresql-cbc8d65639344c390a1d1a7f646c186ff3ad8693.zip |
Code + docs review for escaping of option values (commit 11a020eb6).
Avoid memory leak from incorrect choice of how to free a StringInfo
(resetStringInfo doesn't do it). Now that pg_split_opts doesn't scribble
on the optstr, mark that as "const" for clarity. Attach the commentary in
protocol.sgml to the right place, and add documentation about the
user-visible effects of this change on postgres' -o option and libpq's
PGOPTIONS option.
Diffstat (limited to 'src/include/miscadmin.h')
-rw-r--r-- | src/include/miscadmin.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/include/miscadmin.h b/src/include/miscadmin.h index 71aa505e17d..b5391673609 100644 --- a/src/include/miscadmin.h +++ b/src/include/miscadmin.h @@ -408,7 +408,7 @@ extern AuxProcType MyAuxProcType; *****************************************************************************/ /* in utils/init/postinit.c */ -extern void pg_split_opts(char **argv, int *argcp, char *optstr); +extern void pg_split_opts(char **argv, int *argcp, const char *optstr); extern void InitializeMaxBackends(void); extern void InitPostgres(const char *in_dbname, Oid dboid, const char *username, Oid useroid, char *out_dbname); |