aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/backend/utils/error/elog.c78
-rw-r--r--src/backend/utils/misc/guc.c88
-rw-r--r--src/include/utils/elog.h5
3 files changed, 103 insertions, 68 deletions
diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c
index 07b2d007395..d24242e8409 100644
--- a/src/backend/utils/error/elog.c
+++ b/src/backend/utils/error/elog.c
@@ -42,7 +42,7 @@
*
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/utils/error/elog.c,v 1.163 2005/10/14 16:41:02 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/error/elog.c,v 1.164 2005/10/14 20:53:56 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@@ -80,8 +80,9 @@ char *Log_line_prefix = NULL; /* format for extra log line info */
int Log_destination = LOG_DESTINATION_STDERR;
#ifdef HAVE_SYSLOG
-char *Syslog_facility; /* openlog() parameters */
-char *Syslog_ident;
+static bool openlog_done = false;
+static char *syslog_ident = NULL;
+static int syslog_facility = LOG_LOCAL0;
static void write_syslog(int level, const char *line);
#endif
@@ -1148,6 +1149,39 @@ DebugFileOpen(void)
#ifdef HAVE_SYSLOG
+/*
+ * Set or update the parameters for syslog logging
+ */
+void
+set_syslog_parameters(const char *ident, int facility)
+{
+ /*
+ * guc.c is likely to call us repeatedly with same parameters, so
+ * don't thrash the syslog connection unnecessarily. Also, we do not
+ * re-open the connection until needed, since this routine will get called
+ * whether or not Log_destination actually mentions syslog.
+ *
+ * Note that we make our own copy of the ident string rather than relying
+ * on guc.c's. This may be overly paranoid, but it ensures that we cannot
+ * accidentally free a string that syslog is still using.
+ */
+ if (syslog_ident == NULL || strcmp(syslog_ident, ident) != 0 ||
+ syslog_facility != facility)
+ {
+ if (openlog_done)
+ {
+ closelog();
+ openlog_done = false;
+ }
+ if (syslog_ident)
+ free(syslog_ident);
+ syslog_ident = strdup(ident);
+ /* if the strdup fails, we will cope in write_syslog() */
+ syslog_facility = facility;
+ }
+}
+
+
#ifndef PG_SYSLOG_LIMIT
#define PG_SYSLOG_LIMIT 128
#endif
@@ -1158,46 +1192,16 @@ DebugFileOpen(void)
static void
write_syslog(int level, const char *line)
{
- static bool openlog_done = false;
static unsigned long seq = 0;
int len;
+ /* Open syslog connection if not done yet */
if (!openlog_done)
{
- int syslog_fac;
- char *syslog_ident;
-
- if (pg_strcasecmp(Syslog_facility, "LOCAL0") == 0)
- syslog_fac = LOG_LOCAL0;
- else if (pg_strcasecmp(Syslog_facility, "LOCAL1") == 0)
- syslog_fac = LOG_LOCAL1;
- else if (pg_strcasecmp(Syslog_facility, "LOCAL2") == 0)
- syslog_fac = LOG_LOCAL2;
- else if (pg_strcasecmp(Syslog_facility, "LOCAL3") == 0)
- syslog_fac = LOG_LOCAL3;
- else if (pg_strcasecmp(Syslog_facility, "LOCAL4") == 0)
- syslog_fac = LOG_LOCAL4;
- else if (pg_strcasecmp(Syslog_facility, "LOCAL5") == 0)
- syslog_fac = LOG_LOCAL5;
- else if (pg_strcasecmp(Syslog_facility, "LOCAL6") == 0)
- syslog_fac = LOG_LOCAL6;
- else if (pg_strcasecmp(Syslog_facility, "LOCAL7") == 0)
- syslog_fac = LOG_LOCAL7;
- else
- syslog_fac = LOG_LOCAL0;
- /*
- * openlog() usually just stores the passed char pointer as-is,
- * so we must give it a string that will be unchanged for the life of
- * the process. The Syslog_ident GUC variable does not meet this
- * requirement, so strdup() it. This isn't a memory leak because
- * this code is executed at most once per process.
- */
- syslog_ident = strdup(Syslog_ident);
- if (syslog_ident == NULL) /* out of memory already!? */
- syslog_ident = "postgres";
-
- openlog(syslog_ident, LOG_PID | LOG_NDELAY | LOG_NOWAIT, syslog_fac);
+ openlog(syslog_ident ? syslog_ident : "postgres",
+ LOG_PID | LOG_NDELAY | LOG_NOWAIT,
+ syslog_facility);
openlog_done = true;
}
diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
index 549393ce551..1ba1ac31d3f 100644
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -10,7 +10,7 @@
* Written by Peter Eisentraut <peter_e@gmx.net>.
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.291 2005/10/08 20:08:19 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.292 2005/10/14 20:53:56 tgl Exp $
*
*--------------------------------------------------------------------
*/
@@ -21,6 +21,9 @@
#include <limits.h>
#include <unistd.h>
#include <sys/stat.h>
+#ifdef HAVE_SYSLOG
+#include <syslog.h>
+#endif
#include "utils/guc.h"
#include "utils/guc_tables.h"
@@ -100,10 +103,11 @@ static const char *assign_log_destination(const char *value,
bool doit, GucSource source);
#ifdef HAVE_SYSLOG
-extern char *Syslog_facility;
-extern char *Syslog_ident;
+static int syslog_facility = LOG_LOCAL0;
-static const char *assign_facility(const char *facility,
+static const char *assign_syslog_facility(const char *facility,
+ bool doit, GucSource source);
+static const char *assign_syslog_ident(const char *ident,
bool doit, GucSource source);
#endif
@@ -192,6 +196,8 @@ static char *log_error_verbosity_str;
static char *log_statement_str;
static char *log_min_error_statement_str;
static char *log_destination_string;
+static char *syslog_facility_str;
+static char *syslog_ident_str;
static bool phony_autocommit;
static bool session_auth_is_superuser;
static double phony_random_seed;
@@ -1964,22 +1970,22 @@ static struct config_string ConfigureNamesString[] =
#ifdef HAVE_SYSLOG
{
- {"syslog_facility", PGC_POSTMASTER, LOGGING_WHERE,
+ {"syslog_facility", PGC_SIGHUP, LOGGING_WHERE,
gettext_noop("Sets the syslog \"facility\" to be used when syslog enabled."),
gettext_noop("Valid values are LOCAL0, LOCAL1, LOCAL2, LOCAL3, "
"LOCAL4, LOCAL5, LOCAL6, LOCAL7.")
},
- &Syslog_facility,
- "LOCAL0", assign_facility, NULL
+ &syslog_facility_str,
+ "LOCAL0", assign_syslog_facility, NULL
},
{
- {"syslog_ident", PGC_POSTMASTER, LOGGING_WHERE,
- gettext_noop("Sets the program name used to identify PostgreSQL messages "
- "in syslog."),
+ {"syslog_ident", PGC_SIGHUP, LOGGING_WHERE,
+ gettext_noop("Sets the program name used to identify PostgreSQL "
+ "messages in syslog."),
NULL
},
- &Syslog_ident,
- "postgres", NULL, NULL
+ &syslog_ident_str,
+ "postgres", assign_syslog_ident, NULL
},
#endif
@@ -5552,27 +5558,49 @@ assign_log_destination(const char *value, bool doit, GucSource source)
#ifdef HAVE_SYSLOG
static const char *
-assign_facility(const char *facility, bool doit, GucSource source)
+assign_syslog_facility(const char *facility, bool doit, GucSource source)
{
+ int syslog_fac;
+
if (pg_strcasecmp(facility, "LOCAL0") == 0)
- return facility;
- if (pg_strcasecmp(facility, "LOCAL1") == 0)
- return facility;
- if (pg_strcasecmp(facility, "LOCAL2") == 0)
- return facility;
- if (pg_strcasecmp(facility, "LOCAL3") == 0)
- return facility;
- if (pg_strcasecmp(facility, "LOCAL4") == 0)
- return facility;
- if (pg_strcasecmp(facility, "LOCAL5") == 0)
- return facility;
- if (pg_strcasecmp(facility, "LOCAL6") == 0)
- return facility;
- if (pg_strcasecmp(facility, "LOCAL7") == 0)
- return facility;
- return NULL;
+ syslog_fac = LOG_LOCAL0;
+ else if (pg_strcasecmp(facility, "LOCAL1") == 0)
+ syslog_fac = LOG_LOCAL1;
+ else if (pg_strcasecmp(facility, "LOCAL2") == 0)
+ syslog_fac = LOG_LOCAL2;
+ else if (pg_strcasecmp(facility, "LOCAL3") == 0)
+ syslog_fac = LOG_LOCAL3;
+ else if (pg_strcasecmp(facility, "LOCAL4") == 0)
+ syslog_fac = LOG_LOCAL4;
+ else if (pg_strcasecmp(facility, "LOCAL5") == 0)
+ syslog_fac = LOG_LOCAL5;
+ else if (pg_strcasecmp(facility, "LOCAL6") == 0)
+ syslog_fac = LOG_LOCAL6;
+ else if (pg_strcasecmp(facility, "LOCAL7") == 0)
+ syslog_fac = LOG_LOCAL7;
+ else
+ return NULL; /* reject */
+
+ if (doit)
+ {
+ syslog_facility = syslog_fac;
+ set_syslog_parameters(syslog_ident_str ? syslog_ident_str : "postgres",
+ syslog_facility);
+ }
+
+ return facility;
}
-#endif
+
+static const char *
+assign_syslog_ident(const char *ident, bool doit, GucSource source)
+{
+ if (doit)
+ set_syslog_parameters(ident, syslog_facility);
+
+ return ident;
+}
+
+#endif /* HAVE_SYSLOG */
static const char *
diff --git a/src/include/utils/elog.h b/src/include/utils/elog.h
index f226f772911..264dfcc1429 100644
--- a/src/include/utils/elog.h
+++ b/src/include/utils/elog.h
@@ -7,7 +7,7 @@
* Portions Copyright (c) 1996-2005, PostgreSQL Global Development Group
* Portions Copyright (c) 1994, Regents of the University of California
*
- * $PostgreSQL: pgsql/src/include/utils/elog.h,v 1.79 2005/06/10 16:23:10 neilc Exp $
+ * $PostgreSQL: pgsql/src/include/utils/elog.h,v 1.80 2005/10/14 20:53:56 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@@ -283,6 +283,9 @@ extern int Log_destination;
/* Other exported functions */
extern void DebugFileOpen(void);
extern char *unpack_sql_state(int sql_state);
+#ifdef HAVE_SYSLOG
+extern void set_syslog_parameters(const char *ident, int facility);
+#endif
/*
* Write errors to stderr (or by equal means when stderr is
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-23 05:21:55 +0000