Re: freeing LDAPMessage in CheckLDAPAuth

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Zhihong Yu <zyu(at)yugabyte(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: freeing LDAPMessage in CheckLDAPAuth
Date: 2022-09-06 23:24:59
Message-ID: YxfWy/VZzBbnPmNT@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Sep 05, 2022 at 02:50:09AM -0700, Zhihong Yu wrote:
> Here is patch v4.

FWIW, I am fine with what you are basically doing with v4, so I'd like
to apply that on HEAD on the basis of consistency with libpq. As Tom
said, this authentication path will fail, but I'd like to think that
this is a good practice anyway. I'll wait a few days first, in case
others have comments.
--
Michael

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2022-09-06 23:26:12 Re: pg_auth_members.grantor is bunk
Previous Message Justin Pryzby 2022-09-06 22:10:18 Re: pg_upgrade major version compatibility