diff options
Diffstat (limited to 'src/backend')
| -rw-r--r-- | src/backend/catalog/heap.c | 5 | ||||
| -rw-r--r-- | src/backend/catalog/namespace.c | 4 | ||||
| -rw-r--r-- | src/backend/commands/cluster.c | 7 | ||||
| -rw-r--r-- | src/backend/commands/sequence.c | 26 | ||||
| -rw-r--r-- | src/backend/commands/tablecmds.c | 6 | ||||
| -rw-r--r-- | src/backend/executor/execMain.c | 4 | ||||
| -rw-r--r-- | src/backend/libpq/hba.c | 4 | ||||
| -rw-r--r-- | src/backend/postmaster/pgstat.c | 14 | ||||
| -rw-r--r-- | src/backend/utils/adt/nabstime.c | 22 | ||||
| -rw-r--r-- | src/backend/utils/adt/pgstatfuncs.c | 46 | ||||
| -rw-r--r-- | src/backend/utils/error/elog.c | 9 |
11 files changed, 105 insertions, 42 deletions
diff --git a/src/backend/catalog/heap.c b/src/backend/catalog/heap.c index 8d176c12cd..105026a50e 100644 --- a/src/backend/catalog/heap.c +++ b/src/backend/catalog/heap.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/catalog/heap.c,v 1.239 2003/01/08 22:06:20 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/catalog/heap.c,v 1.240 2003/03/20 03:34:55 momjian Exp $ * * * INTERFACE ROUTINES @@ -961,7 +961,8 @@ RemoveAttributeById(Oid relid, AttrNumber attnum) attStruct->attstattarget = 0; /* Change the column name to something that isn't likely to conflict */ - snprintf(newattname, NAMEDATALEN, "........pg.dropped.%d........", attnum); + snprintf(newattname, sizeof(newattname), + "........pg.dropped.%d........", attnum); namestrcpy(&(attStruct->attname), newattname); simple_heap_update(attr_rel, &tuple->t_self, tuple); diff --git a/src/backend/catalog/namespace.c b/src/backend/catalog/namespace.c index 367903f59f..90c26288f5 100644 --- a/src/backend/catalog/namespace.c +++ b/src/backend/catalog/namespace.c @@ -13,7 +13,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/catalog/namespace.c,v 1.48 2003/03/06 22:54:49 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/catalog/namespace.c,v 1.49 2003/03/20 03:34:55 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -1598,7 +1598,7 @@ InitTempTableNamespace(void) elog(ERROR, "%s: not authorized to create temp tables", DatabaseName); - snprintf(namespaceName, NAMEDATALEN, "pg_temp_%d", MyBackendId); + snprintf(namespaceName, sizeof(namespaceName), "pg_temp_%d", MyBackendId); namespaceId = GetSysCacheOid(NAMESPACENAME, CStringGetDatum(namespaceName), diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c index 30d251869b..0c1a7d3c70 100644 --- a/src/backend/commands/cluster.c +++ b/src/backend/commands/cluster.c @@ -11,7 +11,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/commands/cluster.c,v 1.106 2003/03/03 04:37:37 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/commands/cluster.c,v 1.107 2003/03/20 03:34:55 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -405,7 +405,7 @@ rebuild_relation(Relation OldHeap, Oid indexOid) * namespace from the old, or we will have problems with the TEMP * status of temp tables. */ - snprintf(NewHeapName, NAMEDATALEN, "pg_temp_%u", tableOid); + snprintf(NewHeapName, sizeof(NewHeapName), "pg_temp_%u", tableOid); OIDNewHeap = make_new_heap(tableOid, NewHeapName); /* @@ -625,7 +625,8 @@ rebuild_indexes(Oid OIDOldHeap, List *indexes) Relation pg_index; /* Create the new index under a temporary name */ - snprintf(newIndexName, NAMEDATALEN, "pg_temp_%u", attrs->indexOID); + snprintf(newIndexName, sizeof(newIndexName), + "pg_temp_%u", attrs->indexOID); /* * The new index will have primary and constraint status set to diff --git a/src/backend/commands/sequence.c b/src/backend/commands/sequence.c index c3ea6a14c0..bca6512cef 100644 --- a/src/backend/commands/sequence.c +++ b/src/backend/commands/sequence.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/commands/sequence.c,v 1.91 2003/02/13 05:25:24 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/commands/sequence.c,v 1.92 2003/03/20 03:34:55 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -406,7 +406,7 @@ nextval(PG_FUNCTION_ARGS) { char buf[100]; - snprintf(buf, 100, INT64_FORMAT, maxv); + snprintf(buf, sizeof(buf), INT64_FORMAT, maxv); elog(ERROR, "%s.nextval: reached MAXVALUE (%s)", sequence->relname, buf); } @@ -427,7 +427,7 @@ nextval(PG_FUNCTION_ARGS) { char buf[100]; - snprintf(buf, 100, INT64_FORMAT, minv); + snprintf(buf, sizeof(buf), INT64_FORMAT, minv); elog(ERROR, "%s.nextval: reached MINVALUE (%s)", sequence->relname, buf); } @@ -569,9 +569,9 @@ do_setval(RangeVar *sequence, int64 next, bool iscalled) bufm[100], bufx[100]; - snprintf(bufv, 100, INT64_FORMAT, next); - snprintf(bufm, 100, INT64_FORMAT, seq->min_value); - snprintf(bufx, 100, INT64_FORMAT, seq->max_value); + snprintf(bufv, sizeof(bufv), INT64_FORMAT, next); + snprintf(bufm, sizeof(bufm), INT64_FORMAT, seq->min_value); + snprintf(bufx, sizeof(bufx), INT64_FORMAT, seq->max_value); elog(ERROR, "%s.setval: value %s is out of bounds (%s,%s)", sequence->relname, bufv, bufm, bufx); } @@ -861,8 +861,8 @@ init_params(CreateSeqStmt *seq, Form_pg_sequence new) char bufm[100], bufx[100]; - snprintf(bufm, 100, INT64_FORMAT, new->min_value); - snprintf(bufx, 100, INT64_FORMAT, new->max_value); + snprintf(bufm, sizeof(bufm), INT64_FORMAT, new->min_value); + snprintf(bufx, sizeof(bufx), INT64_FORMAT, new->max_value); elog(ERROR, "DefineSequence: MINVALUE (%s) must be less than MAXVALUE (%s)", bufm, bufx); } @@ -882,8 +882,8 @@ init_params(CreateSeqStmt *seq, Form_pg_sequence new) char bufs[100], bufm[100]; - snprintf(bufs, 100, INT64_FORMAT, new->last_value); - snprintf(bufm, 100, INT64_FORMAT, new->min_value); + snprintf(bufs, sizeof(bufs), INT64_FORMAT, new->last_value); + snprintf(bufm, sizeof(bufm), INT64_FORMAT, new->min_value); elog(ERROR, "DefineSequence: START value (%s) can't be less than MINVALUE (%s)", bufs, bufm); } @@ -892,8 +892,8 @@ init_params(CreateSeqStmt *seq, Form_pg_sequence new) char bufs[100], bufm[100]; - snprintf(bufs, 100, INT64_FORMAT, new->last_value); - snprintf(bufm, 100, INT64_FORMAT, new->max_value); + snprintf(bufs, sizeof(bufs), INT64_FORMAT, new->last_value); + snprintf(bufm, sizeof(bufm), INT64_FORMAT, new->max_value); elog(ERROR, "DefineSequence: START value (%s) can't be greater than MAXVALUE (%s)", bufs, bufm); } @@ -904,7 +904,7 @@ init_params(CreateSeqStmt *seq, Form_pg_sequence new) { char buf[100]; - snprintf(buf, 100, INT64_FORMAT, new->cache_value); + snprintf(buf, sizeof(buf), INT64_FORMAT, new->cache_value); elog(ERROR, "DefineSequence: CACHE (%s) can't be <= 0", buf); } diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c index 13401ed107..f23c88f6fe 100644 --- a/src/backend/commands/tablecmds.c +++ b/src/backend/commands/tablecmds.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/commands/tablecmds.c,v 1.67 2003/02/13 05:19:59 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/commands/tablecmds.c,v 1.68 2003/03/20 03:34:55 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -3875,8 +3875,8 @@ AlterTableCreateToastTable(Oid relOid, bool silent) /* * Create the toast table and its index */ - snprintf(toast_relname, NAMEDATALEN, "pg_toast_%u", relOid); - snprintf(toast_idxname, NAMEDATALEN, "pg_toast_%u_index", relOid); + snprintf(toast_relname, sizeof(toast_relname), "pg_toast_%u", relOid); + snprintf(toast_idxname, sizeof(toast_idxname), "pg_toast_%u_index", relOid); /* this is pretty painful... need a tuple descriptor */ tupdesc = CreateTemplateTupleDesc(3, false); diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c index 638cc61bef..e9cbd734b3 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.202 2003/03/11 19:40:22 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.203 2003/03/20 03:34:55 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -603,7 +603,7 @@ InitPlan(QueryDesc *queryDesc) erm = (execRowMark *) palloc(sizeof(execRowMark)); erm->relation = relation; erm->rti = rti; - snprintf(erm->resname, 32, "ctid%u", rti); + snprintf(erm->resname, sizeof(erm->resname), "ctid%u", rti); estate->es_rowMark = lappend(estate->es_rowMark, erm); } } diff --git a/src/backend/libpq/hba.c b/src/backend/libpq/hba.c index 1cc0c98a57..721ccf621e 100644 --- a/src/backend/libpq/hba.c +++ b/src/backend/libpq/hba.c @@ -10,7 +10,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/libpq/hba.c,v 1.94 2003/03/15 16:18:25 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/libpq/hba.c,v 1.95 2003/03/20 03:34:55 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -1162,7 +1162,7 @@ ident_inet(const struct in_addr remote_ip_addr, char ident_query[80]; /* The query we send to the Ident server */ - snprintf(ident_query, 80, "%d,%d\n", + snprintf(ident_query, sizeof(ident_query), "%d,%d\n", ntohs(remote_port), ntohs(local_port)); /* loop in case send is interrupted */ do diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c index 17c11682cf..86d47db9ef 100644 --- a/src/backend/postmaster/pgstat.c +++ b/src/backend/postmaster/pgstat.c @@ -16,7 +16,7 @@ * * Copyright (c) 2001, PostgreSQL Global Development Group * - * $Header: /cvsroot/pgsql/src/backend/postmaster/pgstat.c,v 1.31 2002/10/24 23:19:13 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/postmaster/pgstat.c,v 1.32 2003/03/20 03:34:56 momjian Exp $ * ---------- */ #include "postgres.h" @@ -1757,6 +1757,8 @@ pgstat_add_backend(PgStat_MsgHdr *msg) beentry->databaseid = msg->m_databaseid; beentry->procpid = msg->m_procpid; beentry->userid = msg->m_userid; + beentry->activity_start_sec = 0; + beentry->activity_start_usec = 0; MemSet(beentry->activity, 0, PGSTAT_ACTIVITY_SIZE); /* @@ -2460,6 +2462,8 @@ pgstat_recv_beterm(PgStat_MsgBeterm *msg, int len) static void pgstat_recv_activity(PgStat_MsgActivity *msg, int len) { + PgStat_StatBeEntry *entry; + /* * Here we check explicitly for 0 return, since we don't want to * mangle the activity of an active backend by a delayed packed from a @@ -2468,8 +2472,12 @@ pgstat_recv_activity(PgStat_MsgActivity *msg, int len) if (pgstat_add_backend(&msg->m_hdr) != 0) return; - strncpy(pgStatBeTable[msg->m_hdr.m_backendid - 1].activity, - msg->m_what, PGSTAT_ACTIVITY_SIZE); + entry = &(pgStatBeTable[msg->m_hdr.m_backendid - 1]); + + strncpy(entry->activity, msg->m_what, PGSTAT_ACTIVITY_SIZE); + + entry->activity_start_sec = + GetCurrentAbsoluteTimeUsec(&entry->activity_start_usec); } diff --git a/src/backend/utils/adt/nabstime.c b/src/backend/utils/adt/nabstime.c index 408b1ee14d..8a75bdffbe 100644 --- a/src/backend/utils/adt/nabstime.c +++ b/src/backend/utils/adt/nabstime.c @@ -10,7 +10,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/utils/adt/nabstime.c,v 1.104 2003/02/22 05:57:45 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/utils/adt/nabstime.c,v 1.105 2003/03/20 03:34:56 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -82,10 +82,13 @@ static int istinterval(char *i_string, AbsoluteTime *i_end); -/* GetCurrentAbsoluteTime() - * Get the current system time. +/* + * GetCurrentAbsoluteTime() + * + * Get the current system time. Set timezone parameters if not specified + * elsewhere. Define HasCTZSet to allow clients to specify the default + * timezone. * - * Returns the number of seconds since epoch (January 1 1970 GMT). */ AbsoluteTime GetCurrentAbsoluteTime(void) @@ -127,9 +130,14 @@ GetCurrentDateTime(struct tm * tm) abstime2tm(GetCurrentTransactionStartTime(), &tz, tm, NULL); } -/* GetCurrentTimeUsec() - * Get the transaction start time ("now()") broken down as a struct tm, - * plus fractional-second and timezone info. +/* + * GetCurrentAbsoluteTimeUsec() + * + * Get the current system time. Set timezone parameters if not specified + * elsewhere. Define HasCTZSet to allow clients to specify the default + * timezone. + * + * Returns the number of seconds since epoch (January 1 1970 GMT) */ void GetCurrentTimeUsec(struct tm * tm, fsec_t *fsec, int *tzp) diff --git a/src/backend/utils/adt/pgstatfuncs.c b/src/backend/utils/adt/pgstatfuncs.c index 42b63b8739..455da1c2aa 100644 --- a/src/backend/utils/adt/pgstatfuncs.c +++ b/src/backend/utils/adt/pgstatfuncs.c @@ -25,6 +25,7 @@ extern Datum pg_stat_get_backend_pid(PG_FUNCTION_ARGS); extern Datum pg_stat_get_backend_dbid(PG_FUNCTION_ARGS); extern Datum pg_stat_get_backend_userid(PG_FUNCTION_ARGS); extern Datum pg_stat_get_backend_activity(PG_FUNCTION_ARGS); +extern Datum pg_stat_get_backend_activity_start(PG_FUNCTION_ARGS); extern Datum pg_stat_get_db_numbackends(PG_FUNCTION_ARGS); extern Datum pg_stat_get_db_xact_commit(PG_FUNCTION_ARGS); @@ -221,7 +222,6 @@ pg_backend_pid(PG_FUNCTION_ARGS) /* * Built-in function for resetting the counters - * */ Datum pg_stat_reset(PG_FUNCTION_ARGS) @@ -302,6 +302,50 @@ pg_stat_get_backend_activity(PG_FUNCTION_ARGS) Datum +pg_stat_get_backend_activity_start(PG_FUNCTION_ARGS) +{ + PgStat_StatBeEntry *beentry; + int32 beid; + AbsoluteTime sec; + int usec; + Timestamp result; + + beid = PG_GETARG_INT32(0); + + if (!superuser()) + PG_RETURN_NULL(); + + if ((beentry = pgstat_fetch_stat_beentry(beid)) == NULL) + PG_RETURN_NULL(); + + sec = beentry->activity_start_sec; + usec = beentry->activity_start_usec; + + /* + * No time recorded for start of current query -- this is the case + * if the user hasn't enabled query-level stats collection. + */ + if (sec == 0 && usec == 0) + PG_RETURN_NULL(); + + /* + * This method of converting "Unix time" (sec/usec since epoch) to a + * PostgreSQL timestamp is an ugly hack -- if you fix it, be sure to + * fix the similar hackery in timestamp.c + */ +#ifdef HAVE_INT64_TIMESTAMP + result = (((sec - ((date2j(2000, 1, 1) - date2j(1970, 1, 1)) * 86400)) + * INT64CONST(1000000)) + usec); +#else + result = (sec + (usec * 1.0e-6) - ((date2j(2000, 1, 1) - + date2j(1970, 1, 1)) * 86400)); +#endif + + PG_RETURN_TIMESTAMP(result); +} + + +Datum pg_stat_get_db_numbackends(PG_FUNCTION_ARGS) { PgStat_StatDBEntry *dbentry; diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c index 868db0907e..763024b577 100644 --- a/src/backend/utils/error/elog.c +++ b/src/backend/utils/error/elog.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/utils/error/elog.c,v 1.106 2003/01/07 22:23:17 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/utils/error/elog.c,v 1.107 2003/03/20 03:34:56 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -215,7 +215,8 @@ elog(int lev, const char *fmt,...) * Prints the failure line of the COPY. Wow, what a hack! bjm * Translator: Error message will be truncated at 31 characters. */ - snprintf(copylineno_buf, 32, gettext("copy: line %d, "), copy_lineno); + snprintf(copylineno_buf, sizeof(copylineno_buf), + gettext("copy: line %d, "), copy_lineno); space_needed += strlen(copylineno_buf); } @@ -787,8 +788,8 @@ useful_strerror(int errnum) * translator: This string will be truncated at 47 characters * expanded. */ - snprintf(errorstr_buf, 48, gettext("operating system error %d"), - errnum); + snprintf(errorstr_buf, sizeof(errorstr_buf), + gettext("operating system error %d"), errnum); str = errorstr_buf; } |
