| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | pgsql-committers(at)postgresql(dot)org |
| Subject: | pgsql: Clean up password prompting logic in streamutil.c. |
| Date: | 2013-11-15 22:27:51 |
| Message-ID: | E1VhRrj-0004oa-Hb@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Clean up password prompting logic in streamutil.c.
The previous coding was fairly unreadable and drew double-free warnings
from clang. I believe the double free was actually not reachable, because
PQconnectionNeedsPassword is coded to not return true if a password was
provided, so that the loop can't iterate more than twice. Nonetheless
it seems worth rewriting. No back-patch since this is just cosmetic.
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/3172eea062e779db39df9a48fca0ad7448163f98
Modified Files
--------------
src/bin/pg_basebackup/streamutil.c | 110 ++++++++++++++++++------------------
1 file changed, 56 insertions(+), 54 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2013-11-15 23:02:12 | pgsql: Speed up printing of INSERT statements in pg_dump. |
| Previous Message | Tom Lane | 2013-11-15 21:46:47 | pgsql: Compute correct em_nullable_relids in get_eclass_for_sort_expr() |