Thread

Commits

  1. ecpg: check return value of replace_variables()

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

    Aleksander Alekseev <aleksander@tigerdata.com> — 2025-10-07T14:04:29Z

    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
    
  2. Re: [PATCH] ecpg: check return value of replace_variables()

    Daniel Gustafsson <daniel@yesql.se> — 2025-10-10T07:48:40Z

    > 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