diff options
author | Ulrich Drepper <drepper@redhat.com> | 2008-08-14 04:23:40 +0000 |
---|---|---|
committer | Ulrich Drepper <drepper@redhat.com> | 2008-08-14 04:23:40 +0000 |
commit | 1bfa05cfaa4177476fd57851d867c5d1a276a34b (patch) | |
tree | e4c8f50704c119835b459e3d515489ae94692f7c | |
parent | c5671698c8e953513a3456dcb60344bd2e7b1125 (diff) | |
download | glibc-1bfa05cfaa4177476fd57851d867c5d1a276a34b.zip glibc-1bfa05cfaa4177476fd57851d867c5d1a276a34b.tar.gz glibc-1bfa05cfaa4177476fd57851d867c5d1a276a34b.tar.bz2 |
[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.
-rw-r--r-- | ChangeLog | 9 | ||||
-rw-r--r-- | login/utmp_file.c | 25 |
2 files changed, 30 insertions, 4 deletions
@@ -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] diff --git a/login/utmp_file.c b/login/utmp_file.c index c0bd229..9033f72 100644 --- a/login/utmp_file.c +++ b/login/utmp_file.c @@ -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) { |