Add errno.h

[oweals/busybox.git] / logger.c

diff --git a/logger.c b/logger.c
index 1c454e5af8594ed70bb9b892be482ad4cbd5f3d5..3d02979c86a119ad7b5544d2c48e05d1de8cc2b4 100644 (file)
--- a/logger.c
+++ b/logger.c
@@ -35,7 +35,7 @@
 
 #else
 /* We have to do this since the header file defines static
- * structues.  Argh.... bad libc, bad, bad...
+ * structures.  Argh.... bad libc, bad, bad...
  */
 #include <sys/syslog.h>
 typedef struct _code {
@@ -101,73 +101,53 @@ extern int logger_main(int argc, char **argv)
 {
        int pri = LOG_USER | LOG_NOTICE;
        int option = 0;
-       int fromStdinFlag = FALSE;
-       int stopLookingAtMeLikeThat = FALSE;
+       int c, i, len, opt;
        char *message=NULL, buf[1024], name[128];
 
-       /* Fill out the name string early (may be overwritten later */
+       /* Fill out the name string early (may be overwritten later) */
        my_getpwuid(name, geteuid());
 
        /* Parse any options */
-       while (--argc > 0 && **(++argv) == '-') {
-               if (*((*argv) + 1) == '\0') {
-                       fromStdinFlag = TRUE;
-               }
-               stopLookingAtMeLikeThat = FALSE;
-               while (*(++(*argv)) && stopLookingAtMeLikeThat == FALSE) {
-                       switch (**argv) {
+       while ((opt = getopt(argc, argv, "p:st:")) > 0) {
+               switch (opt) {
                        case 's':
                                option |= LOG_PERROR;
                                break;
                        case 'p':
-                               if (--argc == 0) {
-                                       usage(logger_usage);
-                               }
-                               pri = pencode(*(++argv));
-                               stopLookingAtMeLikeThat = TRUE;
+                               pri = pencode(optarg);
                                break;
                        case 't':
-                               if (--argc == 0) {
-                                       usage(logger_usage);
-                               }
-                               strncpy(name, *(++argv), sizeof(name));
-                               stopLookingAtMeLikeThat = TRUE;
+                               strncpy(name, optarg, sizeof(name));
                                break;
                        default:
                                usage(logger_usage);
-                       }
                }
        }
 
-       if (fromStdinFlag == TRUE) {
+       if (optind == argc) {
                /* read from stdin */
-               int c;
-               unsigned int i = 0;
-
+               i = 0;
                while ((c = getc(stdin)) != EOF && i < sizeof(buf)) {
                        buf[i++] = c;
                }
+               buf[i++] = '\0';
                message = buf;
        } else {
-               if (argc >= 1) {
-                       int len = 1; /* for the '\0' */
-                       for (; *argv != NULL; argv++) {
-                               len += strlen(*argv);
-                               len += 1;  /* for the space between the args */
-                               message = xrealloc(message, len);
-                               strcat(message, *argv);
-                               strcat(message, " ");
-                       }
-                       message[strlen(message)-1] = '\0';
-               } else {
-                       error_msg_and_die("No message\n");
+               len = 1; /* for the '\0' */
+               message=xcalloc(1, 1);
+               for (i = optind; i < argc; i++) {
+                       len += strlen(argv[i]);
+                       len += 1;  /* for the space between the args */
+                       message = xrealloc(message, len);
+                       strcat(message, argv[i]);
+                       strcat(message, " ");
                }
+               message[strlen(message)-1] = '\0';
        }
 
        openlog(name, option, (pri | LOG_FACMASK));
        syslog(pri, "%s", message);
        closelog();
-
        return EXIT_SUCCESS;
 }