pgsql: Fix write-past-buffer-end in ldapServiceLookup().

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
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/8d0df2048f8c0663b41440eeaf73e8859c922e28

Modified Files
--------------
src/interfaces/libpq/fe-connect.c | 20 +++++++++++++-------
1 files changed, 13 insertions(+), 7 deletions(-)