[PATCH] ecpg: check return value of replace_variables()

Started by Aleksander Alekseev6 months ago2 messageshackers
Jump to latest
#1Aleksander Alekseev
aleksander@timescale.com

Hi,

Currently in src/interfaces/ecpg/ecpglib/prepare.c we don't check the
return value of replace_variables(). The proposed patch fixes this.
The rest of the code looks OK to me.

--
Best regards,
Aleksander Alekseev

Attachments:

v1-0001-ecpg-check-return-value-of-replace_variables.patchtext/x-patch; charset=US-ASCII; name=v1-0001-ecpg-check-return-value-of-replace_variables.patchDownload+7-2
#2Daniel Gustafsson
daniel@yesql.se
In reply to: Aleksander Alekseev (#1)
Re: [PATCH] ecpg: check return value of replace_variables()

On 7 Oct 2025, at 16:04, Aleksander Alekseev <aleksander@tigerdata.com> wrote:

Currently in src/interfaces/ecpg/ecpglib/prepare.c we don't check the
return value of replace_variables(). The proposed patch fixes this.
The rest of the code looks OK to me.

Seems reasonable on a first glance, I'll pick it up.

--
Daniel Gustafsson