Re: freeing LDAPMessage in CheckLDAPAuth - Mailing list pgsql-hackers

From Zhihong Yu
Subject Re: freeing LDAPMessage in CheckLDAPAuth
Date
Msg-id CALNJ-vTA3vayCXWNWy8vQ5A9HgVEC7ZiaGfMsaH5wJ3gRKqvnw@mail.gmail.com
Whole thread Raw
In response to Re: freeing LDAPMessage in CheckLDAPAuth  (Michael Paquier <michael@paquier.xyz>)
Responses Re: freeing LDAPMessage in CheckLDAPAuth
List pgsql-hackers


On Sun, Sep 4, 2022 at 10:37 PM Michael Paquier <michael@paquier.xyz> wrote:
On Sun, Sep 04, 2022 at 06:52:37AM -0700, Zhihong Yu wrote:
> Please take a look at patch v3.

Fine as far as it goes.  I would have put the initialization of
search_message closer to ldap_search_s() for consistency with libpq.
That's what we do with ldap_search_st().
--
Michael
Hi,
Here is patch v4. 
Attachment

pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Handle infinite recursion in logical replication setup
Next
From: "Drouvot, Bertrand"
Date:
Subject: Re: more descriptive message for process termination due to max_slot_wal_keep_size