diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-08-16 18:14:39 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-08-16 18:14:39 +0000 |
| commit | ef75f74f4c6063e6feba503510d3cdb4c24992eb (patch) | |
| tree | fb23d6fecc6fedd70ad04fa4c3a79d97a21706d7 /src | |
| parent | 08d1d6176989153efd68d4db1e4755e74977bab7 (diff) | |
| download | postgresql-ef75f74f4c6063e6feba503510d3cdb4c24992eb.tar.gz postgresql-ef75f74f4c6063e6feba503510d3cdb4c24992eb.zip | |
Fix incorrect encoding-aware name truncation in makeArrayTypeName().
truncate_identifier won't do anything if the passed-in strlen is already
less than NAMEDATALEN, which it always would be given the strlcpy usage.
This has been broken since the arrays-of-composite-types code went in.
Arguably truncate_identifier is suffering from excessive optimization
and should always process the string, but for the moment I'll take the
more localized patch.
Per bug #4987.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/catalog/pg_type.c | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/src/backend/catalog/pg_type.c b/src/backend/catalog/pg_type.c index 4c7e3f50785..bc797fd4081 100644 --- a/src/backend/catalog/pg_type.c +++ b/src/backend/catalog/pg_type.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/catalog/pg_type.c,v 1.126 2009/06/11 14:48:55 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/catalog/pg_type.c,v 1.126.2.1 2009/08/16 18:14:39 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -686,23 +686,27 @@ RenameTypeInternal(Oid typeOid, const char *newTypeName, Oid typeNamespace) char * makeArrayTypeName(const char *typeName, Oid typeNamespace) { - char *arr; - int i; + char *arr = (char *) palloc(NAMEDATALEN); + int namelen = strlen(typeName); Relation pg_type_desc; + int i; /* * The idea is to prepend underscores as needed until we make a name that * doesn't collide with anything... */ - arr = palloc(NAMEDATALEN); - pg_type_desc = heap_open(TypeRelationId, AccessShareLock); for (i = 1; i < NAMEDATALEN - 1; i++) { arr[i - 1] = '_'; - strlcpy(arr + i, typeName, NAMEDATALEN - i); - truncate_identifier(arr, strlen(arr), false); + if (i + namelen < NAMEDATALEN) + strcpy(arr + i, typeName); + else + { + memcpy(arr + i, typeName, NAMEDATALEN - i); + truncate_identifier(arr, NAMEDATALEN, false); + } if (!SearchSysCacheExists(TYPENAMENSP, CStringGetDatum(arr), ObjectIdGetDatum(typeNamespace), |