Fix 'smatch' warnings.
authorBen Kibbey <bjk@luxsci.net>
Sat, 6 Apr 2013 18:37:08 +0000 (6 14:37 -0400)
committerBen Kibbey <bjk@luxsci.net>
Sat, 6 Apr 2013 18:37:08 +0000 (6 14:37 -0400)
src/modules/login.h
src/modules/mail.c
src/ui.c
src/ui.h

index 62fd20b..10135df 100644 (file)
@@ -75,7 +75,7 @@ typedef struct utmp UTMP;
 #ifdef HAVE_PROCFS
 #ifdef HAVE_DIRENT_H
 #include <dirent.h>
-DIR *procdir;
+static DIR *procdir;
 #endif
 #endif
 
index 8c8dc6c..8a9d530 100644 (file)
@@ -153,7 +153,7 @@ static char *mail_aliases(const char *user, const int multi)
        if ((fd = open(ALIAS_FILE, O_RDONLY)) == -1)
            return "!";
 
-       if ((aliasbuf = mmap(0, st.st_size, PROT_READ, MAP_PRIVATE, fd,
+       if ((aliasbuf = mmap(NULL, st.st_size, PROT_READ, MAP_PRIVATE, fd,
                             0)) == MAP_FAILED) {
            warn("%s", "mmap()");
            return "!";
index 2807fd8..87a0201 100644 (file)
--- a/src/ui.c
+++ b/src/ui.c
@@ -73,7 +73,7 @@ char *stamp(time_t epoch, const char *format)
     return buf;
 }
 
-/* 
+/*
  * This may be used in modules to add a string to the buffer (ui_module_exec()).
  */
 void add_string(char ***buf, const char *str)
index 64a7a71..3f6a949 100644 (file)
--- a/src/ui.h
+++ b/src/ui.h
@@ -75,7 +75,7 @@ typedef int (module_options) (int argc, char **argv);
 typedef int (module_exec) (char ***, const struct passwd *, const int,
                           const int, char *);
 
-struct module_s {
+static struct module_s {
     char name[PATH_MAX];       /* The filename of the module. */
     void *m;           /* The module handle. */
     unsigned flags;    /* dup, chained, chainable, and output. */