On Thu, 14 Aug 2025 at 15:10, Heikki Linnakangas <hlinn...@iki.fi> wrote: > Here's a new set of patches, to disconnect on OOM instead of hanging or > silently discarding messages:
Code looks good. Som small nitpicks though. This change seems unnecessary, i.e. free(NULL) is a no-op - free(svname); + if (svname) + free(svname); Small wording suggestion, maybe change this: The caller has already saved the error message in conn->errorMessage. to The caller should have already saved the error message in conn->errorMessage. or even The caller should have already saved the error message using libpq_append_conn_error.
