On Wednesday, July 20, 2022 12:52 PM, Michael Paquier <mich...@paquier.xyz> 
wrote:
> What about the argument of upthread where we could use a goto in
> functions where there are multiple pattern validation checks?  Per se
> v4 attached.

Thanks for your kindly remind and modification.
I checked v4 patch, it looks good but I think there can be some minor 
improvement.
So I deleted some redundant braces around "goto error_return; ".
Also added an error handle section in validateSQLNamePattern.

Kindly to have a check at the attached v5 patch.

Regards,
Tang

Attachment: v5-0001-fix-the-memory-leak-in-psql-describe.patch
Description: v5-0001-fix-the-memory-leak-in-psql-describe.patch

Reply via email to