]> granicus.if.org Git - postgresql/commit
Improve handling of password reuse in src/bin/scripts programs.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 23 Dec 2015 20:45:43 +0000 (15:45 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 23 Dec 2015 20:45:43 +0000 (15:45 -0500)
commit3945b6193240cd7fcb5ad3c67e9dca7f12d68b7e
tree185e9a5df69a036fd8c4c17835256e0e2a815c7d
parenta21994c1bf9e61d263a870c98ba1e5f559107b4e
Improve handling of password reuse in src/bin/scripts programs.

This reverts most of commit 83dec5a71 in favor of having connectDatabase()
store the possibly-reusable password in a static variable, similar to the
coding we've had for a long time in pg_dump's version of that function.
To avoid possible problems with unwanted password reuse, make callers
specify whether it's reasonable to attempt to re-use the password.
This is a wash for cases where re-use isn't needed, but it is far simpler
for callers that do want that.  Functionally there should be no difference.

Even though we're past RC1, it seems like a good idea to back-patch this
into 9.5, like the prior commit.  Otherwise, if there are any third-party
users of connectDatabase(), they'll have to deal with an API change in
9.5 and then another one in 9.6.

Michael Paquier
src/bin/scripts/clusterdb.c
src/bin/scripts/common.c
src/bin/scripts/common.h
src/bin/scripts/createlang.c
src/bin/scripts/createuser.c
src/bin/scripts/droplang.c
src/bin/scripts/dropuser.c
src/bin/scripts/reindexdb.c
src/bin/scripts/vacuumdb.c