On Wed, Nov 23, 2022 at 4:54 PM Ted Yu <yuzhih...@gmail.com> wrote:

> Hi,
> I was looking at the following commit:
>
> commit efc981627a723d91e86865fb363d793282e473d1
> Author: Michael Paquier <mich...@paquier.xyz>
> 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.
>
> Please see attached patch.
>
> I also modified one comment where `deleted` would be more appropriate verb
> for the context.
>
> Cheers
>

Thinking more on this.
The context should be created when the file is successfully opened.

Please take a look at patch v2.

Attachment: open-auth-file-cleanup-v2.patch
Description: Binary data

Reply via email to