]> granicus.if.org Git - linux-pam/blobdiff - modules/pam_limits/pam_limits.c
Fix grammar of messages printed via pam_prompt
[linux-pam] / modules / pam_limits / pam_limits.c
index 20aa794a6a388cf08acba3bf35547d90abfc1651..cac369999183ae928cc69301d7a72a7022205e1b 100644 (file)
@@ -14,7 +14,7 @@
  */
 
 #if !defined(linux) && !defined(__linux)
-#error THIS CODE IS KNOWN TO WORK ONLY ON LINUX !!!
+#warning THIS CODE IS KNOWN TO WORK ONLY ON LINUX !!!
 #endif
 
 #include "config.h"
 #include <errno.h>
 #include <syslog.h>
 #include <stdarg.h>
+#include <signal.h>
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <sys/resource.h>
 #include <limits.h>
-
+#include <glob.h>
 #include <utmp.h>
 #ifndef UT_USER  /* some systems have ut_name instead of ut_user */
 #define UT_USER ut_user
 
 #include <grp.h>
 #include <pwd.h>
+#include <locale.h>
+
+#ifdef HAVE_LIBAUDIT
+#include <libaudit.h>
+#endif
 
 /* Module defines */
 #define LINE_LENGTH 1024
 #define LIMITS_DEF_USER     0 /* limit was set by an user entry */
 #define LIMITS_DEF_GROUP    1 /* limit was set by a group entry */
 #define LIMITS_DEF_ALLGROUP 2 /* limit was set by a group entry */
-#define LIMITS_DEF_ALL      3 /* limit was set by an default entry */
-#define LIMITS_DEF_DEFAULT  4 /* limit was set by an default entry */
-#define LIMITS_DEF_NONE     5 /* this limit was not set yet */
+#define LIMITS_DEF_ALL      3 /* limit was set by an all entry */
+#define LIMITS_DEF_DEFAULT  4 /* limit was set by a default entry */
+#define LIMITS_DEF_KERNEL   5 /* limit was set from /proc/1/limits */
+#define LIMITS_DEF_NONE     6 /* this limit was not set yet */
+
+#define LIMIT_RANGE_ERR    -1 /* error in specified uid/gid range */
+#define LIMIT_RANGE_NONE    0 /* no range specified */
+#define LIMIT_RANGE_ONE     1 /* exact uid/gid specified (:max_uid)*/
+#define LIMIT_RANGE_MIN     2 /* only minimum uid/gid specified (min_uid:) */
+#define LIMIT_RANGE_MM      3 /* both min and max uid/gid specified (min_uid:max_uid) */
 
 static const char *limits_def_names[] = {
        "USER",
@@ -56,6 +69,7 @@ static const char *limits_def_names[] = {
        "ALLGROUP",
        "ALL",
        "DEFAULT",
+       "KERNEL",
        "NONE",
        NULL
 };
@@ -75,7 +89,7 @@ struct pam_limit_s {
                              specific user or to count all logins */
     int priority;       /* the priority to run user process with */
     struct user_limits_struct limits[RLIM_NLIMITS];
-    char conf_file[BUFSIZ];
+    const char *conf_file;
     int utmp_after_pam_call;
     char login_group[LINE_LENGTH];
 };
@@ -98,8 +112,14 @@ struct pam_limit_s {
 /* argument parsing */
 
 #define PAM_DEBUG_ARG       0x0001
-#define PAM_DO_SETREUID     0x0002
 #define PAM_UTMP_EARLY      0x0004
+#define PAM_NO_AUDIT        0x0008
+#define PAM_SET_ALL         0x0010
+
+/* Limits from globbed files. */
+#define LIMITS_CONF_GLOB LIMITS_FILE_DIR
+
+#define CONF_FILE (pl->conf_file != NULL)?pl->conf_file:LIMITS_FILE
 
 static int
 _pam_parse (const pam_handle_t *pamh, int argc, const char **argv,
@@ -115,20 +135,88 @@ _pam_parse (const pam_handle_t *pamh, int argc, const char **argv,
        if (!strcmp(*argv,"debug")) {
            ctrl |= PAM_DEBUG_ARG;
        } else if (!strncmp(*argv,"conf=",5)) {
-           strncpy(pl->conf_file,*argv+5,sizeof(pl->conf_file)-1);
-       } else if (!strncmp(*argv,"change_uid",10)) {
-           ctrl |= PAM_DO_SETREUID;
+           pl->conf_file = *argv+5;
        } else if (!strcmp(*argv,"utmp_early")) {
            ctrl |= PAM_UTMP_EARLY;
+       } else if (!strcmp(*argv,"noaudit")) {
+           ctrl |= PAM_NO_AUDIT;
+       } else if (!strcmp(*argv,"set_all")) {
+           ctrl |= PAM_SET_ALL;
        } else {
            pam_syslog(pamh, LOG_ERR, "unknown option: %s", *argv);
        }
     }
-    pl->conf_file[sizeof(pl->conf_file) - 1] = '\0';
 
     return ctrl;
 }
 
+static const char *
+rlimit2str (int i)
+{
+  switch (i) {
+  case RLIMIT_CPU:
+    return "cpu";
+    break;
+  case RLIMIT_FSIZE:
+    return "fsize";
+    break;
+  case RLIMIT_DATA:
+    return "data";
+    break;
+  case RLIMIT_STACK:
+    return "stack";
+    break;
+  case RLIMIT_CORE:
+    return "core";
+    break;
+  case RLIMIT_RSS:
+    return "rss";
+    break;
+  case RLIMIT_NPROC:
+    return "nproc";
+    break;
+  case RLIMIT_NOFILE:
+    return "nofile";
+    break;
+  case RLIMIT_MEMLOCK:
+    return "memlock";
+    break;
+#ifdef RLIMIT_AS
+  case RLIMIT_AS:
+    return "as";
+    break;
+#endif
+#ifdef RLIMIT_LOCKS
+  case RLIMIT_LOCKS:
+    return "locks";
+    break;
+#endif
+#ifdef RLIMIT_SIGPENDING
+  case RLIMIT_SIGPENDING:
+    return "sigpending";
+    break;
+#endif
+#ifdef RLIMIT_MSGQUEUE
+  case RLIMIT_MSGQUEUE:
+    return "msgqueue";
+    break;
+#endif
+#ifdef RLIMIT_NICE
+  case RLIMIT_NICE:
+    return "nice";
+    break;
+#endif
+#ifdef RLIMIT_RTPRIO
+  case RLIMIT_RTPRIO:
+    return "rtprio";
+    break;
+#endif
+  default:
+    return "UNKNOWN";
+    break;
+  }
+}
+
 
 #define LIMITED_OK 0 /* limit setting appeared to work */
 #define LIMIT_ERR  1 /* error setting a limit */
@@ -182,16 +270,27 @@ check_logins (pam_handle_t *pamh, const char *name, int limit, int ctrl,
             continue;
        }
         if (!pl->flag_numsyslogins) {
+           char user[sizeof(ut->UT_USER) + 1];
+           user[0] = '\0';
+           strncat(user, ut->UT_USER, sizeof(ut->UT_USER));
+
            if (((pl->login_limit_def == LIMITS_DEF_USER)
                 || (pl->login_limit_def == LIMITS_DEF_GROUP)
                 || (pl->login_limit_def == LIMITS_DEF_DEFAULT))
-               && strncmp(name, ut->UT_USER, sizeof(ut->UT_USER)) != 0) {
+               && strcmp(name, user) != 0) {
                 continue;
            }
            if ((pl->login_limit_def == LIMITS_DEF_ALLGROUP)
-               && !pam_modutil_user_in_group_nam_nam(pamh, ut->UT_USER, pl->login_group)) {
+               && !pam_modutil_user_in_group_nam_nam(pamh, user, pl->login_group)) {
                 continue;
            }
+           if (kill(ut->ut_pid, 0) == -1 && errno == ESRCH) {
+               /* process does not exist anymore */
+               pam_syslog(pamh, LOG_INFO,
+                          "Stale utmp entry (pid %d) for '%s' ignored",
+                          ut->ut_pid, user);
+               continue;
+           }
        }
        if (++count > limit) {
            break;
@@ -200,17 +299,150 @@ check_logins (pam_handle_t *pamh, const char *name, int limit, int ctrl,
     endutent();
     if (count > limit) {
        if (name) {
-           pam_syslog(pamh, LOG_WARNING,
+           pam_syslog(pamh, LOG_NOTICE,
                       "Too many logins (max %d) for %s", limit, name);
        } else {
-           pam_syslog(pamh, LOG_WARNING, "Too many system logins (max %d)", limit);
+           pam_syslog(pamh, LOG_NOTICE, "Too many system logins (max %d)", limit);
        }
         return LOGIN_ERR;
     }
     return 0;
 }
 
-static int init_limits(struct pam_limit_s *pl)
+static const char *lnames[RLIM_NLIMITS] = {
+        [RLIMIT_CPU] = "Max cpu time",
+        [RLIMIT_FSIZE] = "Max file size",
+        [RLIMIT_DATA] = "Max data size",
+        [RLIMIT_STACK] = "Max stack size",
+        [RLIMIT_CORE] = "Max core file size",
+        [RLIMIT_RSS] = "Max resident set",
+        [RLIMIT_NPROC] = "Max processes",
+        [RLIMIT_NOFILE] = "Max open files",
+        [RLIMIT_MEMLOCK] = "Max locked memory",
+#ifdef RLIMIT_AS
+        [RLIMIT_AS] = "Max address space",
+#endif
+#ifdef RLIMIT_LOCKS
+        [RLIMIT_LOCKS] = "Max file locks",
+#endif
+#ifdef RLIMIT_SIGPENDING
+        [RLIMIT_SIGPENDING] = "Max pending signals",
+#endif
+#ifdef RLIMIT_MSGQUEUE
+        [RLIMIT_MSGQUEUE] = "Max msgqueue size",
+#endif
+#ifdef RLIMIT_NICE
+        [RLIMIT_NICE] = "Max nice priority",
+#endif
+#ifdef RLIMIT_RTPRIO
+        [RLIMIT_RTPRIO] = "Max realtime priority",
+#endif
+#ifdef RLIMIT_RTTIME
+        [RLIMIT_RTTIME] = "Max realtime timeout",
+#endif
+};
+
+static int str2rlimit(char *name) {
+    int i;
+    if (!name || *name == '\0')
+        return -1;
+    for(i = 0; i < RLIM_NLIMITS; i++) {
+        if (strcmp(name, lnames[i]) == 0) return i;
+    }
+    return -1;
+}
+
+static rlim_t str2rlim_t(char *value) {
+    unsigned long long rlimit = 0;
+
+    if (!value) return (rlim_t)rlimit;
+    if (strcmp(value, "unlimited") == 0) {
+        return RLIM_INFINITY;
+    }
+    rlimit = strtoull(value, NULL, 10);
+    return (rlim_t)rlimit;
+}
+
+#define LIMITS_SKIP_WHITESPACE { \
+        /* step backwards over spaces */ \
+        pos--; \
+        while (pos && line[pos] == ' ') pos--; \
+        if (!pos) continue; \
+        line[pos+1] = '\0'; \
+}
+#define LIMITS_MARK_ITEM(item) { \
+        /* step backwards over non-spaces */ \
+        pos--; \
+        while (pos && line[pos] != ' ') pos--; \
+        if (!pos) continue; \
+        item = line + pos + 1; \
+}
+
+static void parse_kernel_limits(pam_handle_t *pamh, struct pam_limit_s *pl, int ctrl)
+{
+    int i, maxlen = 0;
+    FILE *limitsfile;
+    const char *proclimits = "/proc/1/limits";
+    char line[256];
+    char *units, *hard, *soft, *name;
+
+    if (!(limitsfile = fopen(proclimits, "r"))) {
+        pam_syslog(pamh, LOG_WARNING, "Could not read %s (%s), using PAM defaults", proclimits, strerror(errno));
+        return;
+    }
+
+    while (fgets(line, 256, limitsfile)) {
+        int pos = strlen(line);
+        if (pos < 2) continue;
+
+        /* drop trailing newline */
+        if (line[pos-1] == '\n') {
+            pos--;
+            line[pos] = '\0';
+        }
+
+        /* determine formatting boundry of limits report */
+        if (!maxlen && strncmp(line, "Limit", 5) == 0) {
+            maxlen = pos;
+            continue;
+        }
+
+        if (pos == maxlen) {
+            /* step backwards over "Units" name */
+            LIMITS_SKIP_WHITESPACE;
+            LIMITS_MARK_ITEM(units);
+        }
+        else {
+            units = "";
+        }
+
+        /* step backwards over "Hard Limit" value */
+        LIMITS_SKIP_WHITESPACE;
+        LIMITS_MARK_ITEM(hard);
+
+        /* step backwards over "Soft Limit" value */
+        LIMITS_SKIP_WHITESPACE;
+        LIMITS_MARK_ITEM(soft);
+
+        /* step backwards over name of limit */
+        LIMITS_SKIP_WHITESPACE;
+        name = line;
+
+        i = str2rlimit(name);
+        if (i < 0 || i >= RLIM_NLIMITS) {
+            if (ctrl & PAM_DEBUG_ARG)
+                pam_syslog(pamh, LOG_DEBUG, "Unknown kernel rlimit '%s' ignored", name);
+            continue;
+        }
+        pl->limits[i].limit.rlim_cur = str2rlim_t(soft);
+        pl->limits[i].limit.rlim_max = str2rlim_t(hard);
+        pl->limits[i].src_soft = LIMITS_DEF_KERNEL;
+        pl->limits[i].src_hard = LIMITS_DEF_KERNEL;
+    }
+    fclose(limitsfile);
+}
+
+static int init_limits(pam_handle_t *pamh, struct pam_limit_s *pl, int ctrl)
 {
     int i;
     int retval = PAM_SUCCESS;
@@ -231,6 +463,20 @@ static int init_limits(struct pam_limit_s *pl)
        }
     }
 
+#ifdef __linux__
+    if (ctrl & PAM_SET_ALL) {
+      parse_kernel_limits(pamh, pl, ctrl);
+
+      for(i = 0; i < RLIM_NLIMITS; i++) {
+       if (pl->limits[i].supported &&
+           (pl->limits[i].src_soft == LIMITS_DEF_NONE ||
+            pl->limits[i].src_hard == LIMITS_DEF_NONE)) {
+         pam_syslog(pamh, LOG_WARNING, "Did not find kernel RLIMIT for %s, using PAM default", rlimit2str(i));
+       }
+      }
+    }
+#endif
+
     errno = 0;
     pl->priority = getpriority (PRIO_PROCESS, 0);
     if (pl->priority == -1 && errno != 0)
@@ -276,8 +522,10 @@ process_limit (const pam_handle_t *pamh, int source, const char *lim_type,
        limit_item = RLIMIT_NOFILE;
     else if (strcmp(lim_item, "memlock") == 0)
        limit_item = RLIMIT_MEMLOCK;
+#ifdef RLIMIT_AS
     else if (strcmp(lim_item, "as") == 0)
        limit_item = RLIMIT_AS;
+#endif /*RLIMIT_AS*/
 #ifdef RLIMIT_LOCKS
     else if (strcmp(lim_item, "locks") == 0)
        limit_item = RLIMIT_LOCKS;
@@ -370,8 +618,13 @@ process_limit (const pam_handle_t *pamh, int source, const char *lim_type,
 
     switch(limit_item) {
         case RLIMIT_CPU:
-         if (rlimit_value != RLIM_INFINITY)
-            rlimit_value *= 60;
+         if (rlimit_value != RLIM_INFINITY)
+           {
+             if (rlimit_value >= RLIM_INFINITY/60)
+               rlimit_value = RLIM_INFINITY;
+             else
+               rlimit_value *= 60;
+           }
          break;
         case RLIMIT_FSIZE:
         case RLIMIT_DATA:
@@ -379,17 +632,26 @@ process_limit (const pam_handle_t *pamh, int source, const char *lim_type,
         case RLIMIT_CORE:
         case RLIMIT_RSS:
         case RLIMIT_MEMLOCK:
+#ifdef RLIMIT_AS
         case RLIMIT_AS:
+#endif
          if (rlimit_value != RLIM_INFINITY)
-            rlimit_value *= 1024;
-        break;
+          {
+            if (rlimit_value >= RLIM_INFINITY/1024)
+              rlimit_value = RLIM_INFINITY;
+            else
+              rlimit_value *= 1024;
+          }
+        break;
 #ifdef RLIMIT_NICE
        case RLIMIT_NICE:
         if (int_value > 19)
            int_value = 19;
-        rlimit_value = 19 - int_value;
-#endif
+        if (int_value < -20)
+          int_value = -20;
+        rlimit_value = 20 - int_value;
          break;
+#endif
     }
 
     if ( (limit_item != LIMIT_LOGIN)
@@ -422,19 +684,67 @@ process_limit (const pam_handle_t *pamh, int source, const char *lim_type,
                } else {
                    pl->login_limit = int_value;
                    pl->login_limit_def = source;
-               }
+               }
        }
     }
     return;
 }
 
-static int parse_config_file(pam_handle_t *pamh, const char *uname, int ctrl,
-                            struct pam_limit_s *pl)
+static int
+parse_uid_range(pam_handle_t *pamh, const char *domain,
+               uid_t *min_uid, uid_t *max_uid)
+{
+    const char *range = domain;
+    char *pmax;
+    char *endptr;
+    int rv = LIMIT_RANGE_MM;
+
+    if ((pmax=strchr(range, ':')) == NULL)
+       return LIMIT_RANGE_NONE;
+    ++pmax;
+
+    if (range[0] == '@' || range[0] == '%')
+       ++range;
+
+    if (range[0] == ':')
+       rv = LIMIT_RANGE_ONE;
+    else {
+           errno = 0;
+           *min_uid = strtoul (range, &endptr, 10);
+           if (errno != 0 || (range == endptr) || *endptr != ':') {
+               pam_syslog(pamh, LOG_DEBUG,
+                          "wrong min_uid/gid value in '%s'", domain);
+               return LIMIT_RANGE_ERR;
+           }
+    }
+
+    if (*pmax == '\0') {
+       if (rv == LIMIT_RANGE_ONE)
+           return LIMIT_RANGE_ERR;
+       else
+           return LIMIT_RANGE_MIN;
+    }
+
+    errno = 0;
+    *max_uid = strtoul (pmax, &endptr, 10);
+    if (errno != 0 || (pmax == endptr) || *endptr != '\0') {
+       pam_syslog(pamh, LOG_DEBUG,
+                  "wrong max_uid/gid value in '%s'", domain);
+       return LIMIT_RANGE_ERR;
+    }
+
+    if (rv == LIMIT_RANGE_ONE)
+       *min_uid = *max_uid;
+    return rv;
+}
+
+static int
+parse_config_file(pam_handle_t *pamh, const char *uname, uid_t uid, gid_t gid,
+                            int ctrl, struct pam_limit_s *pl)
 {
     FILE *fil;
     char buf[LINE_LENGTH];
 
-#define CONF_FILE (pl->conf_file[0])?pl->conf_file:LIMITS_FILE
     /* check for the LIMITS_FILE */
     if (ctrl & PAM_DEBUG_ARG)
         pam_syslog(pamh, LOG_DEBUG, "reading settings from '%s'", CONF_FILE);
@@ -444,10 +754,7 @@ static int parse_config_file(pam_handle_t *pamh, const char *uname, int ctrl,
                    "cannot read settings from %s: %m", CONF_FILE);
         return PAM_SERVICE_ERR;
     }
-#undef CONF_FILE
 
-    /* init things */
-    memset(buf, 0, sizeof(buf));
     /* start the show */
     while (fgets(buf, LINE_LENGTH, fil) != NULL) {
         char domain[LINE_LENGTH];
@@ -455,93 +762,180 @@ static int parse_config_file(pam_handle_t *pamh, const char *uname, int ctrl,
         char item[LINE_LENGTH];
         char value[LINE_LENGTH];
         int i;
+        int rngtype;
         size_t j;
-        char *tptr;
+        char *tptr,*line;
+        uid_t min_uid = (uid_t)-1, max_uid = (uid_t)-1;
 
-        tptr = buf;
+        line = buf;
         /* skip the leading white space */
-        while (*tptr && isspace(*tptr))
-            tptr++;
-        strncpy(buf, tptr, sizeof(buf)-1);
-       buf[sizeof(buf)-1] = '\0';
+        while (*line && isspace(*line))
+            line++;
 
         /* Rip off the comments */
-        tptr = strchr(buf,'#');
+        tptr = strchr(line,'#');
         if (tptr)
             *tptr = '\0';
         /* Rip off the newline char */
-        tptr = strchr(buf,'\n');
+        tptr = strchr(line,'\n');
         if (tptr)
             *tptr = '\0';
         /* Anything left ? */
-        if (!strlen(buf)) {
-            memset(buf, 0, sizeof(buf));
+        if (!strlen(line))
             continue;
-        }
 
-        memset(domain, 0, sizeof(domain));
-        memset(ltype, 0, sizeof(ltype));
-        memset(item, 0, sizeof(item));
-        memset(value, 0, sizeof(value));
+       domain[0] = ltype[0] = item[0] = value[0] = '\0';
 
-        i = sscanf(buf,"%s%s%s%s", domain, ltype, item, value);
+       i = sscanf(line,"%s%s%s%s", domain, ltype, item, value);
        D(("scanned line[%d]: domain[%s], ltype[%s], item[%s], value[%s]",
           i, domain, ltype, item, value));
 
         for(j=0; j < strlen(ltype); j++)
             ltype[j]=tolower(ltype[j]);
-        for(j=0; j < strlen(item); j++)
-            item[j]=tolower(item[j]);
-        for(j=0; j < strlen(value); j++)
-            value[j]=tolower(value[j]);
+
+       if ((rngtype=parse_uid_range(pamh, domain, &min_uid, &max_uid)) < 0) {
+           pam_syslog(pamh, LOG_WARNING, "invalid uid range '%s' - skipped", domain);
+           continue;
+       }
 
         if (i == 4) { /* a complete line */
+           for(j=0; j < strlen(item); j++)
+               item[j]=tolower(item[j]);
+           for(j=0; j < strlen(value); j++)
+               value[j]=tolower(value[j]);
+
             if (strcmp(uname, domain) == 0) /* this user have a limit */
                 process_limit(pamh, LIMITS_DEF_USER, ltype, item, value, ctrl, pl);
             else if (domain[0]=='@') {
-                   if (ctrl & PAM_DEBUG_ARG) {
+               if (ctrl & PAM_DEBUG_ARG) {
                        pam_syslog(pamh, LOG_DEBUG,
                                   "checking if %s is in group %s",
                                   uname, domain + 1);
-                   }
-                if (pam_modutil_user_in_group_nam_nam(pamh, uname, domain+1))
-                    process_limit(pamh, LIMITS_DEF_GROUP, ltype, item, value, ctrl,
+               }
+               switch(rngtype) {
+                   case LIMIT_RANGE_NONE:
+                       if (pam_modutil_user_in_group_nam_nam(pamh, uname, domain+1))
+                           process_limit(pamh, LIMITS_DEF_GROUP, ltype, item, value, ctrl,
+                                         pl);
+                       break;
+                   case LIMIT_RANGE_ONE:
+                       if (pam_modutil_user_in_group_nam_gid(pamh, uname, (gid_t)max_uid))
+                           process_limit(pamh, LIMITS_DEF_GROUP, ltype, item, value, ctrl,
                                  pl);
+                       break;
+                   case LIMIT_RANGE_MM:
+                       if (gid > (gid_t)max_uid)
+                           break;
+                       /* fallthrough */
+                   case LIMIT_RANGE_MIN:
+                       if (gid >= (gid_t)min_uid)
+                           process_limit(pamh, LIMITS_DEF_GROUP, ltype, item, value, ctrl,
+                                         pl);
+               }
             } else if (domain[0]=='%') {
-                   if (ctrl & PAM_DEBUG_ARG) {
+               if (ctrl & PAM_DEBUG_ARG) {
                        pam_syslog(pamh, LOG_DEBUG,
                                   "checking if %s is in group %s",
                                   uname, domain + 1);
-                   }
-               if (strcmp(domain,"%") == 0)
-                   process_limit(pamh, LIMITS_DEF_ALL, ltype, item, value, ctrl,
-                                 pl);
-               else if (pam_modutil_user_in_group_nam_nam(pamh, uname, domain+1)) {
-                   strcpy(pl->login_group, domain+1);
-                    process_limit(pamh, LIMITS_DEF_ALLGROUP, ltype, item, value, ctrl,
-                                 pl);
                }
-            } else if (strcmp(domain, "*") == 0)
-                process_limit(pamh, LIMITS_DEF_DEFAULT, ltype, item, value, ctrl,
-                             pl);
+               switch(rngtype) {
+                   case LIMIT_RANGE_NONE:
+                       if (strcmp(domain,"%") == 0)
+                           process_limit(pamh, LIMITS_DEF_ALL, ltype, item, value, ctrl,
+                                         pl);
+                       else if (pam_modutil_user_in_group_nam_nam(pamh, uname, domain+1)) {
+                           strcpy(pl->login_group, domain+1);
+                           process_limit(pamh, LIMITS_DEF_ALLGROUP, ltype, item, value, ctrl,
+                                         pl);
+                       }
+                       break;
+                   case LIMIT_RANGE_ONE:
+                       if (pam_modutil_user_in_group_nam_gid(pamh, uname, (gid_t)max_uid)) {
+                           struct group *grp;
+                           grp = pam_modutil_getgrgid(pamh, (gid_t)max_uid);
+                           strncpy(pl->login_group, grp->gr_name, sizeof(pl->login_group));
+                           pl->login_group[sizeof(pl->login_group)-1] = '\0';
+                           process_limit(pamh, LIMITS_DEF_ALLGROUP, ltype, item, value, ctrl,
+                                         pl);
+                       }
+                       break;
+                   case LIMIT_RANGE_MIN:
+                   case LIMIT_RANGE_MM:
+                       pam_syslog(pamh, LOG_WARNING, "range unsupported for %%group matching - ignored");
+               }
+            } else {
+               switch(rngtype) {
+                   case LIMIT_RANGE_NONE:
+                       if (strcmp(domain, "*") == 0)
+                           process_limit(pamh, LIMITS_DEF_DEFAULT, ltype, item, value, ctrl,
+                                         pl);
+                       break;
+                   case LIMIT_RANGE_ONE:
+                       if (uid != max_uid)
+                           break;
+                       /* fallthrough */
+                   case LIMIT_RANGE_MM:
+                       if (uid > max_uid)
+                           break;
+                       /* fallthrough */
+                   case LIMIT_RANGE_MIN:
+                       if (uid >= min_uid)
+                           process_limit(pamh, LIMITS_DEF_USER, ltype, item, value, ctrl, pl);
+               }
+           }
        } else if (i == 2 && ltype[0] == '-') { /* Probably a no-limit line */
            if (strcmp(uname, domain) == 0) {
                if (ctrl & PAM_DEBUG_ARG) {
                    pam_syslog(pamh, LOG_DEBUG, "no limits for '%s'", uname);
                }
-               fclose(fil);
-               return PAM_IGNORE;
-           } else if (domain[0] == '@' && pam_modutil_user_in_group_nam_nam(pamh, uname, domain+1)) {
+           } else if (domain[0] == '@') {
+               switch(rngtype) {
+                   case LIMIT_RANGE_NONE:
+                       if (!pam_modutil_user_in_group_nam_nam(pamh, uname, domain+1))
+                           continue; /* next line */
+                       break;
+                   case LIMIT_RANGE_ONE:
+                       if (!pam_modutil_user_in_group_nam_gid(pamh, uname, (gid_t)max_uid))
+                           continue; /* next line */
+                       break;
+                   case LIMIT_RANGE_MM:
+                       if (gid > (gid_t)max_uid)
+                           continue;  /* next line */
+                       /* fallthrough */
+                   case LIMIT_RANGE_MIN:
+                       if (gid < (gid_t)min_uid)
+                           continue;  /* next line */
+               }
                if (ctrl & PAM_DEBUG_ARG) {
                    pam_syslog(pamh, LOG_DEBUG,
                               "no limits for '%s' in group '%s'",
                               uname, domain+1);
                }
-               fclose(fil);
-               return PAM_IGNORE;
+           } else {
+               switch(rngtype) {
+                   case LIMIT_RANGE_NONE:
+                       continue;  /* next line */
+                   case LIMIT_RANGE_ONE:
+                       if (uid != max_uid)
+                           continue;  /* next line */
+                       break;
+                   case LIMIT_RANGE_MM:
+                       if (uid > max_uid)
+                           continue;  /* next line */
+                       /* fallthrough */
+                   case LIMIT_RANGE_MIN:
+                       if (uid >= min_uid)
+                           break;
+                       continue;  /* next line */
+               }
+               if (ctrl & PAM_DEBUG_ARG) {
+                   pam_syslog(pamh, LOG_DEBUG, "no limits for '%s'", uname);
+               }
            }
+           fclose(fil);
+           return PAM_IGNORE;
         } else {
-            pam_syslog(pamh, LOG_WARNING, "invalid line '%s' - skipped", buf);
+            pam_syslog(pamh, LOG_WARNING, "invalid line '%s' - skipped", line);
        }
     }
     fclose(fil);
@@ -557,6 +951,8 @@ static int setup_limits(pam_handle_t *pamh,
     int retval = LIMITED_OK;
 
     for (i=0, status=LIMITED_OK; i<RLIM_NLIMITS; i++) {
+      int res;
+
        if (!pl->limits[i].supported) {
            /* skip it if its not known to the system */
            continue;
@@ -568,7 +964,11 @@ static int setup_limits(pam_handle_t *pamh,
        }
         if (pl->limits[i].limit.rlim_cur > pl->limits[i].limit.rlim_max)
             pl->limits[i].limit.rlim_cur = pl->limits[i].limit.rlim_max;
-       status |= setrlimit(i, &pl->limits[i].limit);
+       res = setrlimit(i, &pl->limits[i].limit);
+       if (res != 0)
+         pam_syslog(pamh, LOG_ERR, "Could not set limit for '%s': %m",
+                    rlimit2str(i));
+       status |= res;
     }
 
     if (status) {
@@ -577,6 +977,7 @@ static int setup_limits(pam_handle_t *pamh,
 
     status = setpriority(PRIO_PROCESS, 0, pl->priority);
     if (status != 0) {
+        pam_syslog(pamh, LOG_ERR, "Could not set limit for PRIO_PROCESS: %m");
         retval = LIMIT_ERR;
     }
 
@@ -584,6 +985,13 @@ static int setup_limits(pam_handle_t *pamh,
        D(("skip login limit check for uid=0"));
     } else if (pl->login_limit > 0) {
         if (check_logins(pamh, uname, pl->login_limit, ctrl, pl) == LOGIN_ERR) {
+#ifdef HAVE_LIBAUDIT
+           if (!(ctrl & PAM_NO_AUDIT)) {
+               pam_modutil_audit_write(pamh, AUDIT_ANOM_LOGIN_SESSIONS,
+                   "pam_limits", PAM_PERM_DENIED);
+               /* ignore return value as we fail anyway */
+            }
+#endif
             retval |= LOGIN_ERR;
        }
     } else if (pl->login_limit == 0) {
@@ -594,24 +1002,30 @@ static int setup_limits(pam_handle_t *pamh,
 }
 
 /* now the session stuff */
-PAM_EXTERN int
+int
 pam_sm_open_session (pam_handle_t *pamh, int flags UNUSED,
                     int argc, const char **argv)
 {
     int retval;
+    int i;
+    int glob_rc;
     char *user_name;
     struct passwd *pwd;
     int ctrl;
-    struct pam_limit_s pl;
+    struct pam_limit_s plstruct;
+    struct pam_limit_s *pl = &plstruct;
+    glob_t globbuf;
+    const char *oldlocale;
 
     D(("called."));
 
-    memset(&pl, 0, sizeof(pl));
+    memset(pl, 0, sizeof(*pl));
+    memset(&globbuf, 0, sizeof(globbuf));
 
-    ctrl = _pam_parse(pamh, argc, argv, &pl);
+    ctrl = _pam_parse(pamh, argc, argv, pl);
     retval = pam_get_item( pamh, PAM_USER, (void*) &user_name );
     if ( user_name == NULL || retval != PAM_SUCCESS ) {
-        pam_syslog(pamh, LOG_CRIT, "open_session - error recovering username");
+        pam_syslog(pamh, LOG_ERR, "open_session - error recovering username");
         return PAM_SESSION_ERR;
      }
 
@@ -623,28 +1037,59 @@ pam_sm_open_session (pam_handle_t *pamh, int flags UNUSED,
         return PAM_USER_UNKNOWN;
     }
 
-    retval = init_limits(&pl);
+    retval = init_limits(pamh, pl, ctrl);
     if (retval != PAM_SUCCESS) {
-        pam_syslog(pamh, LOG_WARNING, "cannot initialize");
+        pam_syslog(pamh, LOG_ERR, "cannot initialize");
         return PAM_ABORT;
     }
 
-    retval = parse_config_file(pamh, pwd->pw_name, ctrl, &pl);
+    retval = parse_config_file(pamh, pwd->pw_name, pwd->pw_uid, pwd->pw_gid, ctrl, pl);
     if (retval == PAM_IGNORE) {
-       D(("the configuration file has an applicable '<domain> -' entry"));
+       D(("the configuration file ('%s') has an applicable '<domain> -' entry", CONF_FILE));
        return PAM_SUCCESS;
     }
-    if (retval != PAM_SUCCESS) {
-        pam_syslog(pamh, LOG_WARNING, "error parsing the configuration file");
-        return retval;
+    if (retval != PAM_SUCCESS || pl->conf_file != NULL)
+       /* skip reading limits.d if config file explicitely specified */
+       goto out;
+
+    /* Read subsequent *.conf files, if they exist. */
+
+    /* set the LC_COLLATE so the sorting order doesn't depend
+       on system locale */
+
+    oldlocale = setlocale(LC_COLLATE, "C");
+    glob_rc = glob(LIMITS_CONF_GLOB, GLOB_ERR, NULL, &globbuf);
+
+    if (oldlocale != NULL)
+       setlocale (LC_COLLATE, oldlocale);
+
+    if (!glob_rc) {
+       /* Parse the *.conf files. */
+       for (i = 0; globbuf.gl_pathv[i] != NULL; i++) {
+           pl->conf_file = globbuf.gl_pathv[i];
+           retval = parse_config_file(pamh, pwd->pw_name, pwd->pw_uid, pwd->pw_gid, ctrl, pl);
+           if (retval == PAM_IGNORE) {
+               D(("the configuration file ('%s') has an applicable '<domain> -' entry", pl->conf_file));
+               globfree(&globbuf);
+               return PAM_SUCCESS;
+           }
+           if (retval != PAM_SUCCESS)
+               goto out;
+        }
     }
 
-    if (ctrl & PAM_DO_SETREUID) {
-       setreuid(pwd->pw_uid, -1);
+out:
+    globfree(&globbuf);
+    if (retval != PAM_SUCCESS)
+    {
+       pam_syslog(pamh, LOG_ERR, "error parsing the configuration file: '%s' ",CONF_FILE);
+       return retval;
     }
-    retval = setup_limits(pamh, pwd->pw_name, pwd->pw_uid, ctrl, &pl);
+
+    retval = setup_limits(pamh, pwd->pw_name, pwd->pw_uid, ctrl, pl);
     if (retval & LOGIN_ERR)
-       pam_error(pamh, _("Too many logins for '%s'."), pwd->pw_name);
+       pam_error(pamh, _("There were too many logins for '%s'."),
+                 pwd->pw_name);
     if (retval != LIMITED_OK) {
         return PAM_PERM_DENIED;
     }
@@ -652,7 +1097,7 @@ pam_sm_open_session (pam_handle_t *pamh, int flags UNUSED,
     return PAM_SUCCESS;
 }
 
-PAM_EXTERN int
+int
 pam_sm_close_session (pam_handle_t *pamh UNUSED, int flags UNUSED,
                      int argc UNUSED, const char **argv UNUSED)
 {
@@ -660,21 +1105,6 @@ pam_sm_close_session (pam_handle_t *pamh UNUSED, int flags UNUSED,
      return PAM_SUCCESS;
 }
 
-#ifdef PAM_STATIC
-
-/* static module data */
-
-struct pam_module _pam_limits_modstruct = {
-     "pam_limits",
-     NULL,
-     NULL,
-     NULL,
-     pam_sm_open_session,
-     pam_sm_close_session,
-     NULL
-};
-#endif
-
 /*
  * Copyright (c) Cristian Gafton, 1996-1997, <gafton@redhat.com>
  *                                              All rights reserved.