Re: Fix for bug in ldapServiceLookup in libpq

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Albe Laurenz" <laurenz(dot)albe(at)wien(dot)gv(dot)at>
Cc: "pgsql-hackers" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix for bug in ldapServiceLookup in libpq
Date: 2011-05-12 15:59:50
Message-ID: 17722.1305215990@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

"Albe Laurenz" <laurenz(dot)albe(at)wien(dot)gv(dot)at> writes:
> I have attached a new version of the patch that should address all known
> problems.

You missed one "return" where the string needed to be freed. I've
applied this patch with that fix and a couple of cosmetic changes.
Thanks for the report and patch!

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Smith 2011-05-12 16:07:36 Re: performance-test farm
Previous Message Kevin Grittner 2011-05-12 15:55:26 Re: Formatting Curmudgeons WAS: MMAP Buffers