diff options
| author | Bill Nottingham <notting@redhat.com> | 2004-11-16 06:19:13 +0000 |
|---|---|---|
| committer | Bill Nottingham <notting@redhat.com> | 2004-11-16 06:19:13 +0000 |
| commit | ddef4757714ccc9d566b0fba4494528c32bcaa9d (patch) | |
| tree | 5b290e86c415cd0d5d6731a77626e7b940eb8d9c /src/minilogd.c | |
| parent | 7a97a0567f5a3d489e9383a99ee2416f400acaa1 (diff) | |
| download | initscripts-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.c | 10 |
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; } |