summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRainer Gerhards <rgerhards@adiscon.com>2009-09-14 15:39:16 +0200
committerRainer Gerhards <rgerhards@adiscon.com>2009-09-14 15:39:16 +0200
commit92b4f6d610a3b4de9066d4f26a712cf6b1507250 (patch)
treef21bea8ba3492d5fefde4587960e35f8840fe61a
parent78e9c7c4d2c4ec09cce403066f09a5a7e08e994e (diff)
downloadrsyslog-92b4f6d610a3b4de9066d4f26a712cf6b1507250.tar.gz
rsyslog-92b4f6d610a3b4de9066d4f26a712cf6b1507250.tar.xz
rsyslog-92b4f6d610a3b4de9066d4f26a712cf6b1507250.zip
fixed some minor portability issues
-rw-r--r--ChangeLog1
-rw-r--r--configure.ac4
-rw-r--r--runtime/msg.c4
-rw-r--r--runtime/stream.c10
4 files changed, 14 insertions, 5 deletions
diff --git a/ChangeLog b/ChangeLog
index 1a8cb598..e29e96dc 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -16,6 +16,7 @@ Version 4.5.3 [v4-beta] (rgerhards), 2009-08-??
- bugfix: strings improperly reused, resulting in some message properties
be populated with strings from previous messages. This was caused by
an improper predicate check. [backported from v5]
+- fixed some minor portability issues
- bugfix: reverse lookup reduction logic in imudp do DNS queries too often
[imported from 4.4.2]
---------------------------------------------------------------------------
diff --git a/configure.ac b/configure.ac
index 19d738eb..9a263128 100644
--- a/configure.ac
+++ b/configure.ac
@@ -69,7 +69,7 @@ AC_SUBST(DL_LIBS)
AC_HEADER_RESOLV
AC_HEADER_STDC
AC_HEADER_SYS_WAIT
-AC_CHECK_HEADERS([arpa/inet.h libgen.h fcntl.h locale.h netdb.h netinet/in.h paths.h stddef.h stdlib.h string.h sys/file.h sys/ioctl.h sys/param.h sys/socket.h sys/time.h sys/stat.h syslog.h unistd.h utmp.h sys/epoll.h sys/prctl.h])
+AC_CHECK_HEADERS([arpa/inet.h libgen.h malloc.h fcntl.h locale.h netdb.h netinet/in.h paths.h stddef.h stdlib.h string.h sys/file.h sys/ioctl.h sys/param.h sys/socket.h sys/time.h sys/stat.h syslog.h unistd.h utmp.h sys/epoll.h sys/prctl.h])
# Checks for typedefs, structures, and compiler characteristics.
AC_C_CONST
@@ -104,7 +104,7 @@ AC_TYPE_SIGNAL
AC_FUNC_STAT
AC_FUNC_STRERROR_R
AC_FUNC_VPRINTF
-AC_CHECK_FUNCS([flock basename alarm clock_gettime gethostbyname gethostname gettimeofday localtime_r memset mkdir regcomp select setid socket strcasecmp strchr strdup strerror strndup strnlen strrchr strstr strtol strtoul uname ttyname_r epoll_wait getline malloc_trim prctl])
+AC_CHECK_FUNCS([flock basename alarm clock_gettime gethostbyname gethostname gettimeofday localtime_r memset mkdir regcomp select setid socket strcasecmp strchr strdup strerror strndup strnlen strrchr strstr strtol strtoul uname ttyname_r epoll_wait getline malloc_trim prctl fdatasync])
# Check for MAXHOSTNAMELEN
AC_MSG_CHECKING(for MAXHOSTNAMELEN)
diff --git a/runtime/msg.c b/runtime/msg.c
index 2a370618..208ea77a 100644
--- a/runtime/msg.c
+++ b/runtime/msg.c
@@ -35,7 +35,9 @@
#include <string.h>
#include <assert.h>
#include <ctype.h>
-#include <malloc.h>
+#if HAVE_MALLOC_H
+# include <malloc.h>
+#endif
#include "rsyslog.h"
#include "srUtils.h"
#include "stringbuf.h"
diff --git a/runtime/stream.c b/runtime/stream.c
index 605a9771..8098f778 100644
--- a/runtime/stream.c
+++ b/runtime/stream.c
@@ -961,6 +961,12 @@ finalize_it:
* is worth (read: data loss may occur where we otherwise might not
* have it). -- rgerhards, 2009-06-08
*/
+#undef SYNCCALL
+#if HAVE_FDATASYNC
+# define SYNCCALL(x) fdatasync(x)
+#else
+# define SYNCCALL(x) fsync(x)
+#endif
static rsRetVal
syncFile(strm_t *pThis)
{
@@ -971,7 +977,7 @@ syncFile(strm_t *pThis)
FINALIZE; /* TTYs can not be synced */
DBGPRINTF("syncing file %d\n", pThis->fd);
- ret = fdatasync(pThis->fd);
+ ret = SYNCCALL(pThis->fd);
if(ret != 0) {
char errStr[1024];
int err = errno;
@@ -987,7 +993,7 @@ syncFile(strm_t *pThis)
finalize_it:
RETiRet;
}
-
+#undef SYNCCALL
/* physically write to the output file. the provided data is ready for
* writing (e.g. zipped if we are requested to do that).