Re: [PATCH] psql: tab completion for ALTER ROLE ... IN DATABASE ...

Hi zeng,

Thanks for pointing this out. You’re absolutely right — PQescapeLiteral() allocates memory using libpq’s allocator, so the returned buffers must be released with PQfreemem() rather than pfree(). Using pfree() here would be incorrect, since it expects memory allocated via PostgreSQL’s memory context APIs (palloc/psprintf).

I’ll update the patch to replace pfree() with PQfreemem() for the buffers returned by PQescapeLiteral(),while continuing to use pfree() for memory allocated via psprintf().

Thanks again for catching this.

Best regards,

Vasuki M

C-DAC,Chennai