database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-28 05:21:27 +03:00
Code

pg_upgrade: Fix inconsistency in memory freeing

Browse Source 
The function in charge of freeing the memory from a result created by
PQescapeIdentifier() has to be PQfreemem(), to ensure that both
allocation and free come from libpq.

One spot in pg_upgrade was not respecting that for pg_database's
datlocale (daticulocale in v16) when the collation provider is libc (aka
datlocale/daticulocale is NULL) with an allocation done using
pg_strdup() and a free with PQfreemem().  The code is changed to always
use PQescapeLiteral() when processing the input.

Oversight in 9637badd9f92.  This commit is similar to 48e4ae9a0707 and
5b94e2753439.

Author: Michael Paquier <michael@paquier.xyz>
Co-authored-by: Ranier Vilela <ranier.vf@gmail.com>
Discussion: https://postgr.es/m/Z601RQxTmIUohdkV@paquier.xyz
Backpatch-through: 16
This commit is contained in:
Michael Paquier 2025-02-28 10:15:32 +09:00
parent f903d4da92
commit ee78823ff5
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/bin/pg_upgrade/pg_upgrade.c
View File

@ -408,6 +408,7 @@ set_locale_and_encoding(void)
char *datcollate_literal; char *datcollate_literal;
char *datctype_literal; char *datctype_literal;
char *datlocale_literal = NULL; char *datlocale_literal = NULL;
char *datlocale_src;
DbLocaleInfo *locale = old_cluster.template0; DbLocaleInfo *locale = old_cluster.template0;
prep_status("Setting locale and encoding for new cluster"); prep_status("Setting locale and encoding for new cluster");
@ -421,12 +422,10 @@ set_locale_and_encoding(void)
datctype_literal = PQescapeLiteral(conn_new_template1, datctype_literal = PQescapeLiteral(conn_new_template1,
locale->db_ctype, locale->db_ctype,
strlen(locale->db_ctype)); strlen(locale->db_ctype));
if (locale->db_locale) datlocale_src = locale->db_locale ? locale->db_locale : "NULL";
datlocale_literal = PQescapeLiteral(conn_new_template1, datlocale_literal = PQescapeLiteral(conn_new_template1,
locale->db_locale, datlocale_src,
strlen(locale->db_locale)); strlen(datlocale_src));
else
datlocale_literal = pg_strdup("NULL");
/* update template0 in new cluster */ /* update template0 in new cluster */
if (GET_MAJOR_VERSION(new_cluster.major_version) >= 1700) if (GET_MAJOR_VERSION(new_cluster.major_version) >= 1700)