summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBruce Momjian1997-08-18 20:53:48 +0000
committerBruce Momjian1997-08-18 20:53:48 +0000
commitb99c63cfc029cc0552e98f652d1734aec1124a5b (patch)
tree6e18eedc0b8ebfc0d84479ed26d0ea9b12d8125d
parent022903f22e54cbc78b4acc1ef54f73d19a051630 (diff)
Now that names are null terminated, no need to do all that NAMEDATALEN stuff.
-rw-r--r--src/backend/access/common/printtup.c5
-rw-r--r--src/backend/access/common/tupdesc.c19
-rw-r--r--src/backend/access/hash/hashpage.c22
-rw-r--r--src/backend/bootstrap/bootstrap.c14
-rw-r--r--src/backend/catalog/catalog.c4
-rw-r--r--src/backend/catalog/pg_operator.c6
-rw-r--r--src/backend/commands/cluster.c10
-rw-r--r--src/backend/commands/command.c18
-rw-r--r--src/backend/commands/creatinh.c4
-rw-r--r--src/backend/commands/explain.c4
-rw-r--r--src/backend/commands/remove.c6
-rw-r--r--src/backend/commands/rename.c46
-rw-r--r--src/backend/commands/vacuum.c74
-rw-r--r--src/backend/executor/execMain.c10
-rw-r--r--src/backend/executor/execUtils.c4
-rw-r--r--src/backend/libpq/be-dumpdata.c4
-rw-r--r--src/backend/nodes/outfuncs.c12
-rw-r--r--src/backend/parser/analyze.c8
-rw-r--r--src/backend/parser/dbcommands.c6
-rw-r--r--src/backend/storage/buffer/bufmgr.c10
-rw-r--r--src/backend/storage/smgr/md.c18
-rw-r--r--src/backend/storage/smgr/smgr.c42
-rw-r--r--src/backend/tcop/aclchk.c54
-rw-r--r--src/backend/tcop/utility.c14
-rw-r--r--src/backend/utils/adt/name.c5
-rw-r--r--src/backend/utils/cache/relcache.c8
26 files changed, 207 insertions, 220 deletions
diff --git a/src/backend/access/common/printtup.c b/src/backend/access/common/printtup.c
index b5736d1e7af..cb4b2d0be38 100644
--- a/src/backend/access/common/printtup.c
+++ b/src/backend/access/common/printtup.c
@@ -8,7 +8,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/access/common/printtup.c,v 1.13 1997/01/10 20:16:22 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/access/common/printtup.c,v 1.14 1997/08/18 20:51:30 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -131,9 +131,8 @@ printatt(unsigned attributeId,
AttributeTupleForm attributeP,
char *value)
{
- printf("\t%2d: %.*s%s%s%s\t(typeid = %u, len = %d, byval = %c)\n",
+ printf("\t%2d: %s%s%s%s\t(typeid = %u, len = %d, byval = %c)\n",
attributeId,
- NAMEDATALEN, /* attname is a char16 */
attributeP->attname.data,
value != NULL ? " = \"" : "",
value != NULL ? value : "",
diff --git a/src/backend/access/common/tupdesc.c b/src/backend/access/common/tupdesc.c
index b6d2333f655..f0c8cbec6b1 100644
--- a/src/backend/access/common/tupdesc.c
+++ b/src/backend/access/common/tupdesc.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/access/common/tupdesc.c,v 1.12 1997/08/03 02:34:19 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/access/common/tupdesc.c,v 1.13 1997/08/18 20:51:31 momjian Exp $
*
* NOTES
* some of the executor utility code such as "ExecTypeFromTL" should be
@@ -347,18 +347,11 @@ BuildDescForRelation(List *schema, char *relname)
arry = entry->typename->arrayBounds;
attisset = entry->typename->setof;
- if (arry != NIL) {
- char buf[20];
-
+ strNcpy(typename, entry->typename->name,NAMEDATALEN-1);
+ if (arry != NIL)
attdim = length(arry);
-
- /* array of XXX is _XXX (inherited from release 3) */
- sprintf(buf, "_%.*s", NAMEDATALEN, entry->typename->name);
- strcpy(typename, buf);
- } else {
- strcpy(typename, entry->typename->name);
+ else
attdim = 0;
- }
if (! TupleDescInitEntry(desc, attnum, attname,
typename, attdim, attisset)) {
@@ -372,8 +365,8 @@ BuildDescForRelation(List *schema, char *relname)
if (!strcmp(typename, relname)) {
TupleDescMakeSelfReference(desc, attnum, relname);
} else
- elog(WARN, "DefineRelation: no such type %.*s",
- NAMEDATALEN, typename);
+ elog(WARN, "DefineRelation: no such type %s",
+ typename);
}
/*
diff --git a/src/backend/access/hash/hashpage.c b/src/backend/access/hash/hashpage.c
index 338b13ce513..49c8f03f524 100644
--- a/src/backend/access/hash/hashpage.c
+++ b/src/backend/access/hash/hashpage.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/access/hash/hashpage.c,v 1.8 1997/08/12 22:51:37 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/access/hash/hashpage.c,v 1.9 1997/08/18 20:51:34 momjian Exp $
*
* NOTES
* Postgres hash pages look like ordinary relation pages. The opaque
@@ -195,8 +195,8 @@ _hash_getbuf(Relation rel, BlockNumber blkno, int access)
_hash_setpagelock(rel, blkno, access);
break;
default:
- elog(WARN, "_hash_getbuf: invalid access (%d) on new blk: %.*s",
- access, NAMEDATALEN, RelationGetRelationName(rel));
+ elog(WARN, "_hash_getbuf: invalid access (%d) on new blk: %s",
+ access, RelationGetRelationName(rel));
break;
}
buf = ReadBuffer(rel, blkno);
@@ -221,8 +221,8 @@ _hash_relbuf(Relation rel, Buffer buf, int access)
_hash_unsetpagelock(rel, blkno, access);
break;
default:
- elog(WARN, "_hash_relbuf: invalid access (%d) on blk %x: %.*s",
- access, blkno, NAMEDATALEN, RelationGetRelationName(rel));
+ elog(WARN, "_hash_relbuf: invalid access (%d) on blk %x: %s",
+ access, blkno, RelationGetRelationName(rel));
}
ReleaseBuffer(buf);
@@ -279,8 +279,8 @@ _hash_chgbufaccess(Relation rel,
_hash_relbuf(rel, *bufp, from_access);
break;
default:
- elog(WARN, "_hash_chgbufaccess: invalid access (%d) on blk %x: %.*s",
- from_access, blkno, NAMEDATALEN, RelationGetRelationName(rel));
+ elog(WARN, "_hash_chgbufaccess: invalid access (%d) on blk %x: %s",
+ from_access, blkno, RelationGetRelationName(rel));
break;
}
*bufp = _hash_getbuf(rel, blkno, to_access);
@@ -325,8 +325,8 @@ _hash_setpagelock(Relation rel,
RelationSetSingleRLockPage(rel, &iptr);
break;
default:
- elog(WARN, "_hash_setpagelock: invalid access (%d) on blk %x: %.*s",
- access, blkno, NAMEDATALEN, RelationGetRelationName(rel));
+ elog(WARN, "_hash_setpagelock: invalid access (%d) on blk %x: %s",
+ access, blkno, RelationGetRelationName(rel));
break;
}
}
@@ -350,8 +350,8 @@ _hash_unsetpagelock(Relation rel,
RelationUnsetSingleRLockPage(rel, &iptr);
break;
default:
- elog(WARN, "_hash_unsetpagelock: invalid access (%d) on blk %x: %.*s",
- access, blkno, NAMEDATALEN, RelationGetRelationName(rel));
+ elog(WARN, "_hash_unsetpagelock: invalid access (%d) on blk %x: %s",
+ access, blkno, RelationGetRelationName(rel));
break;
}
}
diff --git a/src/backend/bootstrap/bootstrap.c b/src/backend/bootstrap/bootstrap.c
index 4d8cfbd7406..7d769cddff7 100644
--- a/src/backend/bootstrap/bootstrap.c
+++ b/src/backend/bootstrap/bootstrap.c
@@ -7,7 +7,7 @@
* Copyright (c) 1994, Regents of the University of California
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/bootstrap/bootstrap.c,v 1.19 1997/08/12 22:52:04 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/bootstrap/bootstrap.c,v 1.20 1997/08/18 20:51:44 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -510,8 +510,8 @@ boot_openrel(char *relname)
if (DebugMode) {
AttributeTupleForm at = attrtypes[i];
- printf("create attribute %d name %.*s len %d num %d type %d\n",
- i, NAMEDATALEN, at->attname.data, at->attlen, at->attnum,
+ printf("create attribute %d name %s len %d num %d type %d\n",
+ i, at->attname.data, at->attlen, at->attnum,
at->atttypid
);
fflush(stdout);
@@ -572,16 +572,14 @@ DefineAttr(char *name, char *type, int attnum)
if (Typ != (struct typmap **)NULL) {
attrtypes[attnum]->atttypid = Ap->am_oid;
namestrcpy(&attrtypes[attnum]->attname, name);
- if (!Quiet) printf("<%.*s %s> ", NAMEDATALEN,
- attrtypes[attnum]->attname.data, type);
+ if (!Quiet) printf("<%s %s> ", attrtypes[attnum]->attname.data, type);
attrtypes[attnum]->attnum = 1 + attnum; /* fillatt */
attlen = attrtypes[attnum]->attlen = Ap->am_typ.typlen;
attrtypes[attnum]->attbyval = Ap->am_typ.typbyval;
} else {
attrtypes[attnum]->atttypid = Procid[t].oid;
namestrcpy(&attrtypes[attnum]->attname,name);
- if (!Quiet) printf("<%.*s %s> ", NAMEDATALEN,
- attrtypes[attnum]->attname.data, type);
+ if (!Quiet) printf("<%s %s> ", attrtypes[attnum]->attname.data, type);
attrtypes[attnum]->attnum = 1 + attnum; /* fillatt */
attlen = attrtypes[attnum]->attlen = Procid[t].len;
attrtypes[attnum]->attbyval = (attlen==1) || (attlen==2)||(attlen==4);
@@ -775,7 +773,7 @@ gettype(char *type)
}
}
if (DebugMode)
- printf("bootstrap.c: External Type: %.*s\n", NAMEDATALEN, type);
+ printf("bootstrap.c: External Type: %s\n", type);
rdesc = heap_openr(TypeRelationName);
sdesc = heap_beginscan(rdesc, 0, NowTimeQual, 0, (ScanKey)NULL);
i = 0;
diff --git a/src/backend/catalog/catalog.c b/src/backend/catalog/catalog.c
index bf04012d2ec..90e6dedc18d 100644
--- a/src/backend/catalog/catalog.c
+++ b/src/backend/catalog/catalog.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/catalog/catalog.c,v 1.6 1997/01/10 20:16:34 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/catalog/catalog.c,v 1.7 1997/08/18 20:51:59 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -33,7 +33,7 @@ relpath(char relname[])
if (IsSharedSystemRelationName(relname)) {
path = (char *) palloc(strlen(DataDir) + sizeof(NameData) + 2);
- sprintf(path, "%s/%.*s", DataDir, NAMEDATALEN, relname);
+ sprintf(path, "%s/%s", DataDir, relname);
return (path);
}
return(relname);
diff --git a/src/backend/catalog/pg_operator.c b/src/backend/catalog/pg_operator.c
index 10b6145138f..1151f9f2e69 100644
--- a/src/backend/catalog/pg_operator.c
+++ b/src/backend/catalog/pg_operator.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/catalog/pg_operator.c,v 1.10 1997/08/12 22:52:11 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/catalog/pg_operator.c,v 1.11 1997/08/18 20:52:04 momjian Exp $
*
* NOTES
* these routines moved here from commands/define.c and somewhat cleaned up.
@@ -483,8 +483,8 @@ OperatorDef(char *operatorName,
rightTypeName);
if (OidIsValid(operatorObjectId) && !definedOK)
- elog(WARN, "OperatorDef: operator \"%-.*s\" already defined",
- NAMEDATALEN, operatorName);
+ elog(WARN, "OperatorDef: operator \"%s\" already defined",
+ operatorName);
if (leftTypeName)
leftTypeId = TypeGet(leftTypeName, &leftDefined);
diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c
index 9b009b4ab6c..0e2a5eea7d1 100644
--- a/src/backend/commands/cluster.c
+++ b/src/backend/commands/cluster.c
@@ -14,7 +14,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/commands/cluster.c,v 1.11 1997/08/03 02:34:45 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/commands/cluster.c,v 1.12 1997/08/18 20:52:07 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -116,15 +116,15 @@ cluster(char oldrelname[], char oldindexname[])
OldHeap = heap_openr(oldrelname);
if (!RelationIsValid(OldHeap)) {
- elog(WARN, "cluster: unknown relation: \"%-.*s\"",
- NAMEDATALEN, oldrelname);
+ elog(WARN, "cluster: unknown relation: \"%s\"",
+ oldrelname);
}
OIDOldHeap = OldHeap->rd_id; /* Get OID for the index scan */
OldIndex=index_openr(oldindexname);/* Open old index relation */
if (!RelationIsValid(OldIndex)) {
- elog(WARN, "cluster: unknown index: \"%-.*s\"",
- NAMEDATALEN, oldindexname);
+ elog(WARN, "cluster: unknown index: \"%s\"",
+ oldindexname);
}
OIDOldIndex = OldIndex->rd_id; /* OID for the index scan */
diff --git a/src/backend/commands/command.c b/src/backend/commands/command.c
index 526adbdda71..74941993e47 100644
--- a/src/backend/commands/command.c
+++ b/src/backend/commands/command.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/commands/Attic/command.c,v 1.8 1997/08/12 22:52:18 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/commands/Attic/command.c,v 1.9 1997/08/18 20:52:11 momjian Exp $
*
* NOTES
* The PortalExecutorHeapMemory crap needs to be eliminated
@@ -117,8 +117,8 @@ PerformPortalFetch(char *name,
*/
portal = GetPortalByName(name);
if (! PortalIsValid(portal)) {
- elog(NOTICE, "PerformPortalFetch: portal \"%-.*s\" not found",
- NAMEDATALEN, name);
+ elog(NOTICE, "PerformPortalFetch: portal \"%s\" not found",
+ name);
return;
}
@@ -202,8 +202,8 @@ PerformPortalClose(char *name, CommandDest dest)
*/
portal = GetPortalByName(name);
if (! PortalIsValid(portal)) {
- elog(NOTICE, "PerformPortalClose: portal \"%-.*s\" not found",
- NAMEDATALEN, name);
+ elog(NOTICE, "PerformPortalClose: portal \"%s\" not found",
+ name);
return;
}
@@ -272,8 +272,8 @@ PerformAddAttribute(char *relationName,
* normally, only the owner of a class can change its schema.
*/
if (IsSystemRelationName(relationName))
- elog(WARN, "PerformAddAttribute: class \"%-.*s\" is a system catalog",
- NAMEDATALEN, relationName);
+ elog(WARN, "PerformAddAttribute: class \"%s\" is a system catalog",
+ relationName);
#ifndef NO_SECURITY
if (!pg_ownercheck(userName, relationName, RELNAME))
elog(WARN, "PerformAddAttribute: you do not own class \"%s\"",
@@ -296,8 +296,8 @@ PerformAddAttribute(char *relationName,
relrdesc = heap_openr(relationName);
if (!RelationIsValid(relrdesc)) {
- elog(WARN, "PerformAddAttribute: unknown relation: \"%-.*s\"",
- NAMEDATALEN, relationName);
+ elog(WARN, "PerformAddAttribute: unknown relation: \"%s\"",
+ relationName);
}
myrelid = relrdesc->rd_id;
heap_close(relrdesc);
diff --git a/src/backend/commands/creatinh.c b/src/backend/commands/creatinh.c
index 3f27ade8885..1113a84e26f 100644
--- a/src/backend/commands/creatinh.c
+++ b/src/backend/commands/creatinh.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/commands/Attic/creatinh.c,v 1.10 1997/08/12 22:52:20 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/commands/Attic/creatinh.c,v 1.11 1997/08/18 20:52:16 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -549,7 +549,7 @@ MakeArchiveName(Oid relationId)
* of the relation they archive. Create a string containing
* this name and find the reldesc for the archive relation.
*/
- arch = palloc(NAMEDATALEN);
+ arch = palloc(NAMEDATALEN);
sprintf(arch, "a,%d",relationId);
return arch;
diff --git a/src/backend/commands/explain.c b/src/backend/commands/explain.c
index db852f44780..7d1f34d0327 100644
--- a/src/backend/commands/explain.c
+++ b/src/backend/commands/explain.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/commands/explain.c,v 1.9 1997/04/05 06:42:32 vadim Exp $
+ * $Header: /cvsroot/pgsql/src/backend/commands/explain.c,v 1.10 1997/08/18 20:52:17 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -177,7 +177,7 @@ explain_outNode(StringInfo str, Plan *plan, int indent, ExplainState *es)
case T_IndexScan:
if (((Scan*)plan)->scanrelid > 0) {
RangeTblEntry *rte = nth(((Scan*)plan)->scanrelid-1, es->rtable);
- sprintf(buf, " on %.*s", NAMEDATALEN, rte->refname);
+ sprintf(buf, " on %s", rte->refname);
appendStringInfo(str, buf);
}
break;
diff --git a/src/backend/commands/remove.c b/src/backend/commands/remove.c
index 8d2c1ce65da..7f0198a10b7 100644
--- a/src/backend/commands/remove.c
+++ b/src/backend/commands/remove.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/commands/Attic/remove.c,v 1.9 1997/06/03 06:23:24 vadim Exp $
+ * $Header: /cvsroot/pgsql/src/backend/commands/Attic/remove.c,v 1.10 1997/08/18 20:52:17 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -393,8 +393,8 @@ RemoveFunction(char *functionName, /* function name to be removed */
/* ok, function has been found */
if (the_proc->prolang == INTERNALlanguageId)
- elog(WARN, "RemoveFunction: function \"%-.*s\" is built-in",
- NAMEDATALEN, functionName);
+ elog(WARN, "RemoveFunction: function \"%s\" is built-in",
+ functionName);
ItemPointerCopy(&tup->t_ctid, &itemPointerData);
heap_delete(relation, &itemPointerData);
diff --git a/src/backend/commands/rename.c b/src/backend/commands/rename.c
index 56672659113..5d4e4ab2bb8 100644
--- a/src/backend/commands/rename.c
+++ b/src/backend/commands/rename.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/commands/Attic/rename.c,v 1.6 1997/08/12 22:52:27 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/commands/Attic/rename.c,v 1.7 1997/08/18 20:52:18 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -78,13 +78,13 @@ renameatt(char *relname,
* normally, only the owner of a class can change its schema.
*/
if (IsSystemRelationName(relname))
- elog(WARN, "renameatt: class \"%-.*s\" is a system catalog",
- NAMEDATALEN, relname);
+ elog(WARN, "renameatt: class \"%s\" is a system catalog",
+ relname);
#ifndef NO_SECURITY
if (!IsBootstrapProcessingMode() &&
!pg_ownercheck(userName, relname, RELNAME))
- elog(WARN, "renameatt: you do not own class \"%-.*s\"",
- NAMEDATALEN, relname);
+ elog(WARN, "renameatt: you do not own class \"%s\"",
+ relname);
#endif
/*
@@ -102,8 +102,8 @@ renameatt(char *relname,
relrdesc = heap_openr(relname);
if (!RelationIsValid(relrdesc)) {
- elog(WARN, "renameatt: unknown relation: \"%-.*s\"",
- NAMEDATALEN, relname);
+ elog(WARN, "renameatt: unknown relation: \"%s\"",
+ relname);
}
myrelid = relrdesc->rd_id;
heap_close(relrdesc);
@@ -139,8 +139,8 @@ renameatt(char *relname,
reltup = ClassNameIndexScan(relrdesc, relname);
if (!PointerIsValid(reltup)) {
heap_close(relrdesc);
- elog(WARN, "renameatt: relation \"%-.*s\" nonexistent",
- NAMEDATALEN, relname);
+ elog(WARN, "renameatt: relation \"%s\" nonexistent",
+ relname);
return;
}
heap_close(relrdesc);
@@ -149,20 +149,20 @@ renameatt(char *relname,
oldatttup = AttributeNameIndexScan(attrdesc, reltup->t_oid, oldattname);
if (!PointerIsValid(oldatttup)) {
heap_close(attrdesc);
- elog(WARN, "renameatt: attribute \"%-.*s\" nonexistent",
- NAMEDATALEN, oldattname);
+ elog(WARN, "renameatt: attribute \"%s\" nonexistent",
+ oldattname);
}
if (((AttributeTupleForm ) GETSTRUCT(oldatttup))->attnum < 0) {
- elog(WARN, "renameatt: system attribute \"%-.*s\" not renamed",
- NAMEDATALEN, oldattname);
+ elog(WARN, "renameatt: system attribute \"%s\" not renamed",
+ oldattname);
}
newatttup = AttributeNameIndexScan(attrdesc, reltup->t_oid, newattname);
if (PointerIsValid(newatttup)) {
pfree(oldatttup);
heap_close(attrdesc);
- elog(WARN, "renameatt: attribute \"%-.*s\" exists",
- NAMEDATALEN, newattname);
+ elog(WARN, "renameatt: attribute \"%s\" exists",
+ newattname);
}
namestrcpy(&(((AttributeTupleForm)(GETSTRUCT(oldatttup)))->attname),
@@ -207,13 +207,13 @@ renamerel(char oldrelname[], char newrelname[])
Relation idescs[Num_pg_class_indices];
if (IsSystemRelationName(oldrelname)) {
- elog(WARN, "renamerel: system relation \"%-.*s\" not renamed",
- NAMEDATALEN, oldrelname);
+ elog(WARN, "renamerel: system relation \"%s\" not renamed",
+ oldrelname);
return;
}
if (IsSystemRelationName(newrelname)) {
- elog(WARN, "renamerel: Illegal class name: \"%-.*s\" -- pg_ is reserved for system catalogs",
- NAMEDATALEN, newrelname);
+ elog(WARN, "renamerel: Illegal class name: \"%s\" -- pg_ is reserved for system catalogs",
+ newrelname);
return;
}
@@ -222,16 +222,16 @@ renamerel(char oldrelname[], char newrelname[])
if (!PointerIsValid(oldreltup)) {
heap_close(relrdesc);
- elog(WARN, "renamerel: relation \"%-.*s\" does not exist",
- NAMEDATALEN, oldrelname);
+ elog(WARN, "renamerel: relation \"%s\" does not exist",
+ oldrelname);
}
newreltup = ClassNameIndexScan(relrdesc, newrelname);
if (PointerIsValid(newreltup)) {
pfree(oldreltup);
heap_close(relrdesc);
- elog(WARN, "renamerel: relation \"%-.*s\" exists",
- NAMEDATALEN, newrelname);
+ elog(WARN, "renamerel: relation \"%s\" exists",
+ newrelname);
}
/* rename the directory first, so if this fails the rename's not done */
diff --git a/src/backend/commands/vacuum.c b/src/backend/commands/vacuum.c
index 1b95e9c8026..36fb0948be0 100644
--- a/src/backend/commands/vacuum.c
+++ b/src/backend/commands/vacuum.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/commands/vacuum.c,v 1.38 1997/08/12 22:52:29 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/commands/vacuum.c,v 1.39 1997/08/18 20:52:21 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -318,8 +318,8 @@ vc_getrels(NameData *VacRelP)
(rname[3] == 'v' || rname[3] == 'x') &&
rname[4] >= '0' && rname[4] <= '9')
{
- elog (NOTICE, "Rel %.*s: can't vacuum LargeObjects now",
- NAMEDATALEN, rname);
+ elog (NOTICE, "Rel %s: can't vacuum LargeObjects now",
+ rname);
ReleaseBuffer(buf);
continue;
}
@@ -446,8 +446,8 @@ vc_vacone (Oid relid, bool analyze, List *va_cols)
List *le;
if ( length (va_cols) > attr_cnt )
- elog (WARN, "vacuum: too many attributes specified for relation %.*s",
- NAMEDATALEN, (RelationGetRelationName(onerel))->data);
+ elog (WARN, "vacuum: too many attributes specified for relation %s",
+ (RelationGetRelationName(onerel))->data);
attnums = (int*) palloc (attr_cnt * sizeof (int));
foreach (le, va_cols)
{
@@ -462,8 +462,8 @@ vc_vacone (Oid relid, bool analyze, List *va_cols)
attnums[tcnt++] = i;
else
{
- elog (WARN, "vacuum: there is no attribute %s in %.*s",
- col, NAMEDATALEN, (RelationGetRelationName(onerel))->data);
+ elog (WARN, "vacuum: there is no attribute %s in %s",
+ col, (RelationGetRelationName(onerel))->data);
}
}
attr_cnt = tcnt;
@@ -657,8 +657,8 @@ vc_scanheap (VRelStats *vacrelstats, Relation onerel,
vpc->vpd_noff = 0;
if (PageIsNew(page)) {
- elog (NOTICE, "Rel %.*s: Uninitialized page %u - fixing",
- NAMEDATALEN, relname, blkno);
+ elog (NOTICE, "Rel %s: Uninitialized page %u - fixing",
+ relname, blkno);
PageInit (page, BufferGetPageSize (buf), 0);
vpc->vpd_free = ((PageHeader)page)->pd_upper - ((PageHeader)page)->pd_lower;
frsize += (vpc->vpd_free - sizeof (ItemIdData));
@@ -718,8 +718,8 @@ vc_scanheap (VRelStats *vacrelstats, Relation onerel,
}
else
{
- elog (NOTICE, "Rel %.*s: TID %u/%u: InsertTransactionInProgress %u - can't shrink relation",
- NAMEDATALEN, relname, blkno, offnum, htup->t_xmin);
+ elog (NOTICE, "Rel %s: TID %u/%u: InsertTransactionInProgress %u - can't shrink relation",
+ relname, blkno, offnum, htup->t_xmin);
do_shrinking = false;
}
}
@@ -743,8 +743,8 @@ vc_scanheap (VRelStats *vacrelstats, Relation onerel,
}
else
{
- elog (NOTICE, "Rel %.*s: TID %u/%u: DeleteTransactionInProgress %u - can't shrink relation",
- NAMEDATALEN, relname, blkno, offnum, htup->t_xmax);
+ elog (NOTICE, "Rel %s: TID %u/%u: DeleteTransactionInProgress %u - can't shrink relation",
+ relname, blkno, offnum, htup->t_xmax);
do_shrinking = false;
}
}
@@ -754,9 +754,9 @@ vc_scanheap (VRelStats *vacrelstats, Relation onerel,
*/
if ( !TransactionIdIsValid((TransactionId)htup->t_xmin) )
{
- elog (NOTICE, "Rel %.*s: TID %u/%u: INSERT_TRANSACTION_ID IS INVALID. \
+ elog (NOTICE, "Rel %s: TID %u/%u: INSERT_TRANSACTION_ID IS INVALID. \
DELETE_TRANSACTION_ID_VALID %d, TUPGONE %d.",
- NAMEDATALEN, relname, blkno, offnum,
+ relname, blkno, offnum,
TransactionIdIsValid((TransactionId)htup->t_xmax),
tupgone);
}
@@ -769,8 +769,8 @@ DELETE_TRANSACTION_ID_VALID %d, TUPGONE %d.",
if ( !ItemPointerIsValid (itemptr) ||
BlockIdGetBlockNumber(&(itemptr->ip_blkid)) != blkno )
{
- elog (NOTICE, "Rel %.*s: TID %u/%u: TID IN TUPLEHEADER %u/%u IS NOT THE SAME. TUPGONE %d.",
- NAMEDATALEN, relname, blkno, offnum,
+ elog (NOTICE, "Rel %s: TID %u/%u: TID IN TUPLEHEADER %u/%u IS NOT THE SAME. TUPGONE %d.",
+ relname, blkno, offnum,
BlockIdGetBlockNumber(&(itemptr->ip_blkid)),
itemptr->ip_posid, tupgone);
}
@@ -780,14 +780,14 @@ DELETE_TRANSACTION_ID_VALID %d, TUPGONE %d.",
*/
if ( htup->t_len != itemid->lp_len )
{
- elog (NOTICE, "Rel %.*s: TID %u/%u: TUPLE_LEN IN PAGEHEADER %u IS NOT THE SAME AS IN TUPLEHEADER %u. TUPGONE %d.",
- NAMEDATALEN, relname, blkno, offnum,
+ elog (NOTICE, "Rel %s: TID %u/%u: TUPLE_LEN IN PAGEHEADER %u IS NOT THE SAME AS IN TUPLEHEADER %u. TUPGONE %d.",
+ relname, blkno, offnum,
itemid->lp_len, htup->t_len, tupgone);
}
if ( !OidIsValid(htup->t_oid) )
{
- elog (NOTICE, "Rel %.*s: TID %u/%u: OID IS INVALID. TUPGONE %d.",
- NAMEDATALEN, relname, blkno, offnum, tupgone);
+ elog (NOTICE, "Rel %s: TID %u/%u: OID IS INVALID. TUPGONE %d.",
+ relname, blkno, offnum, tupgone);
}
if (tupgone) {
@@ -890,9 +890,9 @@ DELETE_TRANSACTION_ID_VALID %d, TUPGONE %d.",
getrusage(RUSAGE_SELF, &ru1);
- elog (MESSAGE_LEVEL, "Rel %.*s: Pages %u: Changed %u, Reapped %u, Empty %u, New %u; \
+ elog (MESSAGE_LEVEL, "Rel %s: Pages %u: Changed %u, Reapped %u, Empty %u, New %u; \
Tup %u: Vac %u, Crash %u, UnUsed %u, MinLen %u, MaxLen %u; Re-using: Free/Avail. Space %u/%u; EndEmpty/Avail. Pages %u/%u. Elapsed %u/%u sec.",
- NAMEDATALEN, relname,
+ relname,
nblocks, nchpg, Vvpl->vpl_npages, nempg, nnepg,
ntups, nvac, ncrash, nunused, min_tlen, max_tlen,
frsize, frsusf, nemend, Fvpl->vpl_npages,
@@ -1246,9 +1246,9 @@ failed to add item with len = %u to page %u (free space %u, nusd %u, noff %u)",
getrusage(RUSAGE_SELF, &ru1);
- elog (MESSAGE_LEVEL, "Rel %.*s: Pages: %u --> %u; Tuple(s) moved: %u. \
+ elog (MESSAGE_LEVEL, "Rel %s: Pages: %u --> %u; Tuple(s) moved: %u. \
Elapsed %u/%u sec.",
- NAMEDATALEN, (RelationGetRelationName(onerel))->data,
+ (RelationGetRelationName(onerel))->data,
nblocks, blkno, nmoved,
ru1.ru_stime.tv_sec - ru0.ru_stime.tv_sec,
ru1.ru_utime.tv_sec - ru0.ru_utime.tv_sec);
@@ -1371,8 +1371,8 @@ vc_vacheap (VRelStats *vacrelstats, Relation onerel, VPageList Vvpl)
{
Assert ( vacrelstats->npages >= Vvpl->vpl_nemend );
nblocks = vacrelstats->npages - Vvpl->vpl_nemend;
- elog (MESSAGE_LEVEL, "Rel %.*s: Pages: %u --> %u.",
- NAMEDATALEN, (RelationGetRelationName(onerel))->data,
+ elog (MESSAGE_LEVEL, "Rel %s: Pages: %u --> %u.",
+ (RelationGetRelationName(onerel))->data,
vacrelstats->npages, nblocks);
/*
@@ -1451,14 +1451,14 @@ vc_scanoneind (Relation indrel, int nhtups)
getrusage(RUSAGE_SELF, &ru1);
- elog (MESSAGE_LEVEL, "Ind %.*s: Pages %u; Tuples %u. Elapsed %u/%u sec.",
- NAMEDATALEN, indrel->rd_rel->relname.data, nipages, nitups,
+ elog (MESSAGE_LEVEL, "Ind %s: Pages %u; Tuples %u. Elapsed %u/%u sec.",
+ indrel->rd_rel->relname.data, nipages, nitups,
ru1.ru_stime.tv_sec - ru0.ru_stime.tv_sec,
ru1.ru_utime.tv_sec - ru0.ru_utime.tv_sec);
if ( nitups != nhtups )
- elog (NOTICE, "Ind %.*s: NUMBER OF INDEX' TUPLES (%u) IS NOT THE SAME AS HEAP' (%u)",
- NAMEDATALEN, indrel->rd_rel->relname.data, nitups, nhtups);
+ elog (NOTICE, "Ind %s: NUMBER OF INDEX' TUPLES (%u) IS NOT THE SAME AS HEAP' (%u)",
+ indrel->rd_rel->relname.data, nitups, nhtups);
} /* vc_scanoneind */
@@ -1508,8 +1508,8 @@ vc_vaconeind(VPageList vpl, Relation indrel, int nhtups)
#endif
if ( vp->vpd_noff == 0 )
{ /* this is EmptyPage !!! */
- elog (NOTICE, "Ind %.*s: pointer to EmptyPage (blk %u off %u) - fixing",
- NAMEDATALEN, indrel->rd_rel->relname.data,
+ elog (NOTICE, "Ind %s: pointer to EmptyPage (blk %u off %u) - fixing",
+ indrel->rd_rel->relname.data,
vp->vpd_blkno, ItemPointerGetOffsetNumber(heapptr));
}
++nvac;
@@ -1530,14 +1530,14 @@ vc_vaconeind(VPageList vpl, Relation indrel, int nhtups)
getrusage(RUSAGE_SELF, &ru1);
- elog (MESSAGE_LEVEL, "Ind %.*s: Pages %u; Tuples %u: Deleted %u. Elapsed %u/%u sec.",
- NAMEDATALEN, indrel->rd_rel->relname.data, nipages, nitups, nvac,
+ elog (MESSAGE_LEVEL, "Ind %s: Pages %u; Tuples %u: Deleted %u. Elapsed %u/%u sec.",
+ indrel->rd_rel->relname.data, nipages, nitups, nvac,
ru1.ru_stime.tv_sec - ru0.ru_stime.tv_sec,
ru1.ru_utime.tv_sec - ru0.ru_utime.tv_sec);
if ( nitups != nhtups )
- elog (NOTICE, "Ind %.*s: NUMBER OF INDEX' TUPLES (%u) IS NOT THE SAME AS HEAP' (%u)",
- NAMEDATALEN, indrel->rd_rel->relname.data, nitups, nhtups);
+ elog (NOTICE, "Ind %s: NUMBER OF INDEX' TUPLES (%u) IS NOT THE SAME AS HEAP' (%u)",
+ indrel->rd_rel->relname.data, nitups, nhtups);
} /* vc_vaconeind */
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
index 7c93fe2af66..b79be5d746f 100644
--- a/src/backend/executor/execMain.c
+++ b/src/backend/executor/execMain.c
@@ -26,7 +26,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.14 1997/08/12 22:52:33 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.15 1997/08/18 20:52:25 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -308,9 +308,9 @@ ExecCheckPerms(CmdType operation,
if (!HeapTupleIsValid(htp))
elog(WARN, "ExecCheckPerms: bogus RT relid: %d",
relid);
- strncpy(rname.data,
+ strNcpy(rname.data,
((Form_pg_class) GETSTRUCT(htp))->relname.data,
- NAMEDATALEN);
+ NAMEDATALEN-1);
if (i == resultRelation) { /* this is the result relation */
qvars = pull_varnos(parseTree->qual);
tvars = pull_varnos((Node*)parseTree->targetList);
@@ -348,10 +348,6 @@ ExecCheckPerms(CmdType operation,
++i;
}
if (!ok) {
-/*
- elog(WARN, "%s on \"%-.*s\": permission denied", opstr,
- NAMEDATALEN, rname.data);
-*/
elog(WARN, "%s: %s", rname.data, aclcheck_error_strings[aclcheck_result]);
}
}
diff --git a/src/backend/executor/execUtils.c b/src/backend/executor/execUtils.c
index 45e8fefb70c..a44c0f024ac 100644
--- a/src/backend/executor/execUtils.c
+++ b/src/backend/executor/execUtils.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/executor/execUtils.c,v 1.9 1997/06/13 03:24:26 vadim Exp $
+ * $Header: /cvsroot/pgsql/src/backend/executor/execUtils.c,v 1.10 1997/08/18 20:52:27 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -491,7 +491,7 @@ ExecSetTypeInfo(int index,
att->attrelid = 0; /* dummy value */
if (attName != (char *) NULL)
- strncpy(att->attname.data, attName, NAMEDATALEN);
+ strNcpy(att->attname.data, attName, NAMEDATALEN-1);
else
memset(att->attname.data,0,NAMEDATALEN);
diff --git a/src/backend/libpq/be-dumpdata.c b/src/backend/libpq/be-dumpdata.c
index d5fb4a941d7..b47e79fa65c 100644
--- a/src/backend/libpq/be-dumpdata.c
+++ b/src/backend/libpq/be-dumpdata.c
@@ -8,7 +8,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/libpq/Attic/be-dumpdata.c,v 1.4 1996/11/10 03:00:23 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/libpq/Attic/be-dumpdata.c,v 1.5 1997/08/18 20:52:31 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -189,7 +189,7 @@ be_typeinit(PortalEntry *entry,
if (natts > 0) {
group->types = pbuf_addTypes(natts);
for (i = 0; i < natts; ++i) {
- strncpy(group->types[i].name, attrs[i]->attname.data, NAMEDATALEN);
+ strncpy(group->types[i].name, attrs[i]->attname.data, NAMEDATALEN);
group->types[i].adtid = attrs[i]->atttypid;
group->types[i].adtsize = attrs[i]->attlen;
}
diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c
index 121e9d3ff97..a41848c6188 100644
--- a/src/backend/nodes/outfuncs.c
+++ b/src/backend/nodes/outfuncs.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/nodes/outfuncs.c,v 1.5 1996/12/26 17:46:56 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/nodes/outfuncs.c,v 1.6 1997/08/18 20:52:43 momjian Exp $
*
* NOTES
* Every (plan) node in POSTGRES has an associated "out" routine which
@@ -501,7 +501,7 @@ _outResdom(StringInfo str, Resdom *node)
appendStringInfo(str,buf);
sprintf(buf, " :reslen %d", node->reslen);
appendStringInfo(str,buf);
- sprintf(buf, " :resname \"%.*s\"", NAMEDATALEN,
+ sprintf(buf, " :resname \"%s\"",
((node->resname) ? ((char *) node->resname) : "null"));
appendStringInfo(str,buf);
sprintf(buf, " :reskey %d", node->reskey);
@@ -643,7 +643,7 @@ _outAggreg(StringInfo str, Aggreg *node)
sprintf(buf, "AGGREG");
appendStringInfo(str,buf);
- sprintf(buf, " :aggname \"%.*s\"", NAMEDATALEN, (char*)node->aggname);
+ sprintf(buf, " :aggname \"%s\"", (char*)node->aggname);
appendStringInfo(str,buf);
sprintf(buf, " :basetype %u", node->basetype);
appendStringInfo(str,buf);
@@ -789,7 +789,7 @@ _outParam(StringInfo str, Param *node)
appendStringInfo(str,buf);
sprintf(buf, " :paramid %hd", node->paramid);
appendStringInfo(str,buf);
- sprintf(buf, " :paramname \"%.*s\"", NAMEDATALEN, node->paramname);
+ sprintf(buf, " :paramname \"%s\"", node->paramname);
appendStringInfo(str,buf);
sprintf(buf, " :paramtype %u", node->paramtype);
appendStringInfo(str,buf);
@@ -932,14 +932,14 @@ _outRangeTblEntry(StringInfo str, RangeTblEntry *node)
sprintf(buf, "RTE");
appendStringInfo(str,buf);
- sprintf(buf, " :relname \"%.*s\"", NAMEDATALEN,
+ sprintf(buf, " :relname \"%s\"",
((node->relname) ? ((char *) node->relname) : "null"));
appendStringInfo(str,buf);
sprintf(buf, " :inh %d ", node->inh);
appendStringInfo(str,buf);
- sprintf(buf, " :refname \"%.*s\"", NAMEDATALEN,
+ sprintf(buf, " :refname \"%s\"",
((node->refname) ? ((char *) node->refname) : "null"));
appendStringInfo(str,buf);
diff --git a/src/backend/parser/analyze.c b/src/backend/parser/analyze.c
index 23136e6d0a2..78167e1a0d9 100644
--- a/src/backend/parser/analyze.c
+++ b/src/backend/parser/analyze.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.32 1997/08/12 22:53:17 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.33 1997/08/18 20:53:00 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -1374,10 +1374,10 @@ make_targetlist_expr(ParseState *pstate,
attrlen);
} else {
/* currently, we can't handle casting of expressions */
- elog(WARN, "parser: attribute '%s' is of type '%.*s' but expression is of type '%.*s'",
+ elog(WARN, "parser: attribute '%s' is of type '%s' but expression is of type '%s'",
colname,
- NAMEDATALEN, get_id_typname(attrtype),
- NAMEDATALEN, get_id_typname(type_id));
+ get_id_typname(attrtype),
+ get_id_typname(type_id));
}
}
diff --git a/src/backend/parser/dbcommands.c b/src/backend/parser/dbcommands.c
index 224128eaf44..1b8104d3a2a 100644
--- a/src/backend/parser/dbcommands.c
+++ b/src/backend/parser/dbcommands.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/parser/Attic/dbcommands.c,v 1.4 1997/08/18 02:14:44 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/parser/Attic/dbcommands.c,v 1.5 1997/08/18 20:53:03 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -170,8 +170,8 @@ check_permissions(char *command,
/* Check to make sure user has permission to use createdb */
if (!use_createdb) {
- elog(WARN, "user \"%-.*s\" is not allowed to create/destroy databases",
- NAMEDATALEN, userName);
+ elog(WARN, "user \"%s\" is not allowed to create/destroy databases",
+ userName);
}
/* Make sure we are not mucking with the template database */
diff --git a/src/backend/storage/buffer/bufmgr.c b/src/backend/storage/buffer/bufmgr.c
index 256de81fc0d..47b5d22b6c8 100644
--- a/src/backend/storage/buffer/bufmgr.c
+++ b/src/backend/storage/buffer/bufmgr.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/storage/buffer/bufmgr.c,v 1.17 1997/08/18 02:14:49 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/storage/buffer/bufmgr.c,v 1.18 1997/08/18 20:53:08 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -1419,9 +1419,9 @@ PrintBufferDescs()
if (IsUnderPostmaster) {
SpinAcquire(BufMgrLock);
for (i = 0; i < NBuffers; ++i, ++buf) {
- elog(NOTICE, "[%02d] (freeNext=%d, freePrev=%d, relname=%.*s, \
+ elog(NOTICE, "[%02d] (freeNext=%d, freePrev=%d, relname=%s, \
blockNum=%d, flags=0x%x, refcount=%d %d)",
- i, buf->freeNext, buf->freePrev, NAMEDATALEN,
+ i, buf->freeNext, buf->freePrev,
buf->sb_relname, buf->tag.blockNum, buf->flags,
buf->refcount, PrivateRefCount[i]);
}
@@ -1445,9 +1445,9 @@ PrintPinnedBufs()
SpinAcquire(BufMgrLock);
for (i = 0; i < NBuffers; ++i, ++buf) {
if (PrivateRefCount[i] > 0)
- elog(NOTICE, "[%02d] (freeNext=%d, freePrev=%d, relname=%.*s, \
+ elog(NOTICE, "[%02d] (freeNext=%d, freePrev=%d, relname=%s, \
blockNum=%d, flags=0x%x, refcount=%d %d)\n",
- i, buf->freeNext, buf->freePrev, NAMEDATALEN, buf->sb_relname,
+ i, buf->freeNext, buf->freePrev, buf->sb_relname,
buf->tag.blockNum, buf->flags,
buf->refcount, PrivateRefCount[i]);
}
diff --git a/src/backend/storage/smgr/md.c b/src/backend/storage/smgr/md.c
index ee6ffbd905e..2688ad3aed1 100644
--- a/src/backend/storage/smgr/md.c
+++ b/src/backend/storage/smgr/md.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/storage/smgr/md.c,v 1.17 1997/08/12 22:54:13 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/storage/smgr/md.c,v 1.18 1997/08/18 20:53:14 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -468,17 +468,17 @@ mdblindwrt(char *dbstr,
if (dbid == (Oid) 0) {
path = (char *) palloc(strlen(DataDir) + sizeof(NameData) + 2 + nchars);
if (segno == 0)
- sprintf(path, "%s/%.*s", DataDir, NAMEDATALEN, relstr);
+ sprintf(path, "%s/%s", DataDir, relstr);
else
- sprintf(path, "%s/%.*s.%d", DataDir, NAMEDATALEN, relstr, segno);
+ sprintf(path, "%s/%s.%d", DataDir, relstr, segno);
} else {
path = (char *) palloc(strlen(DataDir) + strlen("/base/") + 2 * sizeof(NameData) + 2 + nchars);
if (segno == 0)
- sprintf(path, "%s/base/%.*s/%.*s", DataDir, NAMEDATALEN,
- dbstr, NAMEDATALEN, relstr);
+ sprintf(path, "%s/base/%s/%s", DataDir,
+ dbstr, relstr);
else
- sprintf(path, "%s/base/%.*s/%.*s.%d", DataDir, NAMEDATALEN, dbstr,
- NAMEDATALEN, relstr, segno);
+ sprintf(path, "%s/base/%s/%s.%d", DataDir, dbstr,
+ relstr, segno);
}
if ((fd = open(path, O_RDWR, 0600)) < 0)
@@ -563,8 +563,8 @@ mdtruncate (Relation reln, int nblocks)
curnblk = mdnblocks (reln);
if ( curnblk / RELSEG_SIZE > 0 )
{
- elog (NOTICE, "Can't truncate multi-segments relation %.*s",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog (NOTICE, "Can't truncate multi-segments relation %s",
+ &(reln->rd_rel->relname.data[0]));
return (curnblk);
}
diff --git a/src/backend/storage/smgr/smgr.c b/src/backend/storage/smgr/smgr.c
index 3181bbf7f61..ecf6a67d6d2 100644
--- a/src/backend/storage/smgr/smgr.c
+++ b/src/backend/storage/smgr/smgr.c
@@ -10,7 +10,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/storage/smgr/smgr.c,v 1.6 1997/05/22 17:08:35 vadim Exp $
+ * $Header: /cvsroot/pgsql/src/backend/storage/smgr/smgr.c,v 1.7 1997/08/18 20:53:18 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -125,8 +125,8 @@ smgrcreate(int16 which, Relation reln)
int fd;
if ((fd = (*(smgrsw[which].smgr_create))(reln)) < 0)
- elog(WARN, "cannot open %.*s",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot open %s",
+ &(reln->rd_rel->relname.data[0]));
return (fd);
}
@@ -142,8 +142,8 @@ smgrunlink(int16 which, Relation reln)
int status;
if ((status = (*(smgrsw[which].smgr_unlink))(reln)) == SM_FAIL)
- elog(WARN, "cannot unlink %.*s",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot unlink %s",
+ &(reln->rd_rel->relname.data[0]));
return (status);
}
@@ -162,8 +162,8 @@ smgrextend(int16 which, Relation reln, char *buffer)
status = (*(smgrsw[which].smgr_extend))(reln, buffer);
if (status == SM_FAIL)
- elog(WARN, "%.*s: cannot extend",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "%s: cannot extend",
+ &(reln->rd_rel->relname.data[0]));
return (status);
}
@@ -180,8 +180,8 @@ smgropen(int16 which, Relation reln)
int fd;
if ((fd = (*(smgrsw[which].smgr_open))(reln)) < 0)
- elog(WARN, "cannot open %.*s",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot open %s",
+ &(reln->rd_rel->relname.data[0]));
return (fd);
}
@@ -202,8 +202,8 @@ int
smgrclose(int16 which, Relation reln)
{
if ((*(smgrsw[which].smgr_close))(reln) == SM_FAIL)
- elog(WARN, "cannot close %.*s",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot close %s",
+ &(reln->rd_rel->relname.data[0]));
return (SM_SUCCESS);
}
@@ -226,8 +226,8 @@ smgrread(int16 which, Relation reln, BlockNumber blocknum, char *buffer)
status = (*(smgrsw[which].smgr_read))(reln, blocknum, buffer);
if (status == SM_FAIL)
- elog(WARN, "cannot read block %d of %.*s",
- blocknum, NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot read block %d of %s",
+ blocknum, &(reln->rd_rel->relname.data[0]));
return (status);
}
@@ -248,8 +248,8 @@ smgrwrite(int16 which, Relation reln, BlockNumber blocknum, char *buffer)
status = (*(smgrsw[which].smgr_write))(reln, blocknum, buffer);
if (status == SM_FAIL)
- elog(WARN, "cannot write block %d of %.*s",
- blocknum, NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot write block %d of %s",
+ blocknum, &(reln->rd_rel->relname.data[0]));
return (status);
}
@@ -265,8 +265,8 @@ smgrflush(int16 which, Relation reln, BlockNumber blocknum, char *buffer)
status = (*(smgrsw[which].smgr_flush))(reln, blocknum, buffer);
if (status == SM_FAIL)
- elog(WARN, "cannot flush block %d of %.*s to stable store",
- blocknum, NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot flush block %d of %s to stable store",
+ blocknum, &(reln->rd_rel->relname.data[0]));
return (status);
}
@@ -325,8 +325,8 @@ smgrnblocks(int16 which, Relation reln)
int nblocks;
if ((nblocks = (*(smgrsw[which].smgr_nblocks))(reln)) < 0)
- elog(WARN, "cannot count blocks for %.*s",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]));
+ elog(WARN, "cannot count blocks for %s",
+ &(reln->rd_rel->relname.data[0]));
return (nblocks);
}
@@ -347,8 +347,8 @@ smgrtruncate(int16 which, Relation reln, int nblocks)
if (smgrsw[which].smgr_truncate)
{
if ((newblks = (*(smgrsw[which].smgr_truncate))(reln, nblocks)) < 0)
- elog(WARN, "cannot truncate %.*s to %d blocks",
- NAMEDATALEN, &(reln->rd_rel->relname.data[0]), nblocks);
+ elog(WARN, "cannot truncate %s to %d blocks",
+ &(reln->rd_rel->relname.data[0]), nblocks);
}
return (newblks);
diff --git a/src/backend/tcop/aclchk.c b/src/backend/tcop/aclchk.c
index a32e6834c1b..f0518903dc2 100644
--- a/src/backend/tcop/aclchk.c
+++ b/src/backend/tcop/aclchk.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/tcop/Attic/aclchk.c,v 1.11 1997/08/12 22:54:17 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/tcop/Attic/aclchk.c,v 1.12 1997/08/18 20:53:29 momjian Exp $
*
* NOTES
* See acl.h.
@@ -356,8 +356,8 @@ pg_aclcheck(char *relname, char *usename, AclMode mode)
htp = SearchSysCacheTuple(USENAME, PointerGetDatum(usename),
0,0,0);
if (!HeapTupleIsValid(htp))
- elog(WARN, "pg_aclcheck: user \"%-.*s\" not found",
- NAMEDATALEN, usename);
+ elog(WARN, "pg_aclcheck: user \"%s\" not found",
+ usename);
id = (AclId) ((Form_pg_user) GETSTRUCT(htp))->usesysid;
/* for the 'pg_database' relation, check the usecreatedb
@@ -379,8 +379,8 @@ pg_aclcheck(char *relname, char *usename, AclMode mode)
if (((mode & ACL_WR) || (mode & ACL_AP)) &&
IsSystemRelationName(relname) &&
!((Form_pg_user) GETSTRUCT(htp))->usecatupd) {
- elog(DEBUG, "pg_aclcheck: catalog update to \"%-.*s\": permission denied",
- NAMEDATALEN, relname);
+ elog(DEBUG, "pg_aclcheck: catalog update to \"%s\": permission denied",
+ relname);
return ACLCHECK_NO_PRIV;
}
@@ -389,8 +389,8 @@ pg_aclcheck(char *relname, char *usename, AclMode mode)
*/
if (((Form_pg_user) GETSTRUCT(htp))->usesuper) {
#ifdef ACLDEBUG_TRACE
- elog(DEBUG, "pg_aclcheck: \"%-.*s\" is superuser",
- NAMEDATALEN, usename);
+ elog(DEBUG, "pg_aclcheck: \"%s\" is superuser",
+ usename);
#endif
return ACLCHECK_OK;
}
@@ -399,8 +399,8 @@ pg_aclcheck(char *relname, char *usename, AclMode mode)
htp = SearchSysCacheTuple(RELNAME, PointerGetDatum(relname),
0,0,0);
if (!HeapTupleIsValid(htp)) {
- elog(WARN, "pg_aclcheck: class \"%-.*s\" not found",
- NAMEDATALEN, relname);
+ elog(WARN, "pg_aclcheck: class \"%s\" not found",
+ relname);
/* an elog(WARN) kills us, so no need to return anything. */
}
if (!heap_attisnull(htp, Anum_pg_class_relacl)) {
@@ -472,8 +472,8 @@ pg_ownercheck(char *usename,
htp = SearchSysCacheTuple(USENAME, PointerGetDatum(usename),
0,0,0);
if (!HeapTupleIsValid(htp))
- elog(WARN, "pg_ownercheck: user \"%-.*s\" not found",
- NAMEDATALEN, usename);
+ elog(WARN, "pg_ownercheck: user \"%s\" not found",
+ usename);
user_id = (AclId) ((Form_pg_user) GETSTRUCT(htp))->usesysid;
/*
@@ -481,8 +481,8 @@ pg_ownercheck(char *usename,
*/
if (((Form_pg_user) GETSTRUCT(htp))->usesuper) {
#ifdef ACLDEBUG_TRACE
- elog(DEBUG, "pg_ownercheck: user \"%-.*s\" is superuser",
- NAMEDATALEN, usename);
+ elog(DEBUG, "pg_ownercheck: user \"%s\" is superuser",
+ usename);
#endif
return(1);
}
@@ -498,20 +498,20 @@ pg_ownercheck(char *usename,
break;
case PRONAME:
if (!HeapTupleIsValid(htp))
- elog(WARN, "pg_ownercheck: function \"%-.*s\" not found",
- NAMEDATALEN, value);
+ elog(WARN, "pg_ownercheck: function \"%s\" not found",
+ value);
owner_id = ((Form_pg_proc) GETSTRUCT(htp))->proowner;
break;
case RELNAME:
if (!HeapTupleIsValid(htp))
- elog(WARN, "pg_ownercheck: class \"%-.*s\" not found",
- NAMEDATALEN, value);
+ elog(WARN, "pg_ownercheck: class \"%s\" not found",
+ value);
owner_id = ((Form_pg_class) GETSTRUCT(htp))->relowner;
break;
case TYPNAME:
if (!HeapTupleIsValid(htp))
- elog(WARN, "pg_ownercheck: type \"%-.*s\" not found",
- NAMEDATALEN, value);
+ elog(WARN, "pg_ownercheck: type \"%s\" not found",
+ value);
owner_id = ((TypeTupleForm) GETSTRUCT(htp))->typowner;
break;
default:
@@ -535,8 +535,8 @@ pg_func_ownercheck(char *usename,
htp = SearchSysCacheTuple(USENAME, PointerGetDatum(usename),
0,0,0);
if (!HeapTupleIsValid(htp))
- elog(WARN, "pg_func_ownercheck: user \"%-.*s\" not found",
- NAMEDATALEN, usename);
+ elog(WARN, "pg_func_ownercheck: user \"%s\" not found",
+ usename);
user_id = (AclId) ((Form_pg_user) GETSTRUCT(htp))->usesysid;
/*
@@ -544,8 +544,8 @@ pg_func_ownercheck(char *usename,
*/
if (((Form_pg_user) GETSTRUCT(htp))->usesuper) {
#ifdef ACLDEBUG_TRACE
- elog(DEBUG, "pg_ownercheck: user \"%-.*s\" is superuser",
- NAMEDATALEN, usename);
+ elog(DEBUG, "pg_ownercheck: user \"%s\" is superuser",
+ usename);
#endif
return(1);
}
@@ -574,8 +574,8 @@ pg_aggr_ownercheck(char *usename,
htp = SearchSysCacheTuple(USENAME, PointerGetDatum(usename),
0,0,0);
if (!HeapTupleIsValid(htp))
- elog(WARN, "pg_aggr_ownercheck: user \"%-.*s\" not found",
- NAMEDATALEN, usename);
+ elog(WARN, "pg_aggr_ownercheck: user \"%s\" not found",
+ usename);
user_id = (AclId) ((Form_pg_user) GETSTRUCT(htp))->usesysid;
/*
@@ -583,8 +583,8 @@ pg_aggr_ownercheck(char *usename,
*/
if (((Form_pg_user) GETSTRUCT(htp))->usesuper) {
#ifdef ACLDEBUG_TRACE
- elog(DEBUG, "pg_aggr_ownercheck: user \"%-.*s\" is superuser",
- NAMEDATALEN, usename);
+ elog(DEBUG, "pg_aggr_ownercheck: user \"%s\" is superuser",
+ usename);
#endif
return(1);
}
diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c
index f0721423560..d8b79965a78 100644
--- a/src/backend/tcop/utility.c
+++ b/src/backend/tcop/utility.c
@@ -9,7 +9,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/tcop/utility.c,v 1.19 1997/08/18 02:14:52 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/tcop/utility.c,v 1.20 1997/08/18 20:53:35 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -171,8 +171,8 @@ ProcessUtility(Node *parsetree,
foreach (arg, args) {
relname = strVal(lfirst(arg));
if (IsSystemRelationName(relname))
- elog(WARN, "class \"%-.*s\" is a system catalog",
- NAMEDATALEN, relname);
+ elog(WARN, "class \"%s\" is a system catalog",
+ relname);
rel = heap_openr (relname);
if ( RelationIsValid (rel) )
{
@@ -188,8 +188,8 @@ ProcessUtility(Node *parsetree,
}
#ifndef NO_SECURITY
if (!pg_ownercheck(userName, relname, RELNAME))
- elog(WARN, "you do not own class \"%-.*s\"",
- NAMEDATALEN, relname);
+ elog(WARN, "you do not own class \"%s\"",
+ relname);
#endif
}
foreach (arg, args) {
@@ -318,8 +318,8 @@ ProcessUtility(Node *parsetree,
foreach (i, stmt->relNames) {
relname = strVal(lfirst(i));
if (!pg_ownercheck(userName, relname, RELNAME))
- elog(WARN, "you do not own class \"%-.*s\"",
- NAMEDATALEN, relname);
+ elog(WARN, "you do not own class \"%s\"",
+ relname);
}
#endif
foreach (i, stmt->relNames) {
diff --git a/src/backend/utils/adt/name.c b/src/backend/utils/adt/name.c
index 43e4c53e841..6c48aa8bcd4 100644
--- a/src/backend/utils/adt/name.c
+++ b/src/backend/utils/adt/name.c
@@ -11,7 +11,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/utils/adt/name.c,v 1.4 1997/08/12 20:16:01 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/utils/adt/name.c,v 1.5 1997/08/18 20:53:41 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -29,7 +29,8 @@
* namein - converts "..." to internal representation
*
* Note:
- * Currently if strlen(s) < NAMEDATALEN, the extra chars are nulls
+ * [Old] Currently if strlen(s) < NAMEDATALEN, the extra chars are nulls
+ * Now, always NULL terminated
*/
NameData *namein(char *s)
{
diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c
index 0282a60d806..d7bb7e79faf 100644
--- a/src/backend/utils/cache/relcache.c
+++ b/src/backend/utils/cache/relcache.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.12 1997/08/12 22:54:41 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.13 1997/08/18 20:53:48 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -284,7 +284,7 @@ BuildDescInfoError(RelationBuildDescInfo buildinfo)
sprintf(errBuf, "(relation id %d)", buildinfo.i.info_id);
break;
case INFO_RELNAME:
- sprintf(errBuf, "(relation name %.*s)", NAMEDATALEN, buildinfo.i.info_name);
+ sprintf(errBuf, "(relation name %s)", buildinfo.i.info_name);
break;
}
@@ -1282,8 +1282,8 @@ RelationForgetRelation (Oid rid)
prev = curr;
}
if ( curr == NIL )
- elog (FATAL, "Local relation %.*s not found in list",
- NAMEDATALEN, (RelationGetRelationName(relation))->data);
+ elog (FATAL, "Local relation %s not found in list",
+ (RelationGetRelationName(relation))->data);
if ( prev == NIL )
newlyCreatedRelns = lnext (newlyCreatedRelns);
else