summaryrefslogtreecommitdiff
path: root/sysklogd
diff options
context:
space:
mode:
authorBernhard Reutner-Fischer2008-01-27 12:50:12 +0000
committerBernhard Reutner-Fischer2008-01-27 12:50:12 +0000
commitf470196a4144dd586f8b80efeef4aca39cf11514 (patch)
tree82fb31007e47f4eb2a31e8bb7bb9e6229a40dc36 /sysklogd
parent61160dd0a155364d30b7009b69e2a15afa651a2c (diff)
downloadbusybox-f470196a4144dd586f8b80efeef4aca39cf11514.zip
busybox-f470196a4144dd586f8b80efeef4aca39cf11514.tar.gz
- fix conflicting decls of syslog related facilitynames and prioritynames tables
Diffstat (limited to 'sysklogd')
-rw-r--r--sysklogd/klogd.c2
-rw-r--r--sysklogd/logger.c42
-rw-r--r--sysklogd/syslogd.c10
3 files changed, 28 insertions, 26 deletions
diff --git a/sysklogd/klogd.c b/sysklogd/klogd.c
index 72f3b55..d65b6f9 100644
--- a/sysklogd/klogd.c
+++ b/sysklogd/klogd.c
@@ -18,7 +18,7 @@
*/
#include "libbb.h"
-#include <sys/syslog.h>
+#include <syslog.h>
#include <sys/klog.h>
static void klogd_signal(int sig ATTRIBUTE_UNUSED)
diff --git a/sysklogd/logger.c b/sysklogd/logger.c
index 970b05d..0907501 100644
--- a/sysklogd/logger.c
+++ b/sysklogd/logger.c
@@ -8,28 +8,32 @@
*/
#include "libbb.h"
-
-#if !defined CONFIG_SYSLOGD
-
-/* SYSLOG_NAMES defined to pull prioritynames[] and facilitynames[]
- * from syslog.h. Grrrr - glibc puts those in _rwdata_! :( */
+#ifndef CONFIG_SYSLOGD
#define SYSLOG_NAMES
-#define SYSLOG_NAMES_CONST /* uclibc is saner :) */
-#include <sys/syslog.h>
-
+#define SYSLOG_NAMES_CONST
+#include <syslog.h>
#else
-#include <sys/syslog.h>
-# ifndef __dietlibc__
- /* We have to do this since the header file defines static
- * structures. Argh.... bad libc, bad, bad...
- */
- typedef struct _code {
- char *c_name;
- int c_val;
- } CODE;
- extern CODE prioritynames[];
- extern CODE facilitynames[];
+/* brokenness alert. Everybody except dietlibc get's this wrong by neither
+ * providing a typedef nor an extern for facilitynames and prioritynames
+ * in syslog.h.
+ */
+# include <syslog.h>
+# ifndef __dietlibc__
+/* We have to do this since the header file does neither provide a sane type
+ * for this structure nor extern definitions. Argh.... bad libc, bad, bad...
+ */
+typedef struct _code {
+ char *c_name; /* FIXME: this should be const char *const c_name ! */
+ int c_val;
+} CODE;
+# ifdef __UCLIBC__
+extern const CODE prioritynames[];
+extern const CODE facilitynames[];
+# else
+extern CODE prioritynames[];
+extern CODE facilitynames[];
# endif
+# endif
#endif
/* Decode a symbolic name to a numeric value
diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c
index 345bf0e..de8aa04 100644
--- a/sysklogd/syslogd.c
+++ b/sysklogd/syslogd.c
@@ -14,14 +14,12 @@
*/
#include "libbb.h"
+#define SYSLOG_NAMES
+#define SYSLOG_NAMES_CONST
+#include <syslog.h>
+
#include <paths.h>
#include <sys/un.h>
-
-/* SYSLOG_NAMES defined to pull prioritynames[] and facilitynames[]
- * from syslog.h. Grrrr - glibc puts those in _rwdata_! :( */
-#define SYSLOG_NAMES
-#define SYSLOG_NAMES_CONST /* uclibc is saner :) */
-#include <sys/syslog.h>
#include <sys/uio.h>
#if ENABLE_FEATURE_REMOTE_LOG