VesperOS/shadow
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

libmisc: call active_sessions_count()

Browse Source 
Replace the utmp dependent code with the call to
`active_sessions_count()`.

Resolves: https://github.com/shadow-maint/shadow/issues/674

Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com>
This commit is contained in:
Iker Pedrosa
2023-07-19 12:02:31 +02:00
committed by Serge Hallyn
parent ede0665a5a
commit fb8f44d73f
1 changed files with 2 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
libmisc/limits.c
View File

@@ -119,7 +119,6 @@ static int set_umask (const char *value)
/* Counts the number of user logins and check against the limit */
static int check_logins (const char *name, const char *maxlogins)
{
struct utmp *ut;
unsigned long limit, count;
if (getulong (maxlogins, &limit) == 0) {
@@ -131,29 +130,8 @@ static int check_logins (const char *name, const char *maxlogins)
return LOGIN_ERROR_LOGIN;
}
count = 0;
setutent ();
while ((ut = getutent ()))
{
if (USER_PROCESS != ut->ut_type) {
continue;
}
if ('\0' == ut->ut_user[0]) {
continue;
}
if (strncmp (name, ut->ut_user, sizeof (ut->ut_user)) != 0) {
continue;
}
count++;
if (count > limit) {
break;
}
}
endutent ();
/*
* This is called after setutmp(), so the number of logins counted
* includes the user who is currently trying to log in.
*/
count = active_sessions_count(name, limit);
if (count > limit) {
SYSLOG ((LOG_WARN,
"Too many logins (max %lu) for %s\n",