emo/shadow
1
0
Fork 0
Code Issues Pull Requests Packages Releases Activity

* src/faillog.c: Remove function calls from within assert().

Browse Source
This commit is contained in:
nekral-guest 2009-04-24 22:27:09 +00:00
parent f28f5f3af4
commit 37eec13774
2 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ChangeLog
View File

@ -1,3 +1,7 @@
2009-04-25 Nicolas François <nicolas.francois@centraliens.net>
* src/faillog.c: Remove function calls from within assert().
2009-04-25 Nicolas François <nicolas.francois@centraliens.net> 2009-04-25 Nicolas François <nicolas.francois@centraliens.net>
* libmisc/mail.c: Ignore the return value of puts(). * libmisc/mail.c: Ignore the return value of puts().

12
src/faillog.c
View File

@ -105,7 +105,8 @@ static void print_one (/*@null@*/const struct passwd *pw, bool force)
offset = pw->pw_uid * sizeof (fl); offset = pw->pw_uid * sizeof (fl);
if (offset <= (statbuf.st_size - sizeof (fl))) { if (offset <= (statbuf.st_size - sizeof (fl))) {
/* fseeko errors are not really relevant for us. */ /* fseeko errors are not really relevant for us. */
assert ( fseeko (fail, offset, SEEK_SET) == 0 ); int err = fseeko (fail, offset, SEEK_SET);
assert (0 == err);
/* faillog is a sparse file. Even if no entries were /* faillog is a sparse file. Even if no entries were
* entered for this user, which should be able to get the * entered for this user, which should be able to get the
* empty entry in this case. * empty entry in this case.
@ -204,7 +205,8 @@ static bool reset_one (uid_t uid)
offset = uid * sizeof (fl); offset = uid * sizeof (fl);
if (offset <= (statbuf.st_size - sizeof (fl))) { if (offset <= (statbuf.st_size - sizeof (fl))) {
/* fseeko errors are not really relevant for us. */ /* fseeko errors are not really relevant for us. */
assert ( fseeko (fail, offset, SEEK_SET) == 0 ); int err = fseeko (fail, offset, SEEK_SET);
assert (0 == err);
/* faillog is a sparse file. Even if no entries were /* faillog is a sparse file. Even if no entries were
* entered for this user, which should be able to get the * entered for this user, which should be able to get the
* empty entry in this case. * empty entry in this case.
@ -290,7 +292,8 @@ static bool setmax_one (uid_t uid, int max)
offset = (off_t) uid * sizeof (fl); offset = (off_t) uid * sizeof (fl);
if (offset <= (statbuf.st_size - sizeof (fl))) { if (offset <= (statbuf.st_size - sizeof (fl))) {
/* fseeko errors are not really relevant for us. */ /* fseeko errors are not really relevant for us. */
assert ( fseeko (fail, offset, SEEK_SET) == 0 ); int err = fseeko (fail, offset, SEEK_SET);
assert (0 == err);
/* faillog is a sparse file. Even if no entries were /* faillog is a sparse file. Even if no entries were
* entered for this user, which should be able to get the * entered for this user, which should be able to get the
* empty entry in this case. * empty entry in this case.
@ -380,7 +383,8 @@ static bool set_locktime_one (uid_t uid, long locktime)
offset = (off_t) uid * sizeof (fl); offset = (off_t) uid * sizeof (fl);
if (offset <= (statbuf.st_size - sizeof (fl))) { if (offset <= (statbuf.st_size - sizeof (fl))) {
/* fseeko errors are not really relevant for us. */ /* fseeko errors are not really relevant for us. */
assert ( fseeko (fail, offset, SEEK_SET) == 0 ); int err = fseeko (fail, offset, SEEK_SET);
assert (0 == err);
/* faillog is a sparse file. Even if no entries were /* faillog is a sparse file. Even if no entries were
* entered for this user, which should be able to get the * entered for this user, which should be able to get the
* empty entry in this case. * empty entry in this case.