dmesg: use SYSLOG_ACTION_* macros rather than magic constatnts

Signed-off-by: Karel Zak <kzak@redhat.com>

diff --git a/sys-utils/dmesg.c b/sys-utils/dmesg.c
index 3d550998a9a7a885f9a0ff3e967a21dfd2e5befe..7734cd0857f1b1b7e414d247d3d011cbf35c885f 100644 (file)
--- a/sys-utils/dmesg.c
+++ b/sys-utils/dmesg.c
@@ -11,24 +11,6 @@
  *
  */
 
-/*
- * Commands to sys_syslog:
- *
- *      0 -- Close the log.  Currently a NOP.
- *      1 -- Open the log. Currently a NOP.
- *      2 -- Read from the log.
- *      3 -- Read all messages remaining in the ring buffer.
- *      4 -- Read and clear all messages remaining in the ring buffer
- *      5 -- Clear ring buffer.
- *      6 -- Disable printk's to console
- *      7 -- Enable printk's to console
- *      8 -- Set level of messages printed to console
- *      9 -- Return number of unread characters in the log buffer
- *           [supported since 2.4.10]
- *
- * Only function 3 is allowed to non-root processes.
- */
-
 #include <linux/unistd.h>
 #include <stdio.h>
 #include <getopt.h>
@@ -41,6 +23,29 @@
 #include "strutils.h"
 #include "xalloc.h"
 
+/* Close the log.  Currently a NOP. */
+#define SYSLOG_ACTION_CLOSE          0
+/* Open the log. Currently a NOP. */
+#define SYSLOG_ACTION_OPEN           1
+/* Read from the log. */
+#define SYSLOG_ACTION_READ           2
+/* Read all messages remaining in the ring buffer. (allowed for non-root) */
+#define SYSLOG_ACTION_READ_ALL       3
+/* Read and clear all messages remaining in the ring buffer */
+#define SYSLOG_ACTION_READ_CLEAR     4
+/* Clear ring buffer. */
+#define SYSLOG_ACTION_CLEAR          5
+/* Disable printk's to console */
+#define SYSLOG_ACTION_CONSOLE_OFF    6
+/* Enable printk's to console */
+#define SYSLOG_ACTION_CONSOLE_ON     7
+/* Set level of messages printed to console */
+#define SYSLOG_ACTION_CONSOLE_LEVEL  8
+/* Return number of unread characters in the log buffer */
+#define SYSLOG_ACTION_SIZE_UNREAD    9
+/* Return size of the log buffer */
+#define SYSLOG_ACTION_SIZE_BUFFER   10
+
 static void __attribute__((__noreturn__)) usage(FILE *out)
 {
        fprintf(out, _(
@@ -69,7 +74,7 @@ int main(int argc, char *argv[])
        int  c;
        int  level = 0;
        int  lastc;
-       int  cmd = 3;           /* Read all messages in the ring buffer */
+       int  cmd = SYSLOG_ACTION_READ_ALL;
        int  raw = 0;
 
        static const struct option longopts[] = {
@@ -89,10 +94,10 @@ int main(int argc, char *argv[])
        while ((c = getopt_long(argc, argv, "chrn:s:V", longopts, NULL)) != -1) {
                switch (c) {
                case 'c':
-                       cmd = 4;        /* Read and clear all messages */
+                       cmd = SYSLOG_ACTION_READ_CLEAR;
                        break;
                case 'n':
-                       cmd = 8;        /* Set level of messages */
+                       cmd = SYSLOG_ACTION_CONSOLE_LEVEL;
                        level = strtol_or_err(optarg, _("failed to parse level"));
                        break;
                case 'r':
@@ -121,7 +126,7 @@ int main(int argc, char *argv[])
        if (argc > 1)
                usage(stderr);
 
-       if (cmd == 8) {
+       if (cmd == SYSLOG_ACTION_CONSOLE_LEVEL) {
                n = klogctl(cmd, NULL, level);
                if (n < 0)
                        err(EXIT_FAILURE, _("klogctl failed"));
@@ -130,7 +135,7 @@ int main(int argc, char *argv[])
        }
 
        if (!bufsize) {
-               n = klogctl(10, NULL, 0);       /* read ringbuffer size */
+               n = klogctl(SYSLOG_ACTION_SIZE_BUFFER, NULL, 0);
                if (n > 0)
                        bufsize = n;
        }
@@ -143,14 +148,14 @@ int main(int argc, char *argv[])
                sz = 16392;
                while (1) {
                        buf = xmalloc(sz * sizeof(char));
-                       n = klogctl(3, buf, sz);        /* read only */
+                       n = klogctl(SYSLOG_ACTION_READ_ALL, buf, sz);   /* read only */
                        if (n != sz || sz > (1 << 28))
                                break;
                        free(buf);
                        sz *= 4;
                }
 
-               if (n > 0 && cmd == 4)
+               if (n > 0 && cmd == SYSLOG_ACTION_READ_CLEAR)
                        n = klogctl(cmd, buf, sz);      /* read and clear */
        }