From 1ffb63a2a1767c3dd0c7611bed6383bd37bfbce6 Mon Sep 17 00:00:00 2001 From: Alvaro Herrera Date: Tue, 6 Mar 2018 13:17:13 -0300 Subject: [PATCH] Fix bogus Name assignment in CreateStatistics Apparently, it doesn't work to use a plain cstring as a Name datum: you may end up having random bytes because of failing to zero the bytes after the terminating \0, as indicated by valgrind. I introduced this bug in 5564c1181548, so backpatch this fix to REL_10_STABLE, like that commit. While at it, fix a slightly misleading comment, pointed out by David Rowley. --- src/backend/commands/statscmds.c | 7 +++++-- src/backend/parser/parse_utilcmd.c | 3 ++- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/backend/commands/statscmds.c b/src/backend/commands/statscmds.c index f46ef01134..c4adfd569e 100644 --- a/src/backend/commands/statscmds.c +++ b/src/backend/commands/statscmds.c @@ -57,6 +57,7 @@ CreateStatistics(CreateStatsStmt *stmt) int16 attnums[STATS_MAX_DIMENSIONS]; int numcols = 0; char *namestr; + NameData stxname; Oid statoid; Oid namespaceId; Oid stxowner = GetUserId(); @@ -135,7 +136,8 @@ CreateStatistics(CreateStatsStmt *stmt) * object in the same namespace as the relation, and cons up a name for it. */ if (stmt->defnames) - namespaceId = QualifiedNameGetCreationNamespace(stmt->defnames, &namestr); + namespaceId = QualifiedNameGetCreationNamespace(stmt->defnames, + &namestr); else { namespaceId = RelationGetNamespace(rel); @@ -144,6 +146,7 @@ CreateStatistics(CreateStatsStmt *stmt) "stat", namespaceId); } + namestrcpy(&stxname, namestr); /* * Deal with the possibility that the statistics object already exists. @@ -307,7 +310,7 @@ CreateStatistics(CreateStatsStmt *stmt) memset(values, 0, sizeof(values)); memset(nulls, false, sizeof(nulls)); values[Anum_pg_statistic_ext_stxrelid - 1] = ObjectIdGetDatum(relid); - values[Anum_pg_statistic_ext_stxname - 1] = CStringGetDatum(namestr); + values[Anum_pg_statistic_ext_stxname - 1] = NameGetDatum(&stxname); values[Anum_pg_statistic_ext_stxnamespace - 1] = ObjectIdGetDatum(namespaceId); values[Anum_pg_statistic_ext_stxowner - 1] = ObjectIdGetDatum(stxowner); values[Anum_pg_statistic_ext_stxkeys - 1] = PointerGetDatum(stxkeys); diff --git a/src/backend/parser/parse_utilcmd.c b/src/backend/parser/parse_utilcmd.c index 8c7bd68639..0fd14f43c6 100644 --- a/src/backend/parser/parse_utilcmd.c +++ b/src/backend/parser/parse_utilcmd.c @@ -2250,7 +2250,8 @@ transformIndexConstraint(Constraint *constraint, CreateStmtContext *cxt) * transformExtendedStatistics * Handle extended statistic objects * - * Right now, there's nothing to do here, so we just copy the list. + * Right now, there's nothing to do here, so we just append the list to + * the existing "after" list. */ static void transformExtendedStatistics(CreateStmtContext *cxt) -- 2.40.0