From 871d93c4620953a4ef19fc9df4865b0e351cd455 Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Tue, 17 Sep 2002 20:06:29 +0000 Subject: [PATCH] Vodz' last_patch57: Ok. Patch with reduce size ready. Notes: syslogd imposible close logsocket, the real programm write logmessages (hmm, cycle loop?) (Excuse me - this have small tab-formats rewroting also) text data bss dec hex filename 2913 332 12388 15633 3d11 syslogd.o 3109 332 9252 12693 3195 syslogd_old.o --- sysklogd/syslogd.c | 197 +++++++++++++++++++++------------------------ 1 file changed, 93 insertions(+), 104 deletions(-) diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c index f0a7f1a84..1b5a209aa 100644 --- a/sysklogd/syslogd.c +++ b/sysklogd/syslogd.c @@ -55,7 +55,7 @@ #define __LOG_FILE "/var/log/messages" /* Path to the unix socket */ -static char lfile[BUFSIZ]; +static char lfile[MAXPATHLEN]; static char *logFilePath = __LOG_FILE; @@ -63,7 +63,7 @@ static char *logFilePath = __LOG_FILE; static int MarkInterval = 20 * 60; /* localhost's name */ -static char LocalHostName[32]; +static char LocalHostName[64]; #ifdef CONFIG_FEATURE_REMOTE_LOG #include @@ -78,9 +78,12 @@ static int doRemoteLog = FALSE; static int local_logging = FALSE; #endif + +#define MAXLINE 1024 /* maximum line length */ + + /* circular buffer variables/structures */ #ifdef CONFIG_FEATURE_IPC_SYSLOG - #if __GNU_LIBRARY__ < 5 #error Sorry. Looks like you are using libc5. #error libc5 shm support isnt good enough. @@ -96,19 +99,19 @@ static const long KEY_ID = 0x414e4547; /*"GENA"*/ // Semaphore operation structures static struct shbuf_ds { - int size; // size of data written - int head; // start of message list - int tail; // end of message list - char data[1]; // data/messages -} *buf = NULL; // shared memory pointer + int size; // size of data written + int head; // start of message list + int tail; // end of message list + char data[1]; // data/messages +} *buf = NULL; // shared memory pointer static struct sembuf SMwup[1] = {{1, -1, IPC_NOWAIT}}; // set SMwup static struct sembuf SMwdn[3] = {{0, 0}, {1, 0}, {1, +1}}; // set SMwdn -static int shmid = -1; // ipc shared memory id -static int s_semid = -1; // ipc semaphore id -int data_size = 16000; // data size -int shm_size = 16000 + sizeof(*buf); // our buffer size +static int shmid = -1; // ipc shared memory id +static int s_semid = -1; // ipc semaphore id +int data_size = 16000; // data size +int shm_size = 16000 + sizeof(*buf); // our buffer size static int circular_logging = FALSE; /* @@ -129,6 +132,7 @@ static inline void sem_down(int semid) perror_msg_and_die("semop[SMwdn]"); } + void ipcsyslog_cleanup(void){ printf("Exiting Syslogd!\n"); if (shmid != -1) @@ -143,11 +147,11 @@ void ipcsyslog_cleanup(void){ void ipcsyslog_init(void){ if (buf == NULL){ if ((shmid = shmget(KEY_ID, shm_size, IPC_CREAT | 1023)) == -1) - perror_msg_and_die("shmget"); + perror_msg_and_die("shmget"); if ((buf = shmat(shmid, NULL, 0)) == NULL) - perror_msg_and_die("shmat"); + perror_msg_and_die("shmat"); buf->size=data_size; @@ -155,11 +159,11 @@ void ipcsyslog_init(void){ // we'll trust the OS to set initial semval to 0 (let's hope) if ((s_semid = semget(KEY_ID, 2, IPC_CREAT | IPC_EXCL | 1023)) == -1){ - if (errno == EEXIST){ + if (errno == EEXIST){ if ((s_semid = semget(KEY_ID, 2, 0)) == -1) perror_msg_and_die("semget"); }else - perror_msg_and_die("semget"); + perror_msg_and_die("semget"); } }else{ printf("Buffer already allocated just grab the semaphore?"); @@ -184,16 +188,16 @@ void circ_message(const char *msg){ * insert the new message. (Note: if the message being added is >1 message then * we will need to "remove" >1 old message from the buffer). The way this is done * is the following: - * When we reach the end of the buffer we set a mark and start from the beginning. - * Now what about the beginning and end of the buffer? Well we have the "head" - * index/pointer which is the starting point for the messages and we have "tail" - * index/pointer which is the ending point for the messages. When we "display" the - * messages we start from the beginning and continue until we reach "tail". If we - * reach end of buffer, then we just start from the beginning (offset 0). "head" and - * "tail" are actually offsets from the beginning of the buffer. + * When we reach the end of the buffer we set a mark and start from the beginning. + * Now what about the beginning and end of the buffer? Well we have the "head" + * index/pointer which is the starting point for the messages and we have "tail" + * index/pointer which is the ending point for the messages. When we "display" the + * messages we start from the beginning and continue until we reach "tail". If we + * reach end of buffer, then we just start from the beginning (offset 0). "head" and + * "tail" are actually offsets from the beginning of the buffer. * * Note: This algorithm uses Linux IPC mechanism w/ shared memory and semaphores to provide - * a threasafe way of handling shared memory operations. + * a threasafe way of handling shared memory operations. */ if ( (buf->tail + l) < buf->size ){ /* before we append the message we need to check the HEAD so that we won't @@ -207,19 +211,19 @@ void circ_message(const char *msg){ * to worry about overflows here! */ int k= buf->tail + l - buf->head; /* we need to know how many bytes - we are overwriting to make + we are overwriting to make enough room */ char *c=memchr(buf->data+buf->head + k,'\0',buf->size - (buf->head + k)); if (c != NULL) {/* do a sanity check just in case! */ - buf->head = c - buf->data + 1; /* we need to convert pointer to + buf->head = c - buf->data + 1; /* we need to convert pointer to offset + skip the '\0' since we need to point to the beginning of the next message */ /* Note: HEAD is only used to "retrieve" messages, it's not used when writing messages into our buffer */ }else{ /* show an error message to know we messed up? */ - printf("Weird! Can't find the terminator token??? \n"); - buf->head=0; + printf("Weird! Can't find the terminator token??? \n"); + buf->head=0; } } } /* in other cases no overflows have been done yet, so we don't care! */ @@ -231,22 +235,22 @@ void circ_message(const char *msg){ /* we need to break up the message and "circle" it around */ char *c; int k=buf->tail + l - buf->size; /* count # of bytes we don't fit */ - + /* We need to move HEAD! This is always the case since we are going * to "circle" the message. */ c=memchr(buf->data + k ,'\0', buf->size - k); - + if (c != NULL) /* if we don't have '\0'??? weird!!! */{ /* move head pointer*/ - buf->head=c-buf->data+1; - - /* now write the first part of the message */ + buf->head=c-buf->data+1; + + /* now write the first part of the message */ strncpy(buf->data + buf->tail, msg, l - k - 1); - + /* ALWAYS terminate end of buffer w/ '\0' */ - buf->data[buf->size-1]='\0'; - + buf->data[buf->size-1]='\0'; + /* now write out the rest of the string to the beginning of the buffer */ strcpy(buf->data, &msg[l-k-1]); @@ -256,11 +260,12 @@ void circ_message(const char *msg){ printf("Weird! Can't find the terminator token from the beginning??? \n"); buf->head = buf->tail = 0; /* reset buffer, since it's probably corrupted */ } - + } sem_up(s_semid); } -#endif +#endif /* CONFIG_FEATURE_IPC_SYSLOG */ + /* Note: There is also a function called "message()" in init.c */ /* Print a message to the log file. */ static void message (char *fmt, ...) __attribute__ ((format (printf, 1, 2))); @@ -275,10 +280,10 @@ static void message (char *fmt, ...) fl.l_len = 1; #ifdef CONFIG_FEATURE_IPC_SYSLOG - if ((circular_logging) && (buf != NULL)){ + if ((circular_logging == TRUE) && (buf != NULL)){ char b[1024]; va_start (arguments, fmt); - vsprintf (b, fmt, arguments); + vsnprintf (b, sizeof(b)-1, fmt, arguments); va_end (arguments); circ_message(b); @@ -354,20 +359,19 @@ static const int IOV_COUNT = 2; memset(&res, 0, sizeof(res)); snprintf(res, sizeof(res), "<%d>", pri); v->iov_base = res ; - v->iov_len = strlen(res); + v->iov_len = strlen(res); v++; v->iov_base = msg; - v->iov_len = strlen(msg); - + v->iov_len = strlen(msg); writev_retry: if ( -1 == writev(remotefd,iov, IOV_COUNT)){ if (errno == EINTR) goto writev_retry; - error_msg_and_die("syslogd: cannot write to remote file handle on" + error_msg_and_die("cannot write to remote file handle on" "%s:%d",RemoteHost,RemotePort); } } - if (local_logging) + if (local_logging == TRUE) #endif /* now spew out the message to wherever it is supposed to go */ message("%s %s %s %s\n", timestamp, LocalHostName, res, msg); @@ -396,7 +400,6 @@ static void domark(int sig) /* This must be a #define, since when DODEBUG and BUFFERS_GO_IN_BSS are * enabled, we otherwise get a "storage size isn't constant error. */ -#define BUFSIZE 1023 static int serveConnection (char* tmpbuf, int n_read) { char *p = tmpbuf; @@ -404,16 +407,14 @@ static int serveConnection (char* tmpbuf, int n_read) while (p < tmpbuf + n_read) { int pri = (LOG_USER | LOG_NOTICE); - char line[ BUFSIZE + 1 ]; + char line[ MAXLINE + 1 ]; unsigned char c; - int gotpri = 0; char *q = line; - while (p && (c = *p) && q < &line[ sizeof (line) - 1 ]) { - if ((c == '<') && !gotpri && isdigit(p[1])) { + while ( (c = *p) && q < &line[ sizeof (line) - 1 ]) { + if (c == '<') { /* Parse the magic priority number. */ - gotpri = 1; pri = 0; while (isdigit (*(++p))) { pri = 10 * pri + (*p - '0'); @@ -441,7 +442,8 @@ static int serveConnection (char* tmpbuf, int n_read) #ifdef CONFIG_FEATURE_REMOTE_LOG -static void init_RemoteLog (void){ +static void init_RemoteLog (void) +{ struct sockaddr_in remoteaddr; struct hostent *hostinfo; @@ -452,7 +454,7 @@ static void init_RemoteLog (void){ remotefd = socket(AF_INET, SOCK_DGRAM, 0); if (remotefd < 0) { - error_msg_and_die("syslogd: cannot create socket"); + error_msg_and_die("cannot create socket"); } hostinfo = xgethostbyname(RemoteHost); @@ -461,26 +463,23 @@ static void init_RemoteLog (void){ remoteaddr.sin_addr = *(struct in_addr *) *hostinfo->h_addr_list; remoteaddr.sin_port = htons(RemotePort); - /* - Since we are using UDP sockets, connect just sets the default host and port + /* + Since we are using UDP sockets, connect just sets the default host and port for future operations */ if ( 0 != (connect(remotefd, (struct sockaddr *) &remoteaddr, len))){ - error_msg_and_die("syslogd: cannot connect to remote host %s:%d", RemoteHost, RemotePort); + error_msg_and_die("cannot connect to remote host %s:%d", RemoteHost, RemotePort); } } #endif -#define MAXLINE 1024 /* maximum line length */ - static void doSyslogd (void) __attribute__ ((noreturn)); static void doSyslogd (void) { struct sockaddr_un sunx; socklen_t addrLength; - int sock_fd; fd_set fds; @@ -507,71 +506,61 @@ static void doSyslogd (void) perror_msg_and_die ("Couldn't get file descriptor for socket " _PATH_LOG); addrLength = sizeof (sunx.sun_family) + strlen (sunx.sun_path); - if (bind (sock_fd, (struct sockaddr *) &sunx, addrLength) < 0) + if (bind(sock_fd, (struct sockaddr *) &sunx, addrLength) < 0) perror_msg_and_die ("Could not connect to socket " _PATH_LOG); if (chmod (lfile, 0666) < 0) perror_msg_and_die ("Could not set permission on " _PATH_LOG); - FD_ZERO (&fds); - FD_SET (sock_fd, &fds); #ifdef CONFIG_FEATURE_IPC_SYSLOG - if (circular_logging ){ + if (circular_logging == TRUE ){ ipcsyslog_init(); } #endif - #ifdef CONFIG_FEATURE_REMOTE_LOG - if (doRemoteLog){ - init_RemoteLog(); - } - #endif +#ifdef CONFIG_FEATURE_REMOTE_LOG + if (doRemoteLog == TRUE){ + init_RemoteLog(); + } +#endif - logMessage (LOG_SYSLOG | LOG_INFO, "syslogd started: " BB_BANNER); + logMessage (LOG_SYSLOG | LOG_INFO, "syslogd started: " CONFIG_BANNER); for (;;) { - fd_set readfds; - int n_ready; - int fd; - - memcpy (&readfds, &fds, sizeof (fds)); + FD_ZERO (&fds); + FD_SET (sock_fd, &fds); - if ((n_ready = select (FD_SETSIZE, &readfds, NULL, NULL, NULL)) < 0) { - if (errno == EINTR) continue; /* alarm may have happened. */ + if (select (sock_fd+1, &fds, NULL, NULL, NULL) < 0) { + if (errno == EINTR) { + /* alarm may have happened. */ + continue; + } perror_msg_and_die ("select error"); } - for (fd = 0; (n_ready > 0) && (fd < FD_SETSIZE); fd++) { - if (FD_ISSET (fd, &readfds)) { - - --n_ready; - - if (fd == sock_fd) { - int i; - RESERVE_CONFIG_BUFFER(tmpbuf, BUFSIZE + 1); - - memset(tmpbuf, '\0', BUFSIZE+1); - if ( (i = recv(fd, tmpbuf, MAXLINE - 2, 0)) > 0) { - if ( serveConnection(tmpbuf, i) <= 0 ) { - close (fd); - FD_CLR(fd, &fds); - } - } else { - perror_msg_and_die ("UNIX socket error"); - } - RELEASE_CONFIG_BUFFER (tmpbuf); - } /* fd == sock_fd */ - }/* FD_ISSET() */ - }/* for */ + if (FD_ISSET (sock_fd, &fds)) { + int i; + RESERVE_CONFIG_BUFFER(tmpbuf, BUFSIZ + 1); + + memset(tmpbuf, '\0', BUFSIZ+1); + if ( (i = recv(sock_fd, tmpbuf, BUFSIZ, 0)) > 0) { + serveConnection(tmpbuf, i); + } else { + perror_msg_and_die ("UNIX socket error"); + } + RELEASE_CONFIG_BUFFER (tmpbuf); + }/* FD_ISSET() */ } /* for main loop */ } extern int syslogd_main(int argc, char **argv) { int opt; +#if ! defined(__uClinux__) int doFork = TRUE; +#endif char *p; @@ -581,9 +570,11 @@ extern int syslogd_main(int argc, char **argv) case 'm': MarkInterval = atoi(optarg) * 60; break; +#if ! defined(__uClinux__) case 'n': doFork = FALSE; break; +#endif case 'O': logFilePath = xstrdup(optarg); break; @@ -612,7 +603,7 @@ extern int syslogd_main(int argc, char **argv) #ifdef CONFIG_FEATURE_REMOTE_LOG /* If they have not specified remote logging, then log locally */ - if (! doRemoteLog) + if (doRemoteLog == FALSE) local_logging = TRUE; #endif @@ -625,14 +616,12 @@ extern int syslogd_main(int argc, char **argv) umask(0); - if (doFork) { -#if !defined(__UCLIBC__) || defined(__UCLIBC_HAS_MMU__) +#if ! defined(__uClinux__) + if (doFork == TRUE) { if (daemon(0, 1) < 0) perror_msg_and_die("daemon"); -#else - error_msg_and_die("daemon not supported"); -#endif } +#endif doSyslogd(); return EXIT_SUCCESS; -- 2.25.1