Fix write-past-buffer-end in ldapServiceLookup().
The code to assemble ldap_get_values_len's output into a single string
wrote the terminating null one byte past where it should. Fix that,
and make some other cosmetic adjustments to make the code a trifle more
readable and more in line with usual Postgres coding style.
Also, free the "result" string when done with it, to avoid a permanent
memory leak.
Bug report and patch by Albe Laurenz, cosmetic adjustments by me.
Branch
------
REL8_4_STABLE
Details
-------
http://git.postgresql.org/pg/commitdiff/891c23ff6ecefe871fb0cff1a6a1dc37cf7d25bf
Modified Files
--------------
src/interfaces/libpq/fe-connect.c | 20 +++++++++++++-------
1 files changed, 13 insertions(+), 7 deletions(-)
