mirror of
https://sourceware.org/git/glibc.git
synced 2024-11-24 14:00:30 +00:00
[BZ #6634]
2008-08-13 Ulrich Drepper <drepper@redhat.com> [BZ #6634] * login/utmp_file.c (getutent_r_file): Take additional parameter. Set to true if locking failed. (getutid_r_file): Adjust caller. (pututline_file): Likewise. Return NULL in this case. Patch mostly by halesh.s@gmail.com.
This commit is contained in:
parent
c5671698c8
commit
1bfa05cfaa
@ -1,3 +1,12 @@
|
||||
2008-08-13 Ulrich Drepper <drepper@redhat.com>
|
||||
|
||||
[BZ #6634]
|
||||
* login/utmp_file.c (getutent_r_file): Take additional parameter.
|
||||
Set to true if locking failed.
|
||||
(getutid_r_file): Adjust caller.
|
||||
(pututline_file): Likewise. Return NULL in this case.
|
||||
Patch mostly by halesh.s@gmail.com.
|
||||
|
||||
2008-08-12 Ulrich Drepper <drepper@redhat.com>
|
||||
|
||||
[BZ #6589]
|
||||
|
@ -22,6 +22,7 @@
|
||||
#include <errno.h>
|
||||
#include <fcntl.h>
|
||||
#include <signal.h>
|
||||
#include <stdbool.h>
|
||||
#include <stdio.h>
|
||||
#include <string.h>
|
||||
#include <unistd.h>
|
||||
@ -244,12 +245,16 @@ getutent_r_file (struct utmp *buffer, struct utmp **result)
|
||||
|
||||
|
||||
static int
|
||||
internal_getut_r (const struct utmp *id, struct utmp *buffer)
|
||||
internal_getut_r (const struct utmp *id, struct utmp *buffer,
|
||||
bool *lock_failed)
|
||||
{
|
||||
int result = -1;
|
||||
|
||||
LOCK_FILE (file_fd, F_RDLCK)
|
||||
LOCKING_FAILED ();
|
||||
{
|
||||
*lock_failed = true;
|
||||
LOCKING_FAILED ();
|
||||
}
|
||||
|
||||
#if _HAVE_UT_TYPE - 0
|
||||
if (id->ut_type == RUN_LVL || id->ut_type == BOOT_TIME
|
||||
@ -320,7 +325,10 @@ getutid_r_file (const struct utmp *id, struct utmp *buffer,
|
||||
return -1;
|
||||
}
|
||||
|
||||
if (internal_getut_r (id, &last_entry) < 0)
|
||||
/* We don't have to distinguish whether we can lock the file or
|
||||
whether there is no entry. */
|
||||
bool lock_failed = false;
|
||||
if (internal_getut_r (id, &last_entry, &lock_failed) < 0)
|
||||
{
|
||||
*result = NULL;
|
||||
return -1;
|
||||
@ -410,7 +418,16 @@ pututline_file (const struct utmp *data)
|
||||
__utmp_equal (&last_entry, data)))
|
||||
found = 1;
|
||||
else
|
||||
found = internal_getut_r (data, &buffer);
|
||||
{
|
||||
bool lock_failed = false;
|
||||
found = internal_getut_r (data, &buffer, &lock_failed);
|
||||
|
||||
if (__builtin_expect (lock_failed, false))
|
||||
{
|
||||
__set_errno (EAGAIN);
|
||||
return NULL;
|
||||
}
|
||||
}
|
||||
|
||||
LOCK_FILE (file_fd, F_WRLCK)
|
||||
{
|
||||
|
Loading…
Reference in New Issue
Block a user