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

* src/faillog.c: Added splint annotations.

Browse Source 
* src/faillog.c: Cast ID to ulongs and use ulong formats for IDs.
	* src/faillog.c: Ignore fflush() return value.
	* src/faillog.c: Added parenthesis.
This commit is contained in:
nekral-guest 2009-04-23 11:23:53 +00:00
parent 988ec76cf8
commit 7b562d96b3
2 changed files with 26 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ChangeLog
View File

@ -1,3 +1,10 @@
2009-04-22 Nicolas François <nicolas.francois@centraliens.net>
* src/faillog.c: Added splint annotations.
* src/faillog.c: Cast ID to ulongs and use ulong formats for IDs.
* src/faillog.c: Ignore fflush() return value.
* src/faillog.c: Added parenthesis.
2009-04-22 Nicolas François <nicolas.francois@centraliens.net> 2009-04-22 Nicolas François <nicolas.francois@centraliens.net>
* src/grpck.c: Ignore puts return value. * src/grpck.c: Ignore puts return value.

38
src/faillog.c
View File

@ -85,7 +85,7 @@ static void usage (void)
exit (E_USAGE); exit (E_USAGE);
} }
static void print_one (const struct passwd *pw, bool force) static void print_one (/*@null@*/const struct passwd *pw, bool force)
{ {
static bool once = false; static bool once = false;
struct tm *tm; struct tm *tm;
@ -112,8 +112,8 @@ static void print_one (const struct passwd *pw, bool force)
*/ */
if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) { if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) {
fprintf (stderr, fprintf (stderr,
_("faillog: Failed to get the entry for UID %d\n"), _("faillog: Failed to get the entry for UID %lu\n"),
pw->pw_uid); (unsigned long int)pw->pw_uid);
return; return;
} }
} else { } else {
@ -211,8 +211,8 @@ static bool reset_one (uid_t uid)
*/ */
if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) { if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) {
fprintf (stderr, fprintf (stderr,
_("faillog: Failed to get the entry for UID %d\n"), _("faillog: Failed to get the entry for UID %lu\n"),
uid); (unsigned long int)uid);
return true; return true;
} }
} else { } else {
@ -241,8 +241,8 @@ static bool reset_one (uid_t uid)
} }
fprintf (stderr, fprintf (stderr,
_("faillog: Failed to reset fail count for UID %d\n"), _("faillog: Failed to reset fail count for UID %lu\n"),
uid); (unsigned long int)uid);
return true; return true;
} }
@ -297,8 +297,8 @@ static bool setmax_one (uid_t uid, int max)
*/ */
if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) { if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) {
fprintf (stderr, fprintf (stderr,
_("faillog: Failed to get the entry for UID %d\n"), _("faillog: Failed to get the entry for UID %lu\n"),
uid); (unsigned long int)uid);
return true; return true;
} }
} else { } else {
@ -323,13 +323,13 @@ static bool setmax_one (uid_t uid, int max)
if ( (fseeko (fail, offset, SEEK_SET) == 0) if ( (fseeko (fail, offset, SEEK_SET) == 0)
&& (fwrite ((char *) &fl, sizeof (fl), 1, fail) == 1)) { && (fwrite ((char *) &fl, sizeof (fl), 1, fail) == 1)) {
fflush (fail); (void) fflush (fail);
return false; return false;
} }
fprintf (stderr, fprintf (stderr,
_("faillog: Failed to set max for UID %d\n"), _("faillog: Failed to set max for UID %lu\n"),
uid); (unsigned long int)uid);
return true; return true;
} }
@ -387,8 +387,8 @@ static bool set_locktime_one (uid_t uid, long locktime)
*/ */
if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) { if (fread ((char *) &fl, sizeof (fl), 1, fail) != 1) {
fprintf (stderr, fprintf (stderr,
_("faillog: Failed to get the entry for UID %d\n"), _("faillog: Failed to get the entry for UID %lu\n"),
uid); (unsigned long int)uid);
return true; return true;
} }
} else { } else {
@ -411,15 +411,15 @@ static bool set_locktime_one (uid_t uid, long locktime)
fl.fail_locktime = locktime; fl.fail_locktime = locktime;
if (fseeko (fail, offset, SEEK_SET) == 0 if ( (fseeko (fail, offset, SEEK_SET) == 0)
&& fwrite ((char *) &fl, sizeof (fl), 1, fail) == 1) { && (fwrite ((char *) &fl, sizeof (fl), 1, fail) == 1)) {
fflush (fail); (void) fflush (fail);
return false; return false;
} }
fprintf (stderr, fprintf (stderr,
_("faillog: Failed to set locktime for UID %d\n"), _("faillog: Failed to set locktime for UID %lu\n"),
uid); (unsigned long int)uid);
return true; return true;
} }