Re: cleanup in open_auth_file

2022-11-23 Thread Michael Paquier
On Wed, Nov 23, 2022 at 05:09:22PM -0800, Ted Yu wrote: > Thinking more on this. > The context should be created when the file is successfully opened. Indeed. Both operations ought to be done in the reverse order, or we would run into leaks in the postmaster on reload if pg_ident.conf has been

Re: cleanup in open_auth_file

2022-11-23 Thread Ted Yu
On Wed, Nov 23, 2022 at 4:54 PM Ted Yu wrote: > Hi, > I was looking at the following commit: > > commit efc981627a723d91e86865fb363d793282e473d1 > Author: Michael Paquier > Date: Thu Nov 24 08:21:55 2022 +0900 > > Rework memory contexts in charge of HBA/ident tokenization > > I think when

cleanup in open_auth_file

2022-11-23 Thread Ted Yu
Hi, I was looking at the following commit: commit efc981627a723d91e86865fb363d793282e473d1 Author: Michael Paquier Date: Thu Nov 24 08:21:55 2022 +0900 Rework memory contexts in charge of HBA/ident tokenization I think when the file cannot be opened, the context should be deleted.