aboutsummaryrefslogtreecommitdiffstats
path: root/src/minilogd.c
diff options
context:
space:
mode:
authorBill Nottingham <notting@redhat.com>2004-11-16 06:19:13 +0000
committerBill Nottingham <notting@redhat.com>2004-11-16 06:19:13 +0000
commitddef4757714ccc9d566b0fba4494528c32bcaa9d (patch)
tree5b290e86c415cd0d5d6731a77626e7b940eb8d9c /src/minilogd.c
parent7a97a0567f5a3d489e9383a99ee2416f400acaa1 (diff)
downloadinitscripts-ddef4757714ccc9d566b0fba4494528c32bcaa9d.tar
initscripts-ddef4757714ccc9d566b0fba4494528c32bcaa9d.tar.gz
initscripts-ddef4757714ccc9d566b0fba4494528c32bcaa9d.tar.bz2
initscripts-ddef4757714ccc9d566b0fba4494528c32bcaa9d.tar.xz
initscripts-ddef4757714ccc9d566b0fba4494528c32bcaa9d.zip
fix various minilogd bogosities (uninitialized variable, handling of wrong-protocol connections, don't check atime) (#106338)
Diffstat (limited to 'src/minilogd.c')
-rw-r--r--src/minilogd.c10
1 files changed, 7 insertions, 3 deletions
diff --git a/src/minilogd.c b/src/minilogd.c
index b7cbc8c0..85430978 100644
--- a/src/minilogd.c
+++ b/src/minilogd.c
@@ -77,7 +77,8 @@ void cleanup(int exitcode) {
void runDaemon(int sock) {
struct sockaddr_un addr;
- int x,len,addrlen,done=0;
+ int x,len,done=0;
+ int addrlen = sizeof(struct sockaddr_un);
char *message;
struct stat s1,s2;
struct pollfd pfds;
@@ -109,10 +110,13 @@ void runDaemon(int sock) {
}
if ( (x>0) && pfds.revents & (POLLIN | POLLPRI)) {
message = calloc(8192,sizeof(char));
+ addrlen = sizeof(struct sockaddr_un);
recvsock = accept(sock,(struct sockaddr *) &addr, &addrlen);
+ if (recvsock == -1)
+ continue;
alarm(2);
signal(SIGALRM, alarm_handler);
- len = read(recvsock,message,8192);
+ len = recv(recvsock,message,8192,0);
alarm(0);
close(recvsock);
if (len>0) {
@@ -135,7 +139,7 @@ void runDaemon(int sock) {
/* Check to see if syslogd's yanked our socket out from under us */
if ( (stat(_PATH_LOG,&s2)!=0) ||
(s1.st_ino != s2.st_ino ) || (s1.st_ctime != s2.st_ctime) ||
- (s1.st_mtime != s2.st_mtime) || (s1.st_atime != s2.st_atime) ) {
+ (s1.st_mtime != s2.st_mtime) ) {
done = 1;
we_own_log = 0;
}