- fix conflicting decls of syslog related facilitynames and prioritynames tables

diff --git a/init/init.c b/init/init.c
index f7eb8f34ba3510d1b463e8acc1b1d18a97795e07..513ff340cdc39f1a8e634d32214b0e6b313f358c 100644 (file)
--- a/init/init.c
+++ b/init/init.c
@@ -10,9 +10,9 @@
  */
 
 #include "libbb.h"
+#include <syslog.h>
 #include <paths.h>
 #include <sys/reboot.h>
-#include <sys/syslog.h>
 
 #define INIT_BUFFS_SIZE 256
 #define CONSOLE_NAME_SIZE 32

diff --git a/libbb/info_msg.c b/libbb/info_msg.c
index b0ce64338e7def89c9bde905ed3743567016259a..3231bc8cff3b79e0a5b92aab502b12294402fa17 100644 (file)
--- a/libbb/info_msg.c
+++ b/libbb/info_msg.c
@@ -7,8 +7,8 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#include <syslog.h>
 #include "libbb.h"
+#include <syslog.h>
 
 void bb_info_msg(const char *s, ...)
 {

diff --git a/libbb/verror_msg.c b/libbb/verror_msg.c
index dbd8323e43e132f5d179f55b170d809909d40be8..ca44030b20bc7d964138e2eafc290ae4c7a7d709 100644 (file)
--- a/libbb/verror_msg.c
+++ b/libbb/verror_msg.c
@@ -7,8 +7,8 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#include <syslog.h>
 #include "libbb.h"
+#include <syslog.h>
 
 smallint logmode = LOGMODE_STDIO;
 const char *msg_eol = "\n";

diff --git a/loginutils/login.c b/loginutils/login.c
index 7e904465474cab7c7e8f6aa92638bf3f392e7caf..55cbfa7cdb40ecf5c6cc2292c15ebc63ac66ab73 100644 (file)
--- a/loginutils/login.c
+++ b/loginutils/login.c
@@ -4,9 +4,9 @@
  */
 
 #include "libbb.h"
+#include <syslog.h>
 #include <utmp.h>
 #include <sys/resource.h>
-#include <syslog.h>
 
 #if ENABLE_SELINUX
 #include <selinux/selinux.h>  /* for is_selinux_enabled()  */

diff --git a/loginutils/passwd.c b/loginutils/passwd.c
index 0842b71bfb34031ac1f0b64c6e1db007ec216188..0df084e5a0ed3b913900844f7cb7a972568d2250 100644 (file)
--- a/loginutils/passwd.c
+++ b/loginutils/passwd.c
@@ -6,7 +6,6 @@
 #include "libbb.h"
 #include <syslog.h>
 
-
 static void nuke_str(char *str)
 {
        if (str) memset(str, 0, strlen(str));

diff --git a/loginutils/sulogin.c b/loginutils/sulogin.c
index af457ef1e52b1ea0c0b095a2166fbac1a87b9c09..faa93eaeac594803f39aff2557838f151369360d 100644 (file)
--- a/loginutils/sulogin.c
+++ b/loginutils/sulogin.c
@@ -5,9 +5,8 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#include <syslog.h>
-
 #include "libbb.h"
+#include <syslog.h>
 
 //static void catchalarm(int ATTRIBUTE_UNUSED junk)
 //{

diff --git a/miscutils/crond.c b/miscutils/crond.c
index fd20e6a573fac81da1250edcb1d5b262c598d92d..0df143ae7556f76b7ec10c166dcd521fac898cdb 100644 (file)
--- a/miscutils/crond.c
+++ b/miscutils/crond.c
@@ -11,8 +11,8 @@
  * Licensed under the GPL v2 or later, see the file LICENSE in this tarball.
  */
 
-#include <sys/syslog.h>
 #include "libbb.h"
+#include <syslog.h>
 
 #ifndef CRONTABS
 #define CRONTABS        "/var/spool/cron/crontabs"

diff --git a/miscutils/devfsd.c b/miscutils/devfsd.c
index 9990142c215effa1f9e50e7e4b78a45204b507d0..52a65bc9843d7949493fcf7a533efb724f3b9c24 100644 (file)
--- a/miscutils/devfsd.c
+++ b/miscutils/devfsd.c
@@ -53,17 +53,12 @@
     The postal address is:
       Richard Gooch, c/o ATNF, P. O. Box 76, Epping, N.S.W., 2121, Australia.
 */
-
-//#include <sys/wait.h>
-//#include <sys/ioctl.h>
-//#include <sys/socket.h>
-#include <sys/un.h>
-#include <dirent.h>
-#include <syslog.h>
-#include <sys/sysmacros.h>
 #include "libbb.h"
 #include "xregex.h"
+#include <syslog.h>
 
+#include <sys/un.h>
+#include <sys/sysmacros.h>
 
 /* Various defines taken from linux/major.h */
 #define IDE0_MAJOR     3

diff --git a/networking/dnsd.c b/networking/dnsd.c
index 8f01d5c06f2b0738cc28fb422deb4d2810777821..19720d6bcd519567fa6529847206b9f8f6c81b41 100644 (file)
--- a/networking/dnsd.c
+++ b/networking/dnsd.c
@@ -17,8 +17,8 @@
  * the first porting of oao' scdns to busybox also.
  */
 
-#include <syslog.h>
 #include "libbb.h"
+#include <syslog.h>
 
 //#define DEBUG 1
 #define DEBUG 0

diff --git a/networking/isrv_identd.c b/networking/isrv_identd.c
index a93de124418defbe1c49716ed2aaf917e5cd79a8..dbdecba24b6b27278cb4158c3927371e6182b29b 100644 (file)
--- a/networking/isrv_identd.c
+++ b/networking/isrv_identd.c
@@ -7,8 +7,8 @@
  * Licensed under GPL version 2, see file LICENSE in this tarball for details.
  */
 
-#include <syslog.h>
 #include "libbb.h"
+#include <syslog.h>
 #include "isrv.h"
 
 enum { TIMEOUT = 20 };

diff --git a/networking/libiproute/iprule.c b/networking/libiproute/iprule.c
index 6b22061a7ae0e84463f9fc56020322e6ab232a98..3f9007ee953dc7e7307894ac764f5509eb2bac19 100644 (file)
--- a/networking/libiproute/iprule.c
+++ b/networking/libiproute/iprule.c
@@ -16,7 +16,6 @@
  * initially integrated into busybox by Bernhard Fischer
  */
 
-#include <syslog.h>
 #include <netinet/in.h>
 #include <netinet/ip.h>
 #include <arpa/inet.h>

diff --git a/networking/telnetd.c b/networking/telnetd.c
index 0201d2636775038736b36c1138c464e388061387..962e5cc7b7398c47f45dc6eef679fcce8fbfce1d 100644 (file)
--- a/networking/telnetd.c
+++ b/networking/telnetd.c
@@ -24,14 +24,13 @@
 #define DEBUG 0
 
 #include "libbb.h"
+#include <syslog.h>
 
 #if DEBUG
 #define TELCMDS
 #define TELOPTS
 #endif
 #include <arpa/telnet.h>
-#include <sys/syslog.h>
-
 
 /* Structure that describes a session */
 struct tsession {

diff --git a/networking/zcip.c b/networking/zcip.c
index ef9aa2186324aaebdf5751f8c09a65865447b90e..6e1c0d148f9411498ad0af48f3824d753771b925 100644 (file)
--- a/networking/zcip.c
+++ b/networking/zcip.c
@@ -23,9 +23,6 @@
 // - avoid silent script failures, especially under load...
 // - link status monitoring (restart on link-up; stop on link-down)
 
-#include <syslog.h>
-#include <poll.h>
-#include <sys/wait.h>
 #include <netinet/ether.h>
 #include <net/ethernet.h>
 #include <net/if.h>
@@ -34,6 +31,7 @@
 #include <linux/sockios.h>
 
 #include "libbb.h"
+#include <syslog.h>
 
 /* We don't need more than 32 bits of the counter */
 #define MONOTONIC_US() ((unsigned)monotonic_us())

diff --git a/sysklogd/klogd.c b/sysklogd/klogd.c
index 72f3b559ca928c712d6b16cf61cd45eac438ec43..d65b6f9a018e82ec80473fb0495f487761059b25 100644 (file)
--- 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 970b05dc6c96ab8b973a1e13654e476c689468d0..090750173188810b52f8eb347528d1aab237a969 100644 (file)
--- 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 345bf0e87940ca8f1d3d7b7bb68076b5196b5b35..de8aa04a896faed0a2040ba1b2b0406a2a5a1095 100644 (file)
--- 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

diff --git a/util-linux/mount.c b/util-linux/mount.c
index abf5cd03f978d33de15b8741d0266389d1559157..a10d28d7c66256892b14f8be9974398e7b7dc73a 100644 (file)
--- a/util-linux/mount.c
+++ b/util-linux/mount.c
@@ -20,9 +20,9 @@
 
 #include <mntent.h>
 #include "libbb.h"
+#include <syslog.h>
 
 /* Needed for nfs support only... */
-#include <syslog.h>
 #include <sys/utsname.h>
 #undef TRUE
 #undef FALSE